fix: conflict multiple asset error (webpack#194)

unchai · evilebottnawi · commit 1023d2290b09 · 2018-06-29T16:35:11.000+03:00
* fix: conflict multiple asset error

* fix: modify to work well in mode:'none'

* test: add contenthash-multiple-entries test code

* fix: remove change, cleanup test, upgrade webpack version
diff --git a/test/cases/contenthash-multiple-entries/entryA.js b/test/cases/contenthash-multiple-entries/entryA.js
@@ -0,0 +1,2 @@
+import "./styleA.css";
+import "./styleB.css";
diff --git a/test/cases/contenthash-multiple-entries/entryB.js b/test/cases/contenthash-multiple-entries/entryB.js
@@ -0,0 +1 @@
+import "./styleA.css";
diff --git a/test/cases/contenthash-multiple-entries/entryC.js b/test/cases/contenthash-multiple-entries/entryC.js
diff --git a/test/cases/contenthash-multiple-entries/entryD.js b/test/cases/contenthash-multiple-entries/entryD.js
@@ -0,0 +1,2 @@
+import "./styleA.css";
+import "./styleB.css";
diff --git a/test/cases/contenthash-multiple-entries/entryE.js b/test/cases/contenthash-multiple-entries/entryE.js
@@ -0,0 +1,2 @@
+import "./styleC.css";
+import "./styleD.css";
diff --git a/test/cases/contenthash-multiple-entries/expected/651d2818ea64cfe50dbe.css b/test/cases/contenthash-multiple-entries/expected/651d2818ea64cfe50dbe.css
@@ -0,0 +1,4 @@
+.styleA { background: red; }
+
+.styleB { background: blue; }
+
diff --git a/test/cases/contenthash-multiple-entries/expected/7bb844e9f5514d392e40.css b/test/cases/contenthash-multiple-entries/expected/7bb844e9f5514d392e40.css
@@ -0,0 +1,4 @@
+.styleA { background: red; }
+
+.styleB { background: blue; }
+
diff --git a/test/cases/contenthash-multiple-entries/expected/f36605fe5d863d4c4785.css b/test/cases/contenthash-multiple-entries/expected/f36605fe5d863d4c4785.css
@@ -0,0 +1,2 @@
+.styleA { background: red; }
+
diff --git a/test/cases/contenthash-multiple-entries/styleA.css b/test/cases/contenthash-multiple-entries/styleA.css
@@ -0,0 +1 @@
+.styleA { background: red; }
diff --git a/test/cases/contenthash-multiple-entries/styleB.css b/test/cases/contenthash-multiple-entries/styleB.css
@@ -0,0 +1 @@
+.styleB { background: blue; }
diff --git a/test/cases/contenthash-multiple-entries/styleC.css b/test/cases/contenthash-multiple-entries/styleC.css
@@ -0,0 +1 @@
+.styleA { background: red; }
diff --git a/test/cases/contenthash-multiple-entries/styleD.css b/test/cases/contenthash-multiple-entries/styleD.css
@@ -0,0 +1 @@
+.styleB { background: blue; }
diff --git a/test/cases/contenthash-multiple-entries/webpack.config.js b/test/cases/contenthash-multiple-entries/webpack.config.js
@@ -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',
+    }),
+  ],
+};

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+import "./styleA.css";`
	`2`	`+import "./styleB.css";`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+import "./styleC.css";`
	`2`	`+import "./styleD.css";`
-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +.styleA { background: red; }
++
 +.styleB { background: blue; }
++