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

Removing DatabaseSupport.clean() #1207

Merged
merged 8 commits into from
Jul 25, 2024
Merged

Removing DatabaseSupport.clean() #1207

merged 8 commits into from
Jul 25, 2024

Conversation

Beckyrose200
Copy link
Contributor

#1144

This PR is continuing in the efforts to remove all DatabaseSupport.clean() functions within the unit tests.

#1144

This PR is continuing in the efforts to remove all DatabaseSupport.clean() functions within the unit tests.
@Beckyrose200 Beckyrose200 added the housekeeping Refactoring, tidying up or other work which supports the project label Jul 24, 2024
@Beckyrose200 Beckyrose200 self-assigned this Jul 24, 2024
@Beckyrose200
Copy link
Contributor Author

Shaved off 96 seconds when running the unit tests
Screenshot 2024-07-25 at 14 40 42
Screenshot 2024-07-25 at 14 46 04

@Beckyrose200 Beckyrose200 marked this pull request as ready for review July 25, 2024 13:52
@Beckyrose200 Beckyrose200 merged commit 29def88 into main Jul 25, 2024
6 checks passed
@Beckyrose200 Beckyrose200 deleted the remove-db-clean branch July 25, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Refactoring, tidying up or other work which supports the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants