Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade testdatamanager and ensures test data follows correct modelin… #227

Merged
merged 2 commits into from
May 25, 2024

Conversation

mseaton
Copy link
Member

@mseaton mseaton commented May 24, 2024

…g for dispositions to use obs groups

This is a non-functional change. It updates some tests that currently pass but which are based off of invalid data, which is only passing due to the specific nature of how the functionality is written.

The goal here is to ensure the test data is modeled correctly and can demonstrate that existing tests pass if the underlying implementation is updated in the future.

@mseaton mseaton requested a review from mogoodrich May 24, 2024 16:16
@mseaton mseaton merged commit 23929fb into master May 25, 2024
1 check passed
@mseaton mseaton deleted the EA-184-1 branch May 28, 2024 14:10
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.

1 participant