Add test for QUARTO_PROJECT_SCRIPT_PROGRESS and QUARTO_PROJECT_SCRIPT_QUIET env vars

Pre/post render scripts now receive these env vars but had no test coverage.
The new test fixture uses a pre-render script to dump the env var values to
a JSON file, then verifies the values match expectations (progress=1 for
multi-file projects, quiet=0 without --quiet flag).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: cderv

llm-docs/testing-patterns.md

@@ -88,6 +88,62 @@ testQuartoCmd(
 - Use absolute paths with `join()` for file verification
 - Clean up output directories in teardown
 
+### Pre/Post Render Script Tests
+
+For testing pre-render or post-render scripts that run during project rendering:
+
+```typescript
+import { docs } from "../../utils.ts";
+import { join } from "../../../src/deno_ral/path.ts";
+import { existsSync } from "../../../src/deno_ral/fs.ts";
+import { testQuartoCmd } from "../../test.ts";
+import { noErrors, validJsonWithFields } from "../../verify.ts";
+import { safeRemoveIfExists } from "../../../src/core/path.ts";
+
+const projectDir = docs("project/prepost/my-test");
+const projectDirAbs = join(Deno.cwd(), projectDir);
+const dumpPath = join(projectDirAbs, "output.json");
+const outDir = join(projectDirAbs, "_site");
+
+testQuartoCmd(
+  "render",
+  [projectDir],
+  [
+    noErrors,
+    validJsonWithFields(dumpPath, {
+      expected: "value",
+    }),
+  ],
+  {
+    teardown: async () => {
+      safeRemoveIfExists(dumpPath);
+      if (existsSync(outDir)) {
+        await Deno.remove(outDir, { recursive: true });
+      }
+    },
+  },
+);
+```
+
+**Fixture structure:**
+
+```
+tests/docs/project/prepost/my-test/
+├── _quarto.yml       # project config with pre-render/post-render scripts
+├── index.qmd         # minimal page (website needs at least one)
+├── check-env.ts      # pre/post-render script (Deno TypeScript)
+└── .gitignore        # exclude .quarto/ and *.quarto_ipynb
+```
+
+**Key points:**
+- Pre/post-render scripts run as subprocesses with `cwd` set to the project directory
+- Scripts access environment variables via `Deno.env.get()` and can write files for verification
+- Use `validJsonWithFields` for JSON file verification (parses and compares field values exactly)
+- Use `ensureFileRegexMatches` for non-JSON files or when regex matching is needed
+- The file dump pattern (script writes JSON, test reads it) is useful for verifying env vars and other runtime state
+- Clean up both the dump file and the output directory in teardown
+- Existing fixtures: `tests/docs/project/prepost/` (mutate-render-list, invalid-mutate, extension, issue-10828, script-env-vars)
+
 ### Extension Template Tests
 
 For testing `quarto use template`:
@@ -164,6 +220,43 @@ folderExists(path: string)
 directoryEmptyButFor(dir: string, allowedFiles: string[])
 ```
 
+### Content Verifiers
+
+```typescript
+// Regex match on file contents (matches required, noMatches must be absent)
+ensureFileRegexMatches(file: string, matches: (string | RegExp)[], noMatches?: (string | RegExp)[])
+
+// Regex match on CSS files linked from HTML
+ensureCssRegexMatches(file: string, matches: (string | RegExp)[], noMatches?: (string | RegExp)[])
+
+// Check HTML elements exist or don't exist (CSS selectors)
+ensureHtmlElements(file: string, noElements: string[], elements: string[])
+
+// Verify JSON structure has expected fields
+validJsonWithFields(file: string, fields: string[])
+
+// Check output message at specific log level
+printsMessage(options: { level: string, regex: RegExp })
+```
+
+### Assertion Helpers
+
+```typescript
+// Assert path exists (throws if missing)
+verifyPath(path: string)
+
+// Assert path does NOT exist (throws if present)
+verifyNoPath(path: string)
+```
+
+### Cleanup Helpers
+
+```typescript
+// Safe file removal (no error if missing) - from src/core/path.ts
+import { safeRemoveIfExists } from "../../../src/core/path.ts";
+safeRemoveIfExists(path: string)
+```
+
 ### Path Helpers
 
 ```typescript

tests/docs/project/prepost/script-env-vars/.gitignore

@@ -0,0 +1,2 @@
+/.quarto/
+**/*.quarto_ipynb

tests/docs/project/prepost/script-env-vars/_quarto.yml

@@ -0,0 +1,10 @@
+project:
+  type: website
+  pre-render: check-env.ts
+
+website:
+  title: "script-env-vars"
+
+format:
+  html:
+    theme: cosmo

tests/docs/project/prepost/script-env-vars/about.qmd

@@ -0,0 +1,5 @@
+---
+title: About
+---
+
+About page

tests/docs/project/prepost/script-env-vars/check-env.ts

@@ -0,0 +1,6 @@
+const env = {
+  progress: Deno.env.get("QUARTO_PROJECT_SCRIPT_PROGRESS") ?? null,
+  quiet: Deno.env.get("QUARTO_PROJECT_SCRIPT_QUIET") ?? null,
+};
+
+Deno.writeTextFileSync("env-dump.json", JSON.stringify(env));

tests/docs/project/prepost/script-env-vars/index.qmd

@@ -0,0 +1,5 @@
+---
+title: Test
+---
+
+Hello

tests/smoke/project/project-prepost.test.ts

@@ -9,7 +9,7 @@ import { docs } from "../../utils.ts";
 import { join } from "../../../src/deno_ral/path.ts";
 import { existsSync } from "../../../src/deno_ral/fs.ts";
 import { testQuartoCmd } from "../../test.ts";
-import { fileExists, noErrors, printsMessage, verifyNoPath, verifyPath } from "../../verify.ts";
+import { fileExists, noErrors, printsMessage, validJsonWithFields, verifyNoPath, verifyPath } from "../../verify.ts";
 import { normalizePath, safeRemoveIfExists } from "../../../src/core/path.ts";
 
 const renderDir = docs("project/prepost/mutate-render-list");
@@ -96,4 +96,31 @@ testQuartoCmd(
         safeRemoveIfExists(outputPath);
       }
     }
-  )
+  )
+
+// Verify that pre-render scripts receive QUARTO_PROJECT_SCRIPT_PROGRESS
+// and QUARTO_PROJECT_SCRIPT_QUIET environment variables
+const scriptEnvDir = docs("project/prepost/script-env-vars");
+const scriptEnvDirAbs = join(Deno.cwd(), scriptEnvDir);
+const envDumpPath = join(scriptEnvDirAbs, "env-dump.json");
+const scriptEnvOutDir = join(scriptEnvDirAbs, "_site");
+
+testQuartoCmd(
+  "render",
+  [scriptEnvDir],
+  [
+    noErrors,
+    validJsonWithFields(envDumpPath, {
+      progress: "1",
+      quiet: "0",
+    }),
+  ],
+  {
+    teardown: async () => {
+      safeRemoveIfExists(envDumpPath);
+      if (existsSync(scriptEnvOutDir)) {
+        await Deno.remove(scriptEnvOutDir, { recursive: true });
+      }
+    },
+  },
+);