test: ?????????

.github/workflows/ci.yaml

@@ -16,9 +16,9 @@ jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-      - name: Set up job
+      - name: Check out code
         uses: actions/checkout@v4
-      - name: Set up Terraform (replace with Coder action if possible)
+      - name: Set up Terraform (REPLACE WITH CODER ACTION WHEN DONE)
         uses: hashicorp/setup-terraform@b9cd54a3c349d3f38e8881555d616ced269862dd # v3.1.2
         with:
           terraform_version: 1.9.8
@@ -29,9 +29,9 @@ jobs:
           bun-version: latest
       - name: Install dependencies
         run: bun install
-      - name: Verify that Terraform is installed
+      - name: Verify that Terraform is installed (REMOVE WHEN DONE)
         run: which terraform
-      - name: Verify that Node.js is installed
+      - name: Verify that Node.js is installed (REMOVE WHEN DONE)
         run: which node
       - name: Run tests
         run: bun test

setup.ts

@@ -25,7 +25,7 @@ const removeOldContainers = async () => {
     "-a",
     "-q",
     "--filter",
-    `label=modules-test`,
+    "label=modules-test",
   ]);
   let containerIDsRaw = await readableStreamToText(proc.stdout);
   let exitCode = await proc.exited;

test.ts

@@ -194,14 +194,19 @@ export const testRequiredVariables = <TVars extends TerraformVariables>(
 export const runTerraformApply = async <TVars extends TerraformVariables>(
   dir: string,
   vars: Readonly<TVars>,
-  env?: Record<string, string>,
+  customEnv?: Record<string, string>,
 ): Promise<TerraformState> => {
   const stateFile = `${dir}/${crypto.randomUUID()}.tfstate`;
 
-  const combinedEnv = env === undefined ? {} : { ...env };
+  const combinedEnv: Record<string, string | undefined> = {
+    ...process.env,
+    ...(customEnv ?? {}),
+  };
   for (const [key, value] of Object.entries(vars)) {
+    if (value !== null) {
       combinedEnv[`TF_VAR_${key}`] = String(value);
     }
+  }
 
   const proc = spawn(
     [