Use same test config for react

Author: Petr Hanák

environments/nodejs/optional.js

@@ -8,20 +8,8 @@
 
 'use strict'
 
-const globs = require('./globs')
-
 module.exports = {
 
   extends: '../shared/optional.js',
 
-  rules: {},
-
-  overrides: {
-    files: globs.test,
-
-    rules: {
-      // Do not require function names in test files
-      'func-names': 'off',
-    },
-  },
 }

environments/nodejs/recommended.js

@@ -8,8 +8,6 @@
 
 'use strict'
 
-const globs = require('./globs')
-
 module.exports = {
 
   extends: '../shared/recommended.js',
@@ -64,19 +62,4 @@ module.exports = {
       allowAtRootLevel: true,
     }],
   },
-
-  overrides: [{
-    files: globs.test,
-
-    env: {
-      mocha: true,
-    },
-  }, {
-    files: globs.config,
-
-    rules: {
-      // Using process.env is encouraged in configuration files
-      'no-process-env': 'off',
-    },
-  }],
 }

environments/nodejs/globs.js renamed to environments/shared/globs.js

@@ -3,11 +3,9 @@
 module.exports = {
   // Test files patterns
   test: [
-    // Classic layout
     'test/**',
-    // Nested test files, ie. Lerna monorepo
-    '**/test/**/*.test.*',
-    '**/test/**/*.spec.*',
+    '**/*.test.*',
+    '**/*.spec.*',
   ],
 
   // Configuration files patterns

environments/shared/optional.js

@@ -8,6 +8,8 @@
 
 'use strict'
 
+const globs = require('./globs')
+
 module.exports = {
 
   rules: {
@@ -64,4 +66,13 @@ module.exports = {
     // Reports if a resolved path is imported more than once
     'import/no-duplicates': 'warn',
   },
+
+  overrides: {
+    files: globs.test,
+
+    rules: {
+      // Do not require function names in test files
+      'func-names': 'off',
+    },
+  },
 }

environments/shared/recommended.js

@@ -8,6 +8,8 @@
 
 'use strict'
 
+const globs = require('./globs')
+
 module.exports = {
 
   plugins: [
@@ -656,4 +658,13 @@ module.exports = {
     // is likely to be a mistake.
     'import/no-named-as-default-member': 'warn',
   },
+
+  overrides: [{
+    files: globs.config,
+
+    rules: {
+      // Using process.env is encouraged in configuration files
+      'no-process-env': 'off',
+    },
+  }],
 }