Improve error message for invalid cwd option

Fixes #151

Author: sindresorhus

index.js

@@ -128,14 +128,16 @@ const checkCwdOption = (cwd, fsImplementation = fs) => {
 		return;
 	}
 
+	let stats;
 	try {
-		if (!fsImplementation.statSync(cwd).isDirectory()) {
-			throw new Error('The `cwd` option must be a path to a directory');
-		}
-	} catch (error) {
-		if (error.message === 'The `cwd` option must be a path to a directory') {
-			throw error;
-		}
+		stats = fsImplementation.statSync(cwd);
+	} catch {
+		// If stat fails (e.g., path doesn't exist), let fast-glob handle it
+		return;
+	}
+
+	if (!stats.isDirectory()) {
+		throw new Error(`The \`cwd\` option must be a path to a directory, got: ${cwd}`);
 	}
 };
 

tests/generate-glob-tasks.js

@@ -14,6 +14,8 @@ import {
 	isUnique,
 } from './utilities.js';
 
+const cwdDirectoryError = {message: /The `cwd` option must be a path to a directory, got:/};
+
 const runGenerateGlobTasks = async (t, patterns, options) => {
 	const promiseResult = await generateGlobTasks(patterns, options);
 	const syncResult = generateGlobTasksSync(patterns, options);
@@ -54,12 +56,10 @@ for (const value of invalidPatterns) {
 }
 
 test('throws when specifying a file as cwd', async t => {
-	const error = {message: 'The `cwd` option must be a path to a directory'};
-
 	for (const file of getPathValues(path.resolve('fixtures/gitignore/bar.js'))) {
 		// eslint-disable-next-line no-await-in-loop
-		await t.throwsAsync(generateGlobTasks('*', {cwd: file}), error);
-		t.throws(() => generateGlobTasksSync('*', {cwd: file}), error);
+		await t.throwsAsync(generateGlobTasks('*', {cwd: file}), cwdDirectoryError);
+		t.throws(() => generateGlobTasksSync('*', {cwd: file}), cwdDirectoryError);
 	}
 });
 

tests/globby.js

@@ -23,6 +23,8 @@ import {
 const cwd = process.cwd();
 const temporary = 'tmp';
 
+const cwdDirectoryError = {message: /The `cwd` option must be a path to a directory, got:/};
+
 const fixture = [
 	'a.tmp',
 	'b.tmp',
@@ -704,29 +706,27 @@ test('`{extension: false}` and `expandDirectories.extensions` option', async t =
 });
 
 test('throws when specifying a file as cwd', async t => {
-	const error = {message: 'The `cwd` option must be a path to a directory'};
-
 	for (const file of getPathValues(path.resolve('fixtures/gitignore/bar.js'))) {
 		// eslint-disable-next-line no-await-in-loop
-		await t.throwsAsync(globby('.', {cwd: file}), error);
+		await t.throwsAsync(globby('.', {cwd: file}), cwdDirectoryError);
 		// eslint-disable-next-line no-await-in-loop
-		await t.throwsAsync(globby('*', {cwd: file}), error);
-		t.throws(() => globbySync('.', {cwd: file}), error);
-		t.throws(() => globbySync('*', {cwd: file}), error);
-		t.throws(() => globbyStream('.', {cwd: file}), error);
-		t.throws(() => globbyStream('*', {cwd: file}), error);
+		await t.throwsAsync(globby('*', {cwd: file}), cwdDirectoryError);
+		t.throws(() => globbySync('.', {cwd: file}), cwdDirectoryError);
+		t.throws(() => globbySync('*', {cwd: file}), cwdDirectoryError);
+		t.throws(() => globbyStream('.', {cwd: file}), cwdDirectoryError);
+		t.throws(() => globbyStream('*', {cwd: file}), cwdDirectoryError);
 	}
 });
 
 test('throws when specifying a file as cwd - isDynamicPattern', t => {
 	for (const file of getPathValues(path.resolve('fixtures/gitignore/bar.js'))) {
 		t.throws(() => {
 			isDynamicPattern('.', {cwd: file});
-		}, {message: 'The `cwd` option must be a path to a directory'});
+		}, cwdDirectoryError);
 
 		t.throws(() => {
 			isDynamicPattern('*', {cwd: file});
-		}, {message: 'The `cwd` option must be a path to a directory'});
+		}, cwdDirectoryError);
 	}
 });