Skip to content

[smg][ci] Migrate Response API e2e tests to shared infrastructure#16680

Merged
slin1237 merged 3 commits intomainfrom
smg-ci-n/21
Jan 8, 2026
Merged

[smg][ci] Migrate Response API e2e tests to shared infrastructure#16680
slin1237 merged 3 commits intomainfrom
smg-ci-n/21

Conversation

@slin1237
Copy link
Copy Markdown
Collaborator

@slin1237 slin1237 commented Jan 7, 2026

Migrate tests from e2e_test/e2e_response_api/features/ to e2e_test/responses/ using the new parallel-ready test infrastructure with pytest markers.

Changes:

  • Add e2e_test/responses/ module with migrated tests
  • Split tests into separate classes per backend (Cloud, Local, Harmony)
  • Use @pytest.mark.e2e, @pytest.mark.model(), @pytest.mark.gateway() markers
  • Add 'responses' job to pr-test-rust.yml workflow with parallel execution

Test coverage (100% migration verified):

  • test_state_management.py: 22 effective test runs
  • test_basic_crud.py: 14 effective test runs
  • test_streaming_events.py: 3 effective test runs
  • test_structured_output.py: 3 effective test runs
  • test_tools_call.py: 23 effective test runs

Checklist

Review Process

  1. Ping Merge Oncalls to start the PR flow. See the PR Merge Process.
  2. Get approvals from CODEOWNERS and other reviewers.
  3. Trigger CI tests with comments (/tag-run-ci-label, /rerun-failed-ci, /tag-and-rerun-ci) or contact authorized users to do so.
  4. After green CI and required approvals, ask Merge Oncalls to merge.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@slin1237 slin1237 added the run-ci label Jan 7, 2026
@slin1237 slin1237 force-pushed the smg-ci-n/21 branch 6 times, most recently from ad5fca4 to f14bab5 Compare January 8, 2026 00:22
Migrate tests from e2e_test/e2e_response_api/features/ to e2e_test/responses/
using the new parallel-ready test infrastructure with pytest markers.

Changes:
- Add e2e_test/responses/ module with migrated tests
- Split tests into separate classes per backend (Cloud, Local, Harmony)
- Use @pytest.mark.e2e, @pytest.mark.model(), @pytest.mark.gateway() markers
- Add 'responses' job to pr-test-rust.yml workflow with parallel execution

Test coverage (100% migration verified):
- test_state_management.py: 22 effective test runs
- test_basic_crud.py: 14 effective test runs
- test_streaming_events.py: 3 effective test runs
- test_structured_output.py: 3 effective test runs
- test_tools_call.py: 23 effective test runs

Co-authored-by: Chang Su <chang.s.su@oracle.com>
Co-authored-by: key4ng <rukeyang@gmail.com>
@slin1237 slin1237 merged commit 109fe03 into main Jan 8, 2026
68 of 82 checks passed
@slin1237 slin1237 deleted the smg-ci-n/21 branch January 8, 2026 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant