chore: ensure the generated asset directory is present

himanshusinghs · himanshusinghs · commit d6930791aff5 · 2025-07-16T13:27:09.000+02:00
diff --git a/.github/workflows/accuracy-tests.yml b/.github/workflows/accuracy-tests.yml
@@ -44,9 +44,9 @@ jobs:
         uses: actions/upload-artifact@v4
         with:
           name: accuracy-test-summary
-          path: .accuracy/tests-summary.html
+          path: .accuracy/test-summary.html
       - name: Comment summary on PR
         if: github.event_name == 'pull_request' && github.event.label.name == 'accuracy-tests'
         uses: marocchino/sticky-pull-request-comment@d2ad0de260ae8b0235ce059e63f2949ba9e05943 # v2
         with:
-          path: .accuracy/tests-summary.html
+          path: .accuracy/test-summary.html
diff --git a/scripts/accuracy/generate-test-summary.ts b/scripts/accuracy/generate-test-summary.ts
@@ -1,4 +1,5 @@
-import { readFile, writeFile } from "fs/promises";
+import path from "path";
+import { readFile, writeFile, mkdir } from "fs/promises";
 import { getAccuracyResultStorage } from "../../tests/accuracy/sdk/accuracy-result-storage/get-accuracy-result-storage.js";
 import {
     AccuracyResult,
@@ -247,6 +248,8 @@ async function generateTestSummary() {
         const testSummary = getTestSummary(comparableAccuracyResult);
         const htmlReport = await generateHtmlReport(comparableAccuracyResult, testSummary, baselineInfo);
 
+        // Ensure that our writable path actually exist.
+        await mkdir(path.dirname(HTML_TESTS_SUMMARY_FILE), { recursive: true });
         await writeFile(HTML_TESTS_SUMMARY_FILE, htmlReport, "utf8");
 
         console.log(`✅ HTML report generated: ${HTML_TESTS_SUMMARY_FILE}`);
diff --git a/scripts/accuracy/run-accuracy-tests.sh b/scripts/accuracy/run-accuracy-tests.sh
@@ -33,7 +33,7 @@ JEST_EXIT_CODE=$?
 # "in-progress". When all the tests are done and jest exits with an exit code of
 # 0, we can safely mark accuracy run as finished otherwise failed.
 
-# This "outside-the-tests-status-update" is arising out of the fact that each
+# This "outside-the-test-status-update" is arising out of the fact that each
 # test suite stores their own accuracy run data in the storage and this setup
 # might lead to data inconsistency when the tests fail. To overcome that each
 # accuracy result entry has a status which by default is "in-progress" and is
diff --git a/tests/accuracy/sdk/constants.ts b/tests/accuracy/sdk/constants.ts
@@ -19,6 +19,6 @@ export const ACCURACY_RESULTS_DIR = path.join(GENERATED_ASSETS_DIR, "results");
 
 export const LATEST_ACCURACY_RUN_NAME = "latest-run";
 
-export const HTML_TESTS_SUMMARY_FILE = path.join(GENERATED_ASSETS_DIR, "tests-summary.html");
+export const HTML_TESTS_SUMMARY_FILE = path.join(GENERATED_ASSETS_DIR, "test-summary.html");
 
 export const HTML_TESTS_SUMMARY_TEMPLATE = path.join(RESOURCES_DIR, "test-summary-template.html");
