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
ci: temp fix for models test #2949
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2949 +/- ##
=======================================
Coverage 70.62% 70.62%
=======================================
Files 104 104
Lines 9534 9534
=======================================
Hits 6733 6733
Misses 2801 2801 |
@@ -49,6 +50,8 @@ def test_models(tmpdir: "Path"): | |||
) as testmodel: | |||
testmodel1tag = testmodel.tag | |||
|
|||
time.sleep(1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we can check whether this is running in actions
time.sleep(1) | |
if os.getenv("GITHUB_ACTIONS"): | |
... |
So that when test run locally we don't have to sleep it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please address address @aarnphm's suggestion in a following PR if needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we shouldn't special case our tests too much, it'll just make debugging harder.
Though I should probably debug what's going on here at some point...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still think we can use the mocked timestamp for unit testing purpose but I'm OK with this change, too.
Another temporary fix for the weird store issue that seems to occur only on actions.