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.

Sign up for GitHub

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

Jump to bottom

Re-use simple Rt definition throughout testing suite #287

Closed
dylanhmorris opened this issue Jul 22, 2024 · 2 comments · Fixed by #372
Closed

Re-use simple Rt definition throughout testing suite #287

dylanhmorris opened this issue Jul 22, 2024 · 2 comments · Fixed by #372
Labels
clean up Good code that could be better testing

Comments

@dylanhmorris
Copy link
Collaborator

see e.g.:

Should we share this across multiple files?

Originally posted by @damonbayer in #275 (comment)
@dylanhmorris dylanhmorris mentioned this issue Jul 22, 2024
Merged
@damonbayer damonbayer added clean up Good code that could be better testing labels Jul 22, 2024
@sbidari
Copy link
Collaborator

sbidari commented Jul 25, 2024 •
edited
Loading

what is DRY? @dylanhmorris @damonbayer

@dylanhmorris
Copy link
Collaborator Author

Stands for "don't repeat yourself"

@sbidari sbidari mentioned this issue Aug 13, 2024
Merged
@damonbayer damonbayer changed the title Consider increasing DRY across testing suite Re-use simple Rt definition throughout testing suite Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
clean up Good code that could be better testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

create and use default rt across tests CDCgov/PyRenew
3 participants
@dylanhmorris @damonbayer @sbidari