Repository navigation
test: assert observed results instead of pins - #1509
Closed
ScriptedAlchemy wants to merge 4 commits into
Closed
ScriptedAlchemy wants to merge 4 commits into
ScriptedAlchemy wants to merge 4 commits into
Conversation
Replace constant pins, absence-only checks, and weak is_ok/is_err assertions with calls that match a literal observed result. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
|
Assert typed refusals and the text users see instead of is_err, is_ok, and toBeTruthy checks that pass for any result. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Pointer memo keyed only on mtime and size, so a same-quantum rewrite of equal length still compared against the previous pointer. Hash the bytes before reuse. Align generation and shard refusals with the error the validator actually returns. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Owner
Author
ScriptedAlchemy
deleted the
cursor/test-behavior-not-implementation-7532
branch
October 11, 2026 00:05
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.
Summary
is_ok/is_err/toBeTruthyassertions do not count.3e6167b265(test(code-index): re-pin partitioned codec onto extractor.rust.v8) into this branch. No conflicts.CompareAndSwapmiss inpublication_over_an_undecodable_active_generation_refuses_a_moved_pointerafter the tip landedNotApplicable/ reused. Does not merge feat: TraceDecay V2 delivery (PR8–PR13) + daemon performance, memory, and reliability overhaul #707 to master.Motivation
Principle: test-behavior-not-implementation.
A constant pin fails when the constant is edited and never observes the mechanism that reads it. An absence-only or
toBeTruthycheck still passes if the subject returns the wrong non-empty value. A weakis_errpasses for the wrong refusal.Changes
Typed refusals and observed readouts:
true/falsefor literal revisions, notREVISION == 12.TooLarge.schema_revision == 7. Exactworkers: 0is the typedNonCanonicalrefusal;workers: 2isOk(()).toBeTruthy. Node count is12.9K(figure plus unit), not the figure fragment alone.Publication store:
CodeIndexDemandAdmissionV1::NotApplicableand no longer reads a missingreusedfield onChangedCodeChunkSetV1.Test plan
12.9Kreadout assertionremote_query_request_enforces_shard_inventory_bounds_and_identitygeneration_manifest_requires_matching_canonical_language_revisions(digest mismatch, then order / set mismatch after restamp)source_edit_effect_rejects_zero_catalog_revision,source_edit_effect_rejects_zero_privacy_key_epoch,source_edit_effect_requires_the_exact_current_grantpublication_storelib tests: 27 passed, includingpublication_over_an_undecodable_active_generation_refuses_a_moved_pointerChecklist