diff --git a/src/index.spec.ts b/src/index.spec.ts index 917e292..0882843 100644 --- a/src/index.spec.ts +++ b/src/index.spec.ts @@ -36,6 +36,20 @@ describe('Package', () => { assert.deepStrictEqual(actual, expected); }); + it('should returns entries with the same name for static and dynamic patterns in the object mode', () => { + const [staticEntry] = fg.globSync('fixtures/file.md', { objectMode: true }); + const [dynamicEntry] = fg.globSync('fixtures/*.md', { objectMode: true }); + + assert.strictEqual(staticEntry.name, 'file.md'); + assert.strictEqual(staticEntry.dirent.name, 'file.md'); + assert.strictEqual(staticEntry.path, 'fixtures/file.md'); + + assert.deepStrictEqual( + [staticEntry.name, staticEntry.path, staticEntry.dirent.name], + [dynamicEntry.name, dynamicEntry.path, dynamicEntry.dirent.name], + ); + }); + it('should returns entries (two sources)', () => { const expected: EntryItem[] = [ 'fixtures/first/file.md', diff --git a/src/readers/reader.spec.ts b/src/readers/reader.spec.ts index e0494a8..a29fb95 100644 --- a/src/readers/reader.spec.ts +++ b/src/readers/reader.spec.ts @@ -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'; diff --git a/src/readers/reader.ts b/src/readers/reader.ts index 427ca72..bdf3258 100644 --- a/src/readers/reader.ts +++ b/src/readers/reader.ts @@ -33,10 +33,15 @@ export abstract class Reader { } 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) {