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

(NFC) Version Check Test - Fix multi-user and concurrent execution #25909

Merged
merged 1 commit into from
Mar 23, 2023

Conversation

totten
Copy link
Member

@totten totten commented Mar 23, 2023

Overview

This is a small change to the way the tests run. It doesn't change the meaning of the test.

Before

  • Writes example file to /tmp/versionCheckTestFile.json.
  • File left behind at end of test.
  • If two users runs the same test at different times, they could interfere with each other.
  • If one user with two concurrent processes run the same test, they could interfere with each other.

After

  • Writes example file to /tmp/VersionCheck-NNNNNNN/versionCheckTestFile.json
  • Temp folder cleaned up at end
  • If two users or two processes run the test, they're fine. They use separate folders.

This is a small change to the way the tests run. It doesn't change the meaning of the test.

Before
------

* Writes example file to `/tmp/versionCheckTestFile.json`.
* File left behind at end of test.
* If two users runs the same test at different times, they could interfere with each other.
* If two processes run the same test at the same time, they could interfere with each other.

After
-----

* Writes example file to `/tmp/VersionCheck-NNNNNNN/versionCheckTestFile.json`
* Temp folder cleaned up at end
* If two users or two processes run the test, they use separate folders
@civibot
Copy link

civibot bot commented Mar 23, 2023

(Standard links)

@civibot civibot bot added the 5.60 label Mar 23, 2023
@totten totten merged commit 7510f3e into civicrm:5.60 Mar 23, 2023
@totten totten deleted the 5.60-vertest branch March 23, 2023 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants