Skip to content

fix: ensure scenario summary artifact is written to CI path#316

Merged
jverdicc merged 1 commit intomainfrom
codex/fix-scenario-summary-artifact-writing
Mar 5, 2026
Merged

fix: ensure scenario summary artifact is written to CI path#316
jverdicc merged 1 commit intomainfrom
codex/fix-scenario-summary-artifact-writing

Conversation

@jverdicc
Copy link
Owner

@jverdicc jverdicc commented Mar 5, 2026

Motivation

  • Ensure the scenario summary artifact artifacts/scenarios/summary.json is produced at the repository root so the CI artifact existence check does not fail when running the scenario system test.

Description

  • Update scripts/run_scenarios.sh to pass an absolute SCENARIO_ARTIFACT_DIR by setting SCENARIO_ARTIFACT_DIR="$(pwd)/artifacts/scenarios" when invoking cargo test -p evidenceos-daemon --test scenarios_system -- --nocapture so the test writes into the repo-level artifact directory.

Testing

  • Ran bash scripts/run_scenarios.sh and confirmed artifacts/scenarios/summary.json exists and is non-empty, ran cargo fmt --all && cargo fmt --check, and ran cargo test --workspace with final verification showing FINAL:0.

Codex Task

@jverdicc jverdicc merged commit ed9891d into main Mar 5, 2026
6 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant