Verify passed test path is a file before resolving (#5317)

* Verify passed test path is a file before resolving

* Update CHANGELOG

* Cut one FS operation with try-catch

* Remove jest version number in CHANGELOG

Author: LINKIWI

CHANGELOG.md

@@ -1,5 +1,10 @@
 ## master
 
+### Fixes
+
+* `[jest-cli]` Fix `EISDIR` when a directory is passed as an argument to `jest`.
+  ([#5317](https://github.com/facebook/jest/pull/5317))
+
 ## jest 22.1.0
 
 ### Features

integration-tests/__tests__/__snapshots__/cli-accepts-exact-filenames.test.js.snap

@@ -22,7 +22,7 @@ exports[`CLI accepts exact filenames 2`] = `
 Tests:       2 passed, 2 total
 Snapshots:   0 total
 Time:        <<REPLACED>>
-Ran all test suites matching /.\\\\/bar.js|.\\\\/foo\\\\/baz.js/i.
+Ran all test suites matching /.\\\\/bar.js|.\\\\/foo\\\\/baz.js|.\\\\/foo/i.
 "
 `;
 

integration-tests/__tests__/cli-accepts-exact-filenames.test.js

@@ -38,6 +38,7 @@ test('CLI accepts exact filenames', () => {
     '--forceExit',
     './bar.js',
     './foo/baz.js',
+    './foo',
   ]);
   const {rest, summary} = extractSummary(stderr);
   expect(status).toBe(0);

packages/jest-cli/src/search_source.js

@@ -209,7 +209,15 @@ export default class SearchSource {
     } else if (globalConfig.findRelatedTests && paths && paths.length) {
       return Promise.resolve(this.findRelatedTestsFromPattern(paths));
     } else {
-      const validTestPaths = paths && paths.filter(fs.existsSync);
+      const validTestPaths =
+        paths &&
+        paths.filter(name => {
+          try {
+            return fs.lstatSync(name).isFile();
+          } catch (e) {
+            return false;
+          }
+        });
 
       if (validTestPaths && validTestPaths.length) {
         return Promise.resolve({tests: toTests(this._context, validTestPaths)});