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

MAYA-000000 Reduce testUsdExportPackage failures #2888

Merged
merged 1 commit into from
Feb 21, 2023

Conversation

pierrebai-adsk
Copy link
Collaborator

The test expects that the temp folder be devoid of certain files, but it sometimes happens that files matching the pattern existed before the test started and are undeletable. (Locked by an other process or owned by another user?)

Record the list of pre-existing files and ignore them.

The test expects that the temp folder be devoid of certain files, but it sometimes happens that files matching the pattern existed before the test started and are undeletable. (Locked by an other process or owned by another user?)

Record the list of pre-existing files and ignore them.
@pierrebai-adsk pierrebai-adsk added enhancement New feature or request unit test Related to unit tests (both python or c++) labels Feb 17, 2023
Copy link
Collaborator

@vlasovi vlasovi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Thanks Pierre for doing that.

@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Feb 17, 2023
@seando-adsk seando-adsk removed the enhancement New feature or request label Feb 21, 2023
@seando-adsk seando-adsk merged commit 9840fa1 into dev Feb 21, 2023
@seando-adsk seando-adsk deleted the bailp/MAYA-000000/reduce-random-test-failures branch February 21, 2023 20:06
@seando-adsk
Copy link
Collaborator

@pierrebai-adsk Thanks for this fix. Hopefully we've see the last of the random failure of this test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Development process is finished, PR is ready for merge unit test Related to unit tests (both python or c++)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants