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

Ensure fixed order in FindFileTimeDependentTest #77

Merged
merged 1 commit into from
Jan 8, 2022

Conversation

pthariensflame
Copy link
Contributor

The order of tests run in org.unix4j.unix.FindFileTimeDependentTest needs to be ensured in case the defaults change in the future (as JUnit 4 reserves the right to do); otherwise find_fileCreatedBeforeNow might run after tests that cause the file being tested on to no longer have the correct creation date.

This issue was found and confirmed by iDFlakies.

The order of tests run in `org.unix4j.unix.FindFileTimeDependentTest` needs to be ensured in case the defaults change in the future (as JUnit 4 reserves the right to do); otherwise `find_fileCreatedBeforeNow` might run after tests that cause the file being tested on to no longer have the correct creation date.

This issue was found and confirmed by [`iDFlakies`](https://github.com/iDFlakies/iDFlakies).
pthariensflame added a commit to pthariensflame/idoft that referenced this pull request Nov 25, 2021
darko-marinov pushed a commit to TestingResearchIllinois/idoft that referenced this pull request Nov 25, 2021
@terzerm terzerm self-assigned this Jan 8, 2022
@terzerm terzerm added bug and removed bug labels Jan 8, 2022
@terzerm
Copy link
Member

terzerm commented Jan 8, 2022

Thanks for reporting. Your fix makes sense!

@terzerm terzerm merged commit 062b0d5 into tools4j:master Jan 8, 2022
@pthariensflame pthariensflame deleted the patch-1 branch January 8, 2022 07:42
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