test(evals): isolate required backend preflight fixture - #1122
Merged
Merged
Conversation
Collaborator
Author
|
Runtime shard 3 passed all 74 selected tests with zero failures. The job failed afterward because GitHub's artifact service timed out ( No code correction is needed for this failure. A job-only rerun will retain the successful suites once GitHub permits the rerun after the active workflow finishes. The full runtime support/adapter suite already passed. |
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
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.
The evaluation runner's missing-backend test used the packaged invariant profile. After #1120 made strategy groups continue on failure, that backend became optional and the test reached an unrelated reference-cache error on clean CI runners.
Use a local topology with one required fixture task, an inert prompt, and a deliberately absent command. This preserves the real blocking preflight, exact error, and no-run/no-workflow assertions while removing dependence on reference caches. Production code is unchanged.
Validation: full evaluation suite passed all 450 tests; the focused regression, strict lint, formatting, and diff checks passed. An independent agent reviewed the change with no findings. This resolves the package-gate failure found during v0.0.25 release validation.
Greptile Summary
This PR isolates the missing-backend preflight regression test from packaged strategy groups and reference caches.
Confidence Score: 5/5
The PR appears safe to merge; the isolated fixture preserves the intended blocking preflight behavior without relying on unrelated packaged resources.
The replacement topology is valid, its command is deterministically absent under the test’s local PATH, and the existing assertions continue to verify failure before any run or workflow is created.
Important Files Changed
Reviews (1): Last reviewed commit: "test(evals): isolate required backend pr..." | Re-trigger Greptile
Context used: