add comment clarifying why the assertion is needed

davidmurdoch · davidmurdoch · commit de13ec9b45d2 · 2026-03-26T20:50:03.000-04:00
diff --git a/test/watchCases/cache/stale-runtime-chunk-filename-map/0/service-worker.js b/test/watchCases/cache/stale-runtime-chunk-filename-map/0/service-worker.js
@@ -14,5 +14,8 @@ it("should update the runtime chunk filename map on rebuild", () => {
 		expect(currentChunkFile).not.toBe(STATE.previousChunkFile);
 	}
 
+	// We need this emitted-source assertion because an incorrect hash order may
+	// still produce a new build, but a wrong one. Check the emitted source to
+	// ensure the new chunk filename is actually referenced by service-worker.js.
 	expect(serviceWorkerSource).toContain(currentChunkFile);
 });

Original file line number	Diff line number	Diff line change
`@@ -14,5 +14,8 @@ it("should update the runtime chunk filename map on rebuild", () => {`
`14`	`14`	`expect(currentChunkFile).not.toBe(STATE.previousChunkFile);`
`15`	`15`	`}`
`16`	`16`
	`17`	`+ // We need this emitted-source assertion because an incorrect hash order may`
	`18`	`+ // still produce a new build, but a wrong one. Check the emitted source to`
	`19`	`+ // ensure the new chunk filename is actually referenced by service-worker.js.`
`17`	`20`	`expect(serviceWorkerSource).toContain(currentChunkFile);`
`18`	`21`	`});`