Skip to content

TEST Add Crescendo scenario resume coverage - #2343

Merged
Roman Lutz (romanlutz) merged 3 commits into
microsoft:mainfrom
romanlutz:romanlutz-scenario-crescendo-resume-test
Aug 7, 2026
Merged

TEST Add Crescendo scenario resume coverage#2343
Roman Lutz (romanlutz) merged 3 commits into
microsoft:mainfrom
romanlutz:romanlutz-scenario-crescendo-resume-test

Conversation

@romanlutz

Copy link
Copy Markdown
Contributor

Description

Crescendo scenario recovery needs end-to-end regression coverage to ensure retries and resumed runs do not leak stale state between objectives. This adds a test-only scenario using real PyRIT orchestration, attack, target, scorer, and SQLite memory components while mocking only the OpenAI SDK request boundary.

The two-objective schedule covers rate limiting, timeout persistence, malformed adversarial responses, refusal backtracking, partial failure, and successful resume. It also verifies conversation attribution, retry metadata, run-state transitions, unique persisted records, and scorer inputs.

Tests and Documentation

  • uv run --offline --no-sync pytest tests/unit/scenario/core/test_scenario_crescendo_resume.py -q (1 passed)
  • uv run --offline --no-sync pytest tests/unit/scenario/core tests/unit/executor/attack/multi_turn/test_crescendo.py -q (508 passed before merging current main)
  • uv run --offline --no-sync ruff format --check tests/unit/scenario/core/test_scenario_crescendo_resume.py
  • uv run --offline --no-sync ruff check tests/unit/scenario/core/test_scenario_crescendo_resume.py
  • uv run --offline --no-sync ty check tests/unit/scenario/core/test_scenario_crescendo_resume.py
  • git diff --check

Online dependency resolution was blocked by a PyPI TLS handshake failure, so validation used the frozen dependency set restored from the local uv cache. Documentation and JupyText are not applicable to this test-only change.

Copilot AI added 3 commits August 4, 2026 01:36
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 0d1f97bd-49a5-4a6d-b814-fecef1e2a8cb
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 67c023ca-8b50-48de-88f1-0e2cc3025996
@varunj-msft varunj-msft self-assigned this Aug 7, 2026
@romanlutz
Roman Lutz (romanlutz) added this pull request to the merge queue Aug 7, 2026
Merged via the queue into microsoft:main with commit f823656 Aug 7, 2026
47 checks passed
@romanlutz
Roman Lutz (romanlutz) deleted the romanlutz-scenario-crescendo-resume-test branch August 7, 2026 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants