No codegen-reference to givePort in tests (#3089)

RobinTail · web-flow · commit bdc13ab345b9 · 2025-11-24T17:54:19.000+01:00
This should simplify CJS test without `tsx` and overall it's redundant

&lt;!-- This is an auto-generated comment: release notes by coderabbit.ai
--&gt;

## Summary by CodeRabbit

* **Chores**
* Optimized internal test generation tooling by streamlining program
assembly logic and updating test content mappings for consistency. No
user-facing changes.

&lt;sub&gt;✏️ Tip: You can customize this high-level summary in your review
settings.&lt;/sub&gt;

&lt;!-- end of auto-generated comment: release notes by coderabbit.ai --&gt;
diff --git a/tools/make-tests.ts b/tools/make-tests.ts
@@ -14,19 +14,15 @@ const extractQuickStartFromReadme = async () => {
     .trim();
 };
 
-const quickStart = await extractQuickStartFromReadme();
-const program = `
-import { givePort } from "../tools/ports";
-${quickStart}
-`;
+const program = await extractQuickStartFromReadme();
 
 const examplePort = String(givePort("example"));
 
 const testContent = {
-  cjs: program.replace(examplePort, "givePort('cjs')"),
-  esm: program.replace(examplePort, "givePort('esm')"),
-  compat: program.replace(examplePort, "givePort('compat')"),
-  issue952: quickStart.replace(/const/g, "export const"),
+  cjs: program.replace(examplePort, String(givePort("cjs"))),
+  esm: program.replace(examplePort, String(givePort("esm"))),
+  compat: program.replace(examplePort, String(givePort("compat"))),
+  issue952: program.replace(/const/g, "export const"),
 };
 
 for (const testName in testContent) {
