Skip to content

test: add tests for ensuring correct properties#686

Merged
korikuzma merged 1 commit intov2from
issue-683
Feb 13, 2026
Merged

test: add tests for ensuring correct properties#686
korikuzma merged 1 commit intov2from
issue-683

Conversation

@korikuzma
Copy link
Contributor

close #683

  • Ensure that ga4gh.inherent properties actually exist in the model (sometimes properties get renamed, but forget to update this field)
  • Ensure that the type property matches the class name (this seems to be the convention)

@korikuzma korikuzma self-assigned this Feb 13, 2026
@korikuzma
Copy link
Contributor Author

Tests are expected to fail since we missed these in #663

@korikuzma korikuzma merged commit 33f8c16 into v2 Feb 13, 2026
13 of 15 checks passed
@korikuzma korikuzma deleted the issue-683 branch February 13, 2026 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consider adding additional tests

2 participants