test: adding and fixing broken tests

Following webpack here and getting my tests passing again

Author: ScriptedAlchemy

test/TestCases.test.js

@@ -1,36 +1,74 @@
-/* eslint-disable no-restricted-syntax, import/no-dynamic-require, global-require,
- no-console, no-undef, no-loop-func */
 import fs from 'fs';
 import path from 'path';
+
 import webpack from 'webpack';
 
 describe('TestCases', () => {
   const casesDirectory = path.resolve(__dirname, 'cases');
   const outputDirectory = path.resolve(__dirname, 'js');
   for (const directory of fs.readdirSync(casesDirectory)) {
     if (!/^(\.|_)/.test(directory)) {
+      // eslint-disable-next-line no-loop-func
       it(`${directory} should compile to the expected result`, (done) => {
         const directoryForCase = path.resolve(casesDirectory, directory);
         const outputDirectoryForCase = path.resolve(outputDirectory, directory);
-        const webpackConfig = require(path.resolve(directoryForCase, 'webpack.config.js'));
+        // eslint-disable-next-line import/no-dynamic-require, global-require
+        const webpackConfig = require(path.resolve(
+          directoryForCase,
+          'webpack.config.js'
+        ));
         for (const config of [].concat(webpackConfig)) {
-          Object.assign(config, { mode: 'none', context: directoryForCase, output: Object.assign({ path: outputDirectoryForCase }, config.output) }, config);
+          Object.assign(
+            config,
+            {
+              mode: 'none',
+              context: directoryForCase,
+              output: Object.assign(
+                {
+                  path: outputDirectoryForCase,
+                },
+                config.output
+              ),
+            },
+            config
+          );
         }
         webpack(webpackConfig, (err, stats) => {
           if (err) {
             done(err);
             return;
           }
           done();
-          console.log(stats.toString({ context: path.resolve(__dirname, '..'), chunks: true, chunkModules: true, modules: false }));
+          // eslint-disable-next-line no-console
+          console.log(
+            stats.toString({
+              context: path.resolve(__dirname, '..'),
+              chunks: true,
+              chunkModules: true,
+              modules: false,
+            })
+          );
           if (stats.hasErrors()) {
-            done(new Error(stats.toString({ context: path.resolve(__dirname, '..'), errorDetails: true })));
+            done(
+              new Error(
+                stats.toString({
+                  context: path.resolve(__dirname, '..'),
+                  errorDetails: true,
+                })
+              )
+            );
             return;
           }
           const expectedDirectory = path.resolve(directoryForCase, 'expected');
           for (const file of fs.readdirSync(expectedDirectory)) {
-            const content = fs.readFileSync(path.resolve(expectedDirectory, file), 'utf-8');
-            const actualContent = fs.readFileSync(path.resolve(outputDirectoryForCase, file), 'utf-8');
+            const content = fs.readFileSync(
+              path.resolve(expectedDirectory, file),
+              'utf-8'
+            );
+            const actualContent = fs.readFileSync(
+              path.resolve(outputDirectoryForCase, file),
+              'utf-8'
+            );
             expect(actualContent).toEqual(content);
           }
           done();

test/TestMemoryFS.test.js

@@ -0,0 +1,40 @@
+import path from 'path';
+
+import MemoryFS from 'memory-fs';
+import webpack from 'webpack';
+
+const assetsNames = (json) => json.assets.map((asset) => asset.name);
+
+describe('TestMemoryFS', () => {
+  it('should preserve asset even if not emitted', (done) => {
+    const casesDirectory = path.resolve(__dirname, 'cases');
+    const directoryForCase = path.resolve(casesDirectory, 'simple-publicpath');
+    const webpackConfig = require(path.resolve(
+      directoryForCase,
+      'webpack.config.js'
+    ));
+    const compiler = webpack({
+      ...webpackConfig,
+      mode: 'development',
+      context: directoryForCase,
+      cache: false,
+    });
+    compiler.outputFileSystem = new MemoryFS();
+    compiler.run((err1, stats1) => {
+      if (err1) {
+        done(err1);
+        return;
+      }
+      compiler.run((err2, stats2) => {
+        if (err2) {
+          done(err2);
+          return;
+        }
+        expect(assetsNames(stats1.toJson())).toEqual(
+          assetsNames(stats2.toJson())
+        );
+        done();
+      });
+    });
+  });
+});

test/cases/at-import/a.css

@@ -0,0 +1,9 @@
+@import './ae.css';
+@import './aa.css';
+@import './ab.css';
+@import './ac.css';
+@import './ad.css';
+
+body {
+  background: red;
+}

test/cases/at-import/aa.css

@@ -0,0 +1,3 @@
+.aa {
+  background: green;
+}

test/cases/at-import/ab.css

@@ -0,0 +1,3 @@
+.ab {
+  background: green;
+}

test/cases/at-import/ac.css

@@ -0,0 +1,3 @@
+.ac {
+  background: green;
+}

test/cases/at-import/ad.css

@@ -0,0 +1,3 @@
+.ad {
+  background: green;
+}

test/cases/at-import/ae.css

@@ -0,0 +1,3 @@
+.ae {
+  background: green;
+}

test/cases/at-import/b.css

@@ -0,0 +1,6 @@
+@import './ba.css';
+@import './bb.css';
+
+body {
+  background: yellow;
+}

test/cases/at-import/ba.css

@@ -0,0 +1,3 @@
+.ba {
+  background: green;
+}

test/cases/at-import/bb.css

@@ -0,0 +1,3 @@
+.bb {
+  background: green;
+}

test/cases/at-import/expected/main.css

@@ -0,0 +1,36 @@
+.ae {
+  background: green;
+}
+
+.aa {
+  background: green;
+}
+
+.ab {
+  background: green;
+}
+
+.ac {
+  background: green;
+}
+
+.ad {
+  background: green;
+}
+
+body {
+  background: red;
+}
+
+.ba {
+  background: green;
+}
+
+.bb {
+  background: green;
+}
+
+body {
+  background: yellow;
+}
+

test/cases/at-import/index.js

@@ -0,0 +1,2 @@
+import './a.css';
+import './b.css';

test/cases/at-import/webpack.config.js

@@ -0,0 +1,21 @@
+const Self = require('../../../');
+
+module.exports = {
+  entry: './index.js',
+  module: {
+    rules: [
+      {
+        test: /\.css$/,
+        use: [
+          Self.loader,
+          'css-loader',
+        ],
+      },
+    ],
+  },
+  plugins: [
+    new Self({
+      filename: '[name].css',
+    }),
+  ],
+};

test/cases/composes-async/expected/1.css

@@ -1,4 +1,4 @@
-.base {
-  background: blue;
+.composed {
+  background: green;
 }
 

test/cases/composes-async/expected/2.css

@@ -0,0 +1,4 @@
+.base {
+  background: blue;
+}
+

test/cases/composes-async/webpack.config.js

@@ -11,9 +11,9 @@ module.exports = {
           {
             loader: 'css-loader',
             options: {
-              localIdentName: '[local]',
-            },
-          },
+              localIdentName: '[local]'
+            }
+          }
         ],
       },
     ],
@@ -25,10 +25,10 @@ module.exports = {
           test: /\.css$/,
           chunks: 'all',
           minChunks: 2,
-          enforce: true,
-        },
-      },
-    },
+          enforce: true
+        }
+      }
+    }
   },
   plugins: [
     new Self({

test/cases/contenthash-multiple-entries/entryA.js

@@ -0,0 +1,2 @@
+import "./styleA.css";
+import "./styleB.css";

test/cases/contenthash-multiple-entries/entryB.js

@@ -0,0 +1 @@
+import "./styleA.css";

test/cases/contenthash-multiple-entries/entryC.js

@@ -0,0 +1,2 @@
+import "./styleA.css";
+import "./styleB.css";

test/cases/contenthash-multiple-entries/entryD.js

@@ -0,0 +1,2 @@
+import "./styleC.css";
+import "./styleD.css";

test/cases/contenthash-multiple-entries/entryE.js

@@ -0,0 +1,4 @@
+.styleA { background: red; }
+
+.styleB { background: blue; }
+

test/cases/contenthash-multiple-entries/expected/10adcc6fa8f8a3cf8d8d.css

@@ -0,0 +1,2 @@
+.styleA { background: red; }
+

test/cases/contenthash-multiple-entries/expected/3023a4e1067adc5b1950.css

@@ -0,0 +1 @@
+.styleA { background: red; }

test/cases/contenthash-multiple-entries/styleA.css

@@ -0,0 +1 @@
+.styleB { background: blue; }

test/cases/contenthash-multiple-entries/styleB.css

@@ -0,0 +1 @@
+.styleA { background: red; }

test/cases/contenthash-multiple-entries/styleC.css

@@ -0,0 +1 @@
+.styleB { background: blue; }

test/cases/contenthash-multiple-entries/styleD.css

@@ -0,0 +1,30 @@
+const Self = require('../../../');
+
+module.exports = {
+  entry: {
+    'entryA': './entryA.js',
+    'entryB': './entryB.js',
+    'entryC': './entryC.js',
+    'entryD': './entryD.js',
+    'entryE': './entryE.js',
+  },
+  module: {
+    rules: [
+      {
+        test: /\.css$/,
+        use: [
+          Self.loader,
+          'css-loader',
+        ],
+      },
+    ],
+  },
+  output: {
+    filename: '[name]-[contenthash].js',
+  },
+  plugins: [
+    new Self({
+      filename: '[contenthash].css',
+    }),
+  ],
+};

test/cases/contenthash-multiple-entries/webpack.config.js

@@ -0,0 +1,2 @@
+body { background: red; }
+

test/cases/contenthash/expected/1.main.e0d1c5b21d098a0a371f.css

@@ -0,0 +1,2 @@
+body { background: green; }
+

test/cases/contenthash/expected/2.main.9728193f8673cfdafd0e.css

@@ -1 +1 @@
-import './style.css'; // eslint-disable-line import/no-unresolved
+import './style.css';

test/cases/contenthash/index.js

@@ -14,12 +14,12 @@ module.exports = [1, 2].map(n => ({
     ],
   },
   output: {
-    filename: `${n}.[name].js`,
+    filename: `${n}.[name].js`
   },
   resolve: {
     alias: {
-      './style.css': `./style${n}.css`,
-    },
+      './style.css': `./style${n}.css`
+    }
   },
   plugins: [
     new Self({

test/cases/contenthash/webpack.config.js

@@ -0,0 +1 @@
+body { background: red; }

test/cases/css-import/a.css

@@ -0,0 +1,5 @@
+.b { background: red; }
+
+@import url("https://some/external/css");
+
+.b { color: yellow; }

test/cases/css-import/b.css

@@ -0,0 +1,5 @@
+.c { background: red; }
+@import './a.css';
+@import url("https://some/other/external/css");
+
+.c { color: yellow; }

test/cases/css-import/c.css

@@ -0,0 +1,14 @@
+@import url(https://some/other/external/css);
+@import url(https://some/external/css);
+body { background: red; }
+
+.c { background: red; }
+
+.c { color: yellow; }
+
+.b { background: red; }
+
+.b { color: yellow; }
+
+
+

test/cases/css-import/expected/main.css

@@ -0,0 +1,2 @@
+@import './c.css';
+@import './b.css';