Repository navigation
test: cover every public member in the allocation properties - #39
Merged
Merged
Conversation
V9 audit (M7): every Combine and CombineAsync arity, FromNullable(factory) for references, reading and comparing the case types and ValidationError now have allocation lines; the class remarks list every exclusion. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012weRUz3QcTAaQw32ZmQkoK
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.
What
Card 7.1 of milestone M7 (1.0.0): the V9 audit. Every public member now has a line in
AllocationPropertiesor is listed as an exclusion in its remarks. Every new line passed on the first run (0 B), so the library did not change.valid,new Valid, conversions (also generic), converting an existingInvalidCreating_a_valid_validation…,Converting_an_existing_invalid_case…switchpositional/type/generic,is),TryGetValuePattern_matching…Match,OrDefault×2Reading…Map,MapError,Then,Ensure×2 (kept),Select, a queryTransforming_a_valid_validation…Zip×2,Apply,Combinefor every arity 2–8 (new: 4–7)Combining_valid_validations…Check×2,FromNullable×4 (new: reference with a factory),new ValidationErrorBuilding_blocks_on_the_valid_path…MatchAsync×2,MapAsync,ThenAsync,CombineAsyncfor every arity 2–8 (new: 3–7) with completed tasksAsync_members_with_completed_tasks…Equals,==,!=,GetHashCode;Valid==!=EqualsGetHashCodeValueDeconstruct(new)Comparing…Invalid:==,!=,Equals,GetHashCode,Errors,Deconstruct;ValidationError==,GetHashCode,DeconstructReading_and_comparing_existing_invalid_cases…ToString(a string); everything that creates errors (invalid,MapErrorof an invalid one, accumulation, a failedEnsure/Check,FromNullable(null));Equals(object)(boxes);is null(boxes the case); sequences (like LINQ)Closes #19
Checklist
dotnet test --solution dotValid.slnxis green locally (Release, allocation properties included)🤖 Generated with Claude Code
https://claude.ai/code/session_012weRUz3QcTAaQw32ZmQkoK