test: replace unit test with configCase for findGraphRoots determinism

veeceey · claude · veeceey · commit 23468af4ff73 · 2026-02-15T16:14:34.000-08:00
Move the findGraphRoots test from a standalone unittest to a configCase
under chunk-graph/deterministic-roots as requested by the reviewer.
The test exercises cyclic module dependencies (A -&gt; C -&gt; A, A -&gt; B -&gt; C)
to verify deterministic graph root selection.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/test/configCases/chunk-graph/deterministic-roots/a.js b/test/configCases/chunk-graph/deterministic-roots/a.js
@@ -0,0 +1,10 @@
+import { getC } from "./c";
+import { getB } from "./b";
+
+export function getA() {
+	return "a";
+}
+
+export function getCombined() {
+	return getA() + getB() + getC();
+}
diff --git a/test/configCases/chunk-graph/deterministic-roots/b.js b/test/configCases/chunk-graph/deterministic-roots/b.js
@@ -0,0 +1,5 @@
+import { getC } from "./c";
+
+export function getB() {
+	return "b" + getC();
+}
diff --git a/test/configCases/chunk-graph/deterministic-roots/c.js b/test/configCases/chunk-graph/deterministic-roots/c.js
@@ -0,0 +1,5 @@
+import { getA } from "./a";
+
+export function getC() {
+	return "c" + getA();
+}
diff --git a/test/configCases/chunk-graph/deterministic-roots/index.js b/test/configCases/chunk-graph/deterministic-roots/index.js
@@ -0,0 +1,8 @@
+import { getCombined } from "./a";
+
+it("should produce deterministic output with cyclic dependencies", () => {
+	const result = getCombined();
+	expect(result).toContain("a");
+	expect(result).toContain("b");
+	expect(result).toContain("c");
+});
diff --git a/test/configCases/chunk-graph/deterministic-roots/webpack.config.js b/test/configCases/chunk-graph/deterministic-roots/webpack.config.js
@@ -0,0 +1,8 @@
+"use strict";
+
+/** @type {import("../../../../").Configuration} */
+module.exports = {
+	optimization: {
+		concatenateModules: true
+	}
+};
diff --git a/test/findGraphRoots.unittest.js b/test/findGraphRoots.unittest.js
