2.2.x cannot convert absolute path with files option

[yhatt]

Hi. I'm using dir-glob through globby in my project.

I found a regression in the case of passing absolute path with files option. It works correctly in v2.0.0, but v2.2.1 does not return globbed path.

const dirGlob = require('dir-glob')
dirGlob(['/app/tmp'], { files: ['*.txt'] }).then(globs => console.log(globs))

// dir-glob@2.0.0 returns ['/app/tmp/**/*.txt']
// dir-glob@2.2.1 returns ['/app/tmp/']

See also RunKit: https://runkit.com/yhatt/5c44923f09b0c40012f808d1

Perhaps it might come from updated getPath.

dir-glob/index.js

Line 6 in 4fc6c6e

const getPath = (filepath, cwd) => path.join(cwd, filepath[0] === '!' ? filepath.slice(1) : filepath);

path.join() returns just concatenated path, and not consider absolute path.

join('/foo/bar', '/foo') // => "/foo/bar/foo"

Apparently, tests did not cover the case of passing absolute path. I want to work on it.

[kevva]

Thanks for reporting @yhatt. It was indeed a regression in the getPath function. I added a check for absolute paths which ignores joining them with cwd.

Did a release with the fix (2.2.2).

[yhatt]

Thank you so much! I've confirmed to work it with absolute path, and my project's test have passed by updating dir-glob.