Skip to content

test(auth): add non-interactive oauth cassette coverage - #2050

Merged
gold-silver-copper merged 3 commits into
mainfrom
add-noninteractive-oauth-cassettes
Jul 8, 2026
Merged

test(auth): add non-interactive oauth cassette coverage#2050
gold-silver-copper merged 3 commits into
mainfrom
add-noninteractive-oauth-cassettes

Conversation

@gold-silver-copper

Copy link
Copy Markdown
Contributor

Summary

  • stacked on top of fix(auth): allow non-interactive OAuth #2021 while preserving its fix(auth): allow non-interactive oauth clients commit in branch history
  • add ChatGPT live cassette coverage for cached OAuth with allow_device_flow(false)
  • add Copilot live cassette coverage for cached OAuth with allow_device_flow(false)
  • register the new cassette helpers in cassette safety checks

Verification

  • cargo fmt
  • RIG_PROVIDER_TEST_MODE=record cargo test -p rig --all-features --test chatgpt chatgpt::cassette::noninteractive_oauth -- --nocapture --test-threads=1
  • RIG_PROVIDER_TEST_MODE=record cargo test -p rig --all-features --test copilot copilot::noninteractive_oauth_cassette -- --nocapture --test-threads=1
  • RIG_PROVIDER_TEST_MODE=replay cargo test -p rig --all-features --test chatgpt chatgpt::cassette::noninteractive_oauth -- --nocapture --test-threads=1
  • RIG_PROVIDER_TEST_MODE=replay cargo test -p rig --all-features --test copilot copilot::noninteractive_oauth_cassette -- --nocapture --test-threads=1
  • cargo test -p rig --all-features --test chatgpt cassette_safety -- --nocapture
  • cargo test -p rig --all-features --test copilot cassette_safety -- --nocapture
  • cargo test -p rig-core noninteractive_oauth_requires_sign_in_instead_of_device_flow
  • cargo test -p rig --all-features --test chatgpt -- --nocapture --test-threads=1
  • cargo test -p rig --all-features --test copilot -- --nocapture --test-threads=1

@gold-silver-copper
gold-silver-copper added this pull request to the merge queue Jul 8, 2026
Merged via the queue into main with commit 9ab30a8 Jul 8, 2026
6 checks passed
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.

2 participants