Merge pull request #5085 from jbellenger/jbellenger/rawmodule-hash

fix RawModule hashing
webpack · Jun 18, 2017 · 3da4f3e · 3da4f3e
2 parents 316d4b9 + 8c9dc14
commit 3da4f3e
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/lib/RawModule.js b/lib/RawModule.js
@@ -47,4 +47,8 @@ module.exports = class RawModule extends Module {
 			return new RawSource(this.sourceStr);
 	}
 
+	updateHash(hash) {
+		hash.update(this.sourceStr);
+		super.updateHash(hash);
+	}
 };
diff --git a/test/RawModule.test.js b/test/RawModule.test.js
@@ -6,6 +6,7 @@ const RawSource = require("webpack-sources").RawSource;
 const RequestShortener = require("../lib/RequestShortener");
 const should = require("should");
 const path = require("path");
+const crypto = require("crypto");
 
 describe("RawModule", () => {
 	let myRawModule;
@@ -62,4 +63,18 @@ describe("RawModule", () => {
 			}
 		);
 	});
+
+	describe("updateHash", () => {
+		it("should include sourceStr in its hash", () => {
+			const hashModule = (module) => {
+				const hash = crypto.createHash("sha256");
+				module.updateHash(hash);
+				return hash.digest("hex");
+			};
+
+			const hashFoo = hashModule(new RawModule("\"foo\""));
+			const hashBar = hashModule(new RawModule("\"bar\""));
+			hashFoo.should.not.equal(hashBar);
+		});
+	});
 });