-
Notifications
You must be signed in to change notification settings - Fork 9
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
Revisit tests #180
Comments
|
Hard to debug something that happens sporadically and I cannot reproduce locally 😞 I'll change the test logic to something simpler. It is only trying to check if the new directory has a different inode to ensure it is a new directory, but I have another (simpler) idea that might do the same. |
@ocefpaf today commented:
Sure, but not right now! Those tests are not very important.
I'm opening this issue to remind us to give the test suite another look later on.
Ref: #179 and 27b25e8 (the latter disabled part of the cbi tests)
The text was updated successfully, but these errors were encountered: