Continuation to #3771 for snake case file naming (#3920)

* Resolved conflicts

* Including eslint file for forcing file names

* renaming files

* renaming files

Author: anilreddykatta

.eslintrc.js

@@ -9,7 +9,7 @@
 module.exports = {
   extends: [
     './packages/eslint-config-fb-strict/index.js',
-    'plugin:import/errors',
+    'plugin:import/errors'
   ],
   parser: 'babel-eslint',
   rules: {
@@ -20,15 +20,16 @@ module.exports = {
     'max-len': 0,
     'no-multiple-empty-lines': 1,
     'import/no-duplicates': 2,
-    'import/no-unresolved': [2, { ignore: ['^types/'] }],
+    'import/no-unresolved': [2, {ignore: ['^types/']}],
     // This has to be disabled until all type and module imports are combined
     // https://github.com/benmosher/eslint-plugin-import/issues/645
     'import/order': 0,
     // These has to be disabled until the whole code base is converted to ESM
     'import/default': 0,
     'import/named': 0,
+    'unicorn/filename-case': [2, {case: 'snakeCase'}]
   },
-  plugins: ['markdown', 'import'],
+  plugins: ['markdown', 'import', 'unicorn'],
   overrides: [
     // to make it more suitable for running on code examples in docs/ folder
     {
@@ -43,22 +44,31 @@ module.exports = {
         'no-unused-vars': 0,
         'consistent-return': 0,
         'import/no-unresolved': 0,
-        'sort-keys': 0
-      },
+        'sort-keys': 0,
+        'unicorn/filename-case': 0
+      }
     },
     {
       files: ['examples/**/*'],
       rules: {
         'babel/func-params-comma-dangle': 0,
         'import/order': 0,
-        'import/no-unresolved': [2, { 'ignore': ['^react-native$'] }]
-      },
+        'import/no-unresolved': [2, {ignore: ['^react-native$']}],
+        'unicorn/filename-case': 0
+      }
     },
     {
       files: ['scripts/**/*', 'integration_tests/**/*'],
       rules: {
         'babel/func-params-comma-dangle': 0,
-      },
+        'unicorn/filename-case': 0
+      }
     },
-  ],
+    {
+      files: ['**/__mocks__/**/*', 'website/**/*', '**/jest-runtime/**/*', '**/src/Console*'],
+      rules: {
+        'unicorn/filename-case': 0
+      }
+    }
+  ]
 };

examples/react-native/.flowconfig

@@ -26,7 +26,7 @@ experimental.strict_type_args=true
 
 munge_underscores=true
 
-module.name_mapper='^image![a-zA-Z0-9$_-]+$' -> 'GlobalImageStub'
+module.name_mapper='^image![a-zA-Z0-9$_-]+$' -> 'global_image_stub'
 module.name_mapper='^[./a-zA-Z0-9$_-]+\.\(bmp\|gif\|jpg\|jpeg\|png\|psd\|svg\|webp\|m4v\|mov\|mp4\|mpeg\|mpg\|webm\|aac\|aiff\|caf\|m4a\|mp3\|wav\|html\|pdf\)$' -> 'RelativeImageStub'
 
 suppress_type=$FlowIssue

fixtures/parserTests.js fixtures/parser_tests.jsfixtures/parserTests.js renamed to fixtures/parser_tests.js

@@ -28,6 +28,7 @@
     "eslint-plugin-import": "^2.6.0",
     "eslint-plugin-markdown": "^1.0.0-beta.6",
     "eslint-plugin-react": "^7.1.0",
+    "eslint-plugin-unicorn": "^2.1.2",
     "flow-bin": "^0.48.0",
     "glob": "^7.1.1",
     "graceful-fs": "^4.1.11",
@@ -114,7 +115,7 @@
     "transform": {
       "^.+\\.js$": "<rootDir>/packages/babel-jest"
     },
-    "setupTestFrameworkScriptFile": "<rootDir>/testSetupFile.js",
+    "setupTestFrameworkScriptFile": "<rootDir>/test_setup_file.js",
     "snapshotSerializers": [
       "<rootDir>/packages/pretty-format/build/plugins/convert_ansi.js"
     ],
@@ -130,5 +131,6 @@
     "testMatch": [
       "**/*.test.js"
     ]
-  }
+  },
+  "dependencies": {}
 }

package.json

@@ -11,7 +11,7 @@
 
 const os = require('os');
 
-const generateEmptyCoverage = require('../generateEmptyCoverage');
+const generateEmptyCoverage = require('../generate_empty_coverage');
 
 jest.mock('jest-runtime', () => {
   const realRuntime = require.requireActual('jest-runtime');

packages/jest-cli/src/__tests__/generate_empty_coverage.test.js

@@ -34,7 +34,7 @@ describe('SearchSource', () => {
 
   beforeEach(() => {
     Runtime = require('jest-runtime');
-    SearchSource = require('../SearchSource');
+    SearchSource = require('../search_source');
     normalize = require('jest-config').normalize;
   });
 
@@ -415,7 +415,7 @@ describe('SearchSource', () => {
     });
 
     it('returns empty search result if no related tests were found', () => {
-      const input = ['no tests.js'];
+      const input = ['no_tests.js'];
       const data = searchSource.findRelatedTestsFromPattern(input);
       expect(data.tests).toEqual([]);
     });

packages/jest-cli/src/__tests__/search_source.test.js

@@ -10,9 +10,9 @@
 
 'use strict';
 
-const TestRunner = require('../TestRunner');
-const TestWatcher = require('../TestWatcher');
-const SummaryReporter = require('../reporters/SummaryReporter');
+const TestRunner = require('../test_runner');
+const TestWatcher = require('../test_watcher');
+const SummaryReporter = require('../reporters/summary_reporter');
 
 let workerFarmMock;
 
@@ -27,8 +27,8 @@ jest.mock('worker-farm', () => {
   return mock;
 });
 
-jest.mock('../TestWorker', () => {});
-jest.mock('../reporters/DefaultReporter');
+jest.mock('../test_worker', () => {});
+jest.mock('../reporters/default_reporter');
 
 test('.addReporter() .removeReporter()', () => {
   const runner = new TestRunner({}, {});

…-cli/src/__tests__/test_root/no tests.js …-cli/src/__tests__/test_root/no_tests.jspackages/jest-cli/src/__tests__/test_root/no tests.js renamed to packages/jest-cli/src/__tests__/test_root/no_tests.js packages/jest-cli/src/__tests__/test_root/no tests.js renamed to packages/jest-cli/src/__tests__/test_root/no_tests.js

@@ -12,7 +12,7 @@ jest.mock('fs');
 const fs = require('fs');
 const path = require('path');
 
-const TestSequencer = require('../TestSequencer');
+const TestSequencer = require('../test_sequencer');
 
 const FAIL = 0;
 const SUCCESS = 1;

packages/jest-cli/src/__tests__/test_runner.test.js

@@ -11,14 +11,14 @@
 'use strict';
 
 const chalk = require('chalk');
-const TestWatcher = require('../TestWatcher');
+const TestWatcher = require('../test_watcher');
 const {KEYS} = require('../constants');
 
 const runJestMock = jest.fn();
 
 jest.doMock('chalk', () => new chalk.constructor({enabled: false}));
 jest.doMock(
-  '../runJest',
+  '../run_jest',
   () =>
     function() {
       const args = Array.from(arguments);