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

Fix populate_aip_stored_dates command #698

Merged

Conversation

replaceafill
Copy link
Member

@replaceafill replaceafill commented Mar 19, 2024

This also modifies the coverage configuration file to include management commands in coverage reports.

I also noticed flakiness on the test_space test when the new test runs first which ended up being a problem with the caplog fixture similar to #680. So I added an autouse fixture to the common tests that avoids resetting the root logger from any management command.

Connected to archivematica/Issues#1670

@replaceafill replaceafill force-pushed the dev/issue-1670-fix-populate-aip-stored-dates-command branch from 8db3e58 to be59b73 Compare March 19, 2024 17:02
@replaceafill replaceafill force-pushed the dev/issue-1670-fix-populate-aip-stored-dates-command branch from be59b73 to e538093 Compare March 22, 2024 17:02
@replaceafill replaceafill merged commit 05387d3 into qa/0.x Mar 22, 2024
9 checks passed
@replaceafill replaceafill deleted the dev/issue-1670-fix-populate-aip-stored-dates-command branch March 22, 2024 17:09
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.

2 participants