fix test context pollution

Author: danitt

packages/openapi-typescript/test/transform/schema-object/enum.test.ts

@@ -1,91 +1,13 @@
-import { fileURLToPath } from "node:url";
 import { transformSchema } from "../../../src/index.js";
 import { astToString } from "../../../src/lib/ts.js";
-import type { GlobalContext } from "../../../src/types.js";
 import { DEFAULT_CTX, type TestCase } from "../../test-helpers.js";
 
-const DEFAULT_OPTIONS = DEFAULT_CTX;
-
-const schema = {
-  openapi: "3.0.0",
-  info: {
-    title: "Status API",
-    version: "1.0.0",
-  },
-  paths: {
-    "/status": {
-      get: {
-        summary: "Get current status",
-        responses: {
-          "200": {
-            description: "Status response",
-            content: {
-              "application/json": {
-                schema: {
-                  type: "object",
-                  properties: {
-                    required: {
-                      "status": true,
-                      "statusEnum": true,
-                    },
-                    status: {
-                      $ref: "#/components/schemas/StatusResponse",
-                    },
-                    statusEnum: {
-                      $ref: "#/components/schemas/StatusEnumResponse",
-                    }
-                  }
-                },
-              },
-            },
-          },
-        },
-      },
-    },
-  },
-  components: {
-    schemas: {
-      StatusResponse: {
-        type: "object",
-        properties: {
-          status: {
-            $ref: "#/components/schemas/Status",
-          },
-        },
-      },
-      Status: {
-        type: "string",
-        enum: ["pending", "active", "done"],
-      },
-      StatusEnumResponse: {
-        type: "object",
-        properties: {
-          status: {
-            $ref: "#/components/schemas/StatusEnum",
-          },
-        },
-      },
-      StatusEnum: {
-        type: "string",
-        enum: ["pending", "active", "done"],
-        "x-enum-varnames": ["Pending", "Active", "Done"],
-        "x-enum-descriptions": [
-          "The task is pending",
-          "The task is active",
-          "The task is done",
-        ],
-      },
-    },
-  },
-};
-
-describe("transformComponentsObject", () => {
-  const tests: TestCase<any, GlobalContext>[] = [
-    [
-      "options > enum: true and conditionalEnums: false",
-      {
-        given: schema,
-        want: `export interface paths {
+const tests: TestCase[] = [
+  [
+    "options > enum: true and conditionalEnums: false",
+    {
+      given: mockSchema(),
+      want: `export interface paths {
     "/status": {
         parameters: {
             query?: never;
@@ -162,14 +84,14 @@ export enum StatusEnum {
     Done = "done"
 }
 export type operations = Record<string, never>;`,
-        options: { ...DEFAULT_OPTIONS, enum: true, conditionalEnums: false },
-      },
-    ],
-    [
-      "options > enum: true and conditionalEnums: true",
-      {
-        given: schema,
-        want: `export interface paths {
+      options: { ctx: createTestContext({ enum: true, conditionalEnums: false }) },
+    },
+  ],
+  [
+    "options > enum: true and conditionalEnums: true",
+    {
+      given: mockSchema(),
+      want: `export interface paths {
     "/status": {
         parameters: {
             query?: never;
@@ -232,19 +154,6 @@ export interface components {
     pathItems: never;
 }
 export type $defs = Record<string, never>;
-export enum Status {
-    pending = "pending",
-    active = "active",
-    done = "done"
-}
-export enum StatusEnum {
-    // The task is pending
-    Pending = "pending",
-    // The task is active
-    Active = "active",
-    // The task is done
-    Done = "done"
-}
 export enum StatusEnum {
     // The task is pending
     Pending = "pending",
@@ -254,23 +163,105 @@ export enum StatusEnum {
     Done = "done"
 }
 export type operations = Record<string, never>;`,
-        options: { ...DEFAULT_OPTIONS, enum: true, conditionalEnums: true },
-      },
-    ],
-  ];
+      options: { ctx: createTestContext({ enum: true, conditionalEnums: true }) },
+    },
+  ],
+];
 
-  for (const [testName, { given, want, options, ci }] of tests) {
+describe("transformComponentsObject", () => {
+  describe.each(tests)("Case: %s", (name, { given, want, options, ci }) => {
     test.skipIf(ci?.skipIf)(
-      testName,
+      "it matches the snapshot",
       async () => {
-        const result = astToString(transformSchema(given, options ?? DEFAULT_OPTIONS));
-        if (want instanceof URL) {
-          await expect(result).toMatchFileSnapshot(fileURLToPath(want));
-        } else {
-          expect(result.trim()).toBe(want.trim());
-        }
+        assert(typeof want === "string");
+        const result = astToString(transformSchema(given, options?.ctx ?? DEFAULT_CTX), { fileName: name });
+        expect(result.trim()).toBe(want.trim());
       },
       ci?.timeout,
     );
-  }
+  });
 });
+
+function mockSchema() {
+  return {
+    openapi: "3.0.0",
+    info: {
+      title: "Status API",
+      version: "1.0.0",
+    },
+    paths: {
+      "/status": {
+        get: {
+          summary: "Get current status",
+          responses: {
+            "200": {
+              description: "Status response",
+              content: {
+                "application/json": {
+                  schema: {
+                    type: "object",
+                    properties: {
+                      required: {
+                        status: true,
+                        statusEnum: true,
+                      },
+                      status: {
+                        $ref: "#/components/schemas/StatusResponse",
+                      },
+                      statusEnum: {
+                        $ref: "#/components/schemas/StatusEnumResponse",
+                      },
+                    },
+                  },
+                },
+              },
+            },
+          },
+        },
+      },
+    },
+    components: {
+      schemas: {
+        StatusResponse: {
+          type: "object",
+          properties: {
+            status: {
+              $ref: "#/components/schemas/Status",
+            },
+          },
+        },
+        Status: {
+          type: "string",
+          enum: ["pending", "active", "done"],
+        },
+        StatusEnumResponse: {
+          type: "object",
+          properties: {
+            status: {
+              $ref: "#/components/schemas/StatusEnum",
+            },
+          },
+        },
+        StatusEnum: {
+          type: "string",
+          enum: ["pending", "active", "done"],
+          "x-enum-varnames": ["Pending", "Active", "Done"],
+          "x-enum-descriptions": ["The task is pending", "The task is active", "The task is done"],
+        },
+      },
+    },
+  };
+}
+
+function createTestContext(overrides: Partial<typeof DEFAULT_CTX> = {}) {
+  return {
+    ...DEFAULT_CTX,
+    ...overrides,
+    // Deep copy mutable properties to avoid scope pollution
+    discriminators: {
+      objects: {},
+      refsHandled: [],
+    },
+    injectFooter: [],
+  };
+}