Repository navigation
test(history): give the coverage fixtures the knownGaps field the spec now requires - #40
Merged
Merged
Conversation
…c now requires The upstream spec lists knownGaps as required on both the entity and the park coverage documents. The three fixtures in test_history.py omit it, so the regenerated models in the spec-drift PR reject them and eight tests fail. Models allow extra fields, so the fixtures pass on the current models too and can land first. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
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
PARK_COVERAGE = { ... "fields": {}, "entities": [], + "knownGaps": [], } ENTITY_COVERAGE = { ..., "kinds": {}, + "knownGaps": [] } # plus the inline body in TestCoverage.test_coverage_returns_what_existsThe upstream spec lists
knownGapsas required on the entity and park coverage documents. The spec-drift PR (#35) regenerates the models with that field required, and these three fixtures omit it, so eight tests intests/unit/test_history.pyfail validation there.ApiModelallows extra fields, so the fixtures also pass on the current models and can land first.Evidence
tests/unit/test_history.pyhas 8 failures:HistoryParkCoverageDocumentand the entity document reportknownGaps"Field required".main,pytest tests/unitgives 457 passed. With these fixtures applied on top of the spec-drift branch,tests/unit/test_history.pygives 25 passed.tests/unit/test_backfill.pyfailures on the spec-drift branch, which are the CSV column set changing (36 to 40 data columns). That file is a contract shared with the JavaScript SDK and needs its own change.Merge Danger
Door: two-way
Blast Radius: tests only. No library code changes.
🤖 Generated with Claude Code