test(javascript): assert wrapper ownership in the correct source mode

Fixes #3341

Co-authored-by: Kaoru0822 <208281322+kaoru0822-kitauji@users.noreply.github.com>

src/domain/javascript/javascriptSemanticAnalysis.require_links.test.ts

@@ -62,6 +62,22 @@ describe("TypeScript import-equals links", () => {
 });
 
 describe("CommonJS wrapper parameter redeclarations", () => {
+  it("keeps a top-level wrapper require redeclaration", () => {
+    const path = "app/main.cjs";
+    const parsed = parseJavaScriptSource(
+      'var require; require("./wrapper.cjs");',
+      path,
+    );
+    expect(parsed).not.toBeNull();
+    const ir = analyzeParsedJavaScriptSemantics(
+      parsed!,
+      moduleResolutionModeForJavaScriptReference(path, "static-import"),
+    );
+    expect(ir.moduleLinks).toContainEqual(
+      expect.objectContaining({ kind: "require", specifier: "./wrapper.cjs" }),
+    );
+  });
+
   it("retains ambient exports and require through uninitialized top-level vars", () => {
     const path = "app/main.cjs";
     const parsed = parseJavaScriptSource(

src/domain/javascript/javascriptStaticAnalysisModuleIndex.test.ts

@@ -78,12 +78,12 @@ it("does not resolve a parameter named require as a module dependency", () => {
   );
 });
 
-it("keeps a top-level wrapper require redeclaration", () => {
+it("keeps a formatless top-level require redeclaration unresolved", () => {
   const analysis = analyzeJavaScriptStaticSource(
     `var require; require("./wrapper.cjs");`,
   );
   expect(analysis.references).toContainEqual(
-    expect.objectContaining({ specifier: "./wrapper.cjs" }),
+    expect.objectContaining({ kind: "require", specifier: null }),
   );
 });