Repository navigation
[v4/p1c-fixtures] test: delete self-generated contract fixtures and tests that pin Rust-only behaviour - #245
Closed
cunninghamcard-bit wants to merge 1 commit into
Closed
cunninghamcard-bit wants to merge 1 commit into
cunninghamcard-bit wants to merge 1 commit into
Conversation
…that pin Rust-only behaviour The V4 migration that follows replaces these tests with tests ported from the AI SDK. They are deleted first, on their own, so the PRs after this one show only code changes. This commit removes only test files (Rust, the seven bindings, the shared contract fixtures and their Node runner) and the CI steps that ran the contract fixtures. No library code changes. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01A3WxDBWBA4ituJjXsHaEAa
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by li
Summary
Before: the shared contract fixtures and several hand-written tests record Rust's own pre-alignment output rather than real data.
After: those self-generated fixtures and the tests that pin Rust-only behaviour are deleted. Two of the deleted files come back later in a form that compiles against the new types: the vendor-alignment PR (#240) restores the upstream-translated Anthropic and Bedrock remaining cases, and the results PR (#238) ports upstream's stream timeout tests in place of the old timeout matrix.
Type of change
Tests, fixtures, CI steps and docs only.
Notes for reviewers
v4/p1b-provider-name-web-types: thescripts/local-ci.shcontract section it removes also carries theProviderNamedrift check, which only goes away with that PR. After it merges, this PR is retargeted to master.v4/p1-prep, unchanged.🤖 Generated with Claude Code