fix: use the basename as the entry name for static patterns
The static reader built the entry from the pattern as is, so `name` and
`dirent.name` were the whole pattern, while the dynamic reader returns
the basename. The same `objectMode: true` option therefore produced a
different entry shape depending on whether the pattern was static:
globSync('fixtures/file.md', { objectMode: true })
// name: 'fixtures/file.md', path: 'fixtures/file.md'
globSync('fixtures/*.md', { objectMode: true })
// name: 'file.md', path: 'fixtures/file.md'
Note that `path` already agreed between the two readers, only `name`
diverged. Use the basename for the entry name and for the dirent built
from the stats, which matches both the dynamic reader and the entry
builder the tests already use.
Fixes #487
This commit is contained in:
1 parent
ed3aca1ad6
commit
e246fef883
3 files changed
+45
-2
No files matched your search
@@ -77,6 +77,30 @@ describe('Readers → Reader', () => {
|
||||
assert.ok(actual.dirent.isFile());
|
||||
});
|
||||
|
||||
it('should return created entry with the basename as name when the pattern has a directory', () => {
|
||||
const reader = getReader();
|
||||
const pattern = 'directory/config.json';
|
||||
const stats = new Stats({ mode: StatsMode.File });
|
||||
|
||||
const actual = reader.makeEntry(stats, pattern);
|
||||
|
||||
assert.strictEqual(actual.name, 'config.json');
|
||||
assert.strictEqual(actual.path, pattern);
|
||||
assert.strictEqual(actual.dirent.name, 'config.json');
|
||||
});
|
||||
|
||||
it('should return created entry with the basename as name when the pattern has a leading dot segment', () => {
|
||||
const reader = getReader();
|
||||
const pattern = './config.json';
|
||||
const stats = new Stats({ mode: StatsMode.File });
|
||||
|
||||
const actual = reader.makeEntry(stats, pattern);
|
||||
|
||||
assert.strictEqual(actual.name, 'config.json');
|
||||
assert.strictEqual(actual.path, pattern);
|
||||
assert.strictEqual(actual.dirent.name, 'config.json');
|
||||
});
|
||||
|
||||
it('should return created entry with fs.Stats', () => {
|
||||
const reader = getReader({ stats: true });
|
||||
const pattern = 'config.json';
|
||||
|
||||
@@ -33,10 +33,15 @@ export abstract class Reader<T> {
|
||||
}
|
||||
|
||||
protected _makeEntry(stats: FsStats, pattern: Pattern): Entry {
|
||||
// A static pattern is the path to the entry, so it is used as the entry path as is.
|
||||
// The name, however, is the basename of that path, the same as for the entries that
|
||||
// the dynamic reader produces, so that both readers return the same shape.
|
||||
const name = path.basename(pattern);
|
||||
|
||||
const entry: Entry = {
|
||||
name: pattern,
|
||||
name,
|
||||
path: pattern,
|
||||
dirent: utils.fs.createDirentFromStats(pattern, stats),
|
||||
dirent: utils.fs.createDirentFromStats(name, stats),
|
||||
};
|
||||
|
||||
if (this.#settings.stats) {
|
||||
|
||||
Reference in new issue
Block a user