Skip to content

[Fix] Astra provider CI fails when focused tests are omitted - #1530

Draft
zoomote[bot] wants to merge 3 commits into
mainfrom
fix/gpt-6-astra-mutation-ci-342bv6tzrna6w
Draft

[Fix] Astra provider CI fails when focused tests are omitted#1530
zoomote[bot] wants to merge 3 commits into
mainfrom
fix/gpt-6-astra-mutation-ci-342bv6tzrna6w

Conversation

@zoomote

@zoomote zoomote Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

​Opened on behalf of @navedmerchant. View the task or mention @roomote for follow-up asks.

What changed

  • Keeps hyphen-suffixed direct tests such as openai-native-usage.spec.ts in changed-code mutation runs instead of dropping them when a dot-suffixed direct test also exists.
  • Adds focused decision-table coverage for Astra reasoning normalization, Responses Lite routing, cache behavior, temperature, token fields, and sequential tool calls across the providers merged in [Feat] Add verified GPT-6 Astra support across providers #1506.
  • Simplifies equivalent cache-total and exact-model predicates so mutation testing measures observable behavior rather than module-initialization artifacts.

Why this change was made

The mutation gate on #1506 omitted an existing focused usage suite and left provider guard branches under-specified. The feature was merged while the final CI remediation was still being validated, so these fixes are isolated in this follow-up.

Impact

Changed-code mutation testing now exercises all directly related focused suites and passes for the Astra provider changes. Runtime behavior from #1506 is preserved.

Related PRs

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Review status

This PR was opened by an automated account. A human maintainer must verify the change intent, provenance, and validation before merging.

Current step: Resolve the merge conflicts. The review sequence resumes after the branch is mergeable.

Review-state labels are managed by this workflow; do not edit them manually.

@codecov

codecov Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions github-actions Bot added has-conflicts PR has merge conflicts with the base branch and removed has-conflicts PR has merge conflicts with the base branch labels Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-conflicts PR has merge conflicts with the base branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant