chore: ignore tests from npm (#2563)

anshumanv · web-flow · commit f8d6606a7996 · 2021-03-27T11:35:41.000+05:30
diff --git a/.npmignore b/.npmignore
diff --git a/packages/webpack-cli/package.json b/packages/webpack-cli/package.json
@@ -24,7 +24,8 @@
   ],
   "files": [
     "bin",
-    "lib"
+    "lib",
+    "!**/*__tests__"
   ],
   "dependencies": {
     "@discoveryjs/json-ext": "^0.5.0",
diff --git a/smoketests/index.js b/smoketests/index.js
@@ -1,5 +1,3 @@
-/* eslint-disable node/no-unpublished-require */
-
 const tests = [
     require('./missing-packages/webpack-dev-server.test.js'),
     require('./missing-packages/webpack.test.js'),
diff --git a/test/build/analyze/analyze.config.js b/test/build/analyze/analyze.config.js
@@ -1,3 +1,4 @@
+// eslint-disable-next-line node/no-unpublished-require
 const { BundleAnalyzerPlugin } = require('webpack-bundle-analyzer');
 
 module.exports = {
diff --git a/test/build/cache/cache.test.js b/test/build/cache/cache.test.js
@@ -1,6 +1,7 @@
 'use strict';
 
 const path = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const rimraf = require('rimraf');
 const { run, isWebpack5 } = require('../../utils/test-utils');
 
diff --git a/test/build/core-flags/cache-flags.test.js b/test/build/core-flags/cache-flags.test.js
@@ -1,6 +1,7 @@
 'use strict';
 
 const path = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const rimraf = require('rimraf');
 const { run } = require('../../utils/test-utils');
 const { existsSync, writeFileSync, unlinkSync } = require('fs');
diff --git a/test/build/entry/scss/webpack.config.js b/test/build/entry/scss/webpack.config.js
@@ -1,4 +1,4 @@
-// eslint-disable-next-line
+// eslint-disable-next-line node/no-missing-require
 const MiniCssExtractPlugin = require('mini-css-extract-plugin');
 
 module.exports = {
diff --git a/test/build/env/array/array-env.test.js b/test/build/env/array/array-env.test.js
@@ -1,6 +1,7 @@
 'use strict';
 
 const path = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const execa = require('execa');
 
 const { sync: spawnSync } = execa;
diff --git a/test/build/env/object/object-env.test.js b/test/build/env/object/object-env.test.js
@@ -1,6 +1,7 @@
 'use strict';
 
 const path = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const execa = require('execa');
 const { sync: spawnSync } = execa;
 
diff --git a/test/build/prefetch/prefetch.test.js b/test/build/prefetch/prefetch.test.js
@@ -3,6 +3,7 @@
 const fs = require('fs');
 const { join } = require('path');
 const { run } = require('../../utils/test-utils');
+// eslint-disable-next-line node/no-unpublished-require
 const rimraf = require('rimraf');
 
 describe('prefetch', () => {
diff --git a/test/help/help.test.js b/test/help/help.test.js
@@ -1,5 +1,6 @@
 'use strict';
 const { run } = require('../utils/test-utils');
+// eslint-disable-next-line node/no-unpublished-require
 const serializer = require('jest-serializer-ansi');
 
 // TODO fix it
diff --git a/test/init/init.test.js b/test/init/init.test.js
@@ -1,5 +1,6 @@
 const { mkdirSync, existsSync, readFileSync } = require('fs');
 const { join, resolve } = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const rimraf = require('rimraf');
 const { run, runPromptWithAnswers } = require('../utils/test-utils');
 
diff --git a/test/loader/loader.test.js b/test/loader/loader.test.js
@@ -2,6 +2,7 @@
 
 const { existsSync, mkdirSync } = require('fs');
 const { join, resolve } = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const rimraf = require('rimraf');
 const { run, runPromptWithAnswers } = require('../utils/test-utils');
 
diff --git a/test/plugin/plugin.test.js b/test/plugin/plugin.test.js
@@ -1,5 +1,6 @@
 const { existsSync, mkdirSync } = require('fs');
 const { join, resolve } = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const rimraf = require('rimraf');
 const { run, runPromptWithAnswers } = require('../utils/test-utils');
 
diff --git a/test/serve/basic/multi-dev-server.config.js b/test/serve/basic/multi-dev-server.config.js
@@ -1,3 +1,4 @@
+// eslint-disable-next-line node/no-unpublished-require
 const getPort = require('get-port');
 
 const WebpackCLITestPlugin = require('../../utils/webpack-cli-test-plugin');
diff --git a/test/serve/basic/serve-basic.test.js b/test/serve/basic/serve-basic.test.js
@@ -1,6 +1,7 @@
 'use strict';
 
 const path = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const getPort = require('get-port');
 const { runWatch, isWebpack5, isDevServer4 } = require('../../utils/test-utils');
 
diff --git a/test/serve/serve-variable/serve-basic.test.js b/test/serve/serve-variable/serve-basic.test.js
@@ -1,6 +1,7 @@
 'use strict';
 
 const path = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const getPort = require('get-port');
 const { runWatch } = require('../../utils/test-utils');
 
diff --git a/test/serve/with-custom-port/serve-custom-config.test.js b/test/serve/with-custom-port/serve-custom-config.test.js
@@ -1,6 +1,7 @@
 'use strict';
 
 const path = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const getPort = require('get-port');
 const { runWatch } = require('../../utils/test-utils');
 
diff --git a/test/utils/test-utils.js b/test/utils/test-utils.js
@@ -1,3 +1,5 @@
+/* eslint-disable node/no-unpublished-require */
+
 'use strict';
 const path = require('path');
 const fs = require('fs');
diff --git a/test/utils/test-utils.test.js b/test/utils/test-utils.test.js
@@ -3,6 +3,7 @@
 const { appendDataIfFileExists, run, runAndGetWatchProc, hyphenToUpperCase } = require('./test-utils');
 const { writeFileSync, unlinkSync, readFileSync, mkdirSync } = require('fs');
 const { resolve } = require('path');
+// eslint-disable-next-line node/no-unpublished-require
 const rimraf = require('rimraf');
 
 const ENTER = '\x0D';
diff --git a/test/version/version.test.js b/test/version/version.test.js
@@ -7,6 +7,7 @@ const pkgJSON = require('../../packages/webpack-cli/package.json');
 const servePkgJSON = require('../../packages/serve/package.json');
 const infoPkgJSON = require('../../packages/info/package.json');
 const generatorsPkgJSON = require('../../packages/generators/package.json');
+// eslint-disable-next-line node/no-unpublished-require
 const webpackDevServerPkgJSON = require('webpack-dev-server/package.json');
 
 describe('single version flag', () => {
diff --git a/test/watch/analyze/analyze.config.js b/test/watch/analyze/analyze.config.js
@@ -1,3 +1,4 @@
+// eslint-disable-next-line node/no-unpublished-require
 const { BundleAnalyzerPlugin } = require('webpack-bundle-analyzer');
 
 module.exports = {

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,3 @@`
`1`		`-/* eslint-disable node/no-unpublished-require */`
`2`		`-`
`3`	`1`	`const tests = [`
`4`	`2`	`require('./missing-packages/webpack-dev-server.test.js'),`
`5`	`3`	`require('./missing-packages/webpack.test.js'),`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+// eslint-disable-next-line node/no-unpublished-require`
`1`	`2`	`const { BundleAnalyzerPlugin } = require('webpack-bundle-analyzer');`
`2`	`3`
`3`	`4`	`module.exports = {`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-// eslint-disable-next-line`
	`1`	`+// eslint-disable-next-line node/no-missing-require`
`2`	`2`	`const MiniCssExtractPlugin = require('mini-css-extract-plugin');`
`3`	`3`
`4`	`4`	`module.exports = {`