Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactor cache handling test to make it more robust #1548

Merged
merged 1 commit into from
Oct 27, 2024

Conversation

mre
Copy link
Member

@mre mre commented Oct 27, 2024

This pull request includes a refactoring of the cache handling test to make it more robust. The cache handling test now ensures a clean state before running.

@mre mre force-pushed the bugfix/cache-test-robustness branch from 04a5258 to 4ad08fb Compare October 27, 2024 00:12
@mre mre force-pushed the bugfix/cache-test-robustness branch from 4ad08fb to ad1c84f Compare October 27, 2024 00:14
@mre
Copy link
Member Author

mre commented Oct 27, 2024

@dmathieu, this should make the cache check a bit more robust.

We're improving the test setup and cleanup process, as well as ensuring a clean state before running tests. The most important changes include adding a delay to ensure file operations complete and improving file handling for test consistency.

Here's hoping that this will deal with the flakiness.

@mre mre changed the title Refactor cache handling test and add JSON format support Refactor cache handling test to make it more robust Oct 27, 2024
@mre mre merged commit bc0b05b into master Oct 27, 2024
7 checks passed
@mre mre deleted the bugfix/cache-test-robustness branch October 27, 2024 00:19
@mre mre mentioned this pull request Oct 27, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant