-
Notifications
You must be signed in to change notification settings - Fork 236
Closed
Labels
P1Medium priority - Should doMedium priority - Should docuda.coreEverything related to the cuda.core moduleEverything related to the cuda.core moduletestImprovements or additions to testsImprovements or additions to tests
Milestone
Description
This code
cuda-python/cuda_core/tests/conftest.py
Lines 63 to 66 in 1c6f3bc
| try: # noqa: SIM105 | |
| os.remove(f) | |
| except FileNotFoundError: | |
| pass # noqa: SIM105 |
could fail on Windows if the module is still loaded (and therefore the module file is in use):
On Windows, attempting to remove a file that is in use causes an exception to be raised
https://docs.python.org/3/library/os.html#os.remove
As per suggestion by @rwgk we should move cleanup code from test shutdown to test setup time.
Metadata
Metadata
Assignees
Labels
P1Medium priority - Should doMedium priority - Should docuda.coreEverything related to the cuda.core moduleEverything related to the cuda.core moduletestImprovements or additions to testsImprovements or additions to tests
Type
Projects
Status
Done