-
-
Notifications
You must be signed in to change notification settings - Fork 56.4k
Fix tests writing to current work dir #24343
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
Conversation
| char const * filenames[] = { | ||
| "core_io_base64_other_test.yml", | ||
| "core_io_base64_other_test.xml", | ||
| "core_io_base64_other_test.json", | ||
| "core_io_base64_other_test.yml?base64", | ||
| "core_io_base64_other_test.xml?base64", | ||
| "core_io_base64_other_test.json?base64", | ||
| 0 | ||
| }; | ||
| char const * real_name[] = { | ||
| "core_io_base64_other_test.yml", | ||
| "core_io_base64_other_test.xml", | ||
| "core_io_base64_other_test.json", | ||
| "core_io_base64_other_test.yml", | ||
| "core_io_base64_other_test.xml", | ||
| "core_io_base64_other_test.json", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you remove it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this test we iterate over 3 formats (yml, xml,json) and 2 modes (default, base64). These two arrays contained 6 items each for actual file name and accessed file name (with base64 suffix). I've replaced these arrays with calculated values: name = filename[n/2], suffix = n%2 ? '' : 'base64'.
|
i tried every thing even that change file location D drive to C Drive . Usage questions should go to Users OpenCV Q/A forum: https://forum.opencv.org/ |
asmorkalov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Fix tests writing to current work dir opencv#24343 Several tests were writing files in the current work directory and did not clean up after test. Moved all temporary files to the `/tmp` dir and added a cleanup code.
Fix tests writing to current work dir opencv#24343 Several tests were writing files in the current work directory and did not clean up after test. Moved all temporary files to the `/tmp` dir and added a cleanup code.
Fix tests writing to current work dir opencv#24343 Several tests were writing files in the current work directory and did not clean up after test. Moved all temporary files to the `/tmp` dir and added a cleanup code.
Several tests were writing files in the current work directory and did not clean up after test. Moved all temporary files to the
/tmpdir and added a cleanup code.