test: Failing test for css ordering

webpack-contrib · Jul 7, 2017 · 55f2cf0 · 55f2cf0
1 parent 28363ab
commit 55f2cf0
Show file tree

Hide file tree

Showing 9 changed files with 47 additions and 1 deletion.
diff --git a/src/index.js b/src/index.js
@@ -144,7 +144,7 @@ class ExtractTextPlugin {
         async.forEach(chunks, (chunk, callback) => { // eslint-disable-line no-shadow
           const extractedChunk = extractedChunks[chunks.indexOf(chunk)];
           const shouldExtract = !!(options.allChunks || isInitialOrHasNoParents(chunk));
-          chunk.sortModules();
+          chunk.sortModules(module);
           async.forEach(chunk.mapModules(c => c), (module, callback) => { // eslint-disable-line no-shadow
             let meta = module[NS];
             if (meta && (!meta.options.id || meta.options.id === id)) {

diff --git a/test/__snapshots__/webpack-integration.test.js.snap b/test/__snapshots__/webpack-integration.test.js.snap
@@ -1,5 +1,15 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`Webpack Integration Tests chunk-modules-css-wrong-order 1`] = `
+".block {
+	color: tomato;
+}
+.App {
+	color: black;
+}
+"
+`;
+
 exports[`Webpack Integration Tests chunk-modules-nested-ordered-by-id 1`] = `
 "body {
   margin: 0;

diff --git a/test/cases/chunk-modules-css-wrong-order/a.css b/test/cases/chunk-modules-css-wrong-order/a.css
@@ -0,0 +1,3 @@
+.App {
+  text-align: center;
+}
diff --git a/test/cases/chunk-modules-css-wrong-order/a.js b/test/cases/chunk-modules-css-wrong-order/a.js
@@ -0,0 +1,2 @@
+require('./b.js');
+require('./a.css');
diff --git a/test/cases/chunk-modules-css-wrong-order/b.css b/test/cases/chunk-modules-css-wrong-order/b.css
@@ -0,0 +1,3 @@
+.block {
+  font-size: 16px;
+}
diff --git a/test/cases/chunk-modules-css-wrong-order/b.js b/test/cases/chunk-modules-css-wrong-order/b.js
@@ -0,0 +1 @@
+require('./b.css');
diff --git a/test/cases/chunk-modules-css-wrong-order/expected/file.css b/test/cases/chunk-modules-css-wrong-order/expected/file.css
@@ -0,0 +1,7 @@
+.block {
+  font-size: 16px;
+}
+.App {
+  text-align: center;
+}
+
diff --git a/test/cases/chunk-modules-css-wrong-order/index.js b/test/cases/chunk-modules-css-wrong-order/index.js
@@ -0,0 +1 @@
+require('./a.js');
diff --git a/test/cases/chunk-modules-css-wrong-order/webpack.config.js b/test/cases/chunk-modules-css-wrong-order/webpack.config.js
@@ -0,0 +1,19 @@
+import ExtractTextPlugin from '../../../src/index';
+
+module.exports = {
+  entry: './index',
+  module: {
+    loaders: [
+      {
+        test: /\.css$/,
+        use: ExtractTextPlugin.extract({
+          fallback: { loader: 'style-loader' },
+          use: { loader: 'css-loader', },
+        })
+      },
+    ],
+  },
+  plugins: [
+    new ExtractTextPlugin('file.css'),
+  ],
+};