-
Notifications
You must be signed in to change notification settings - Fork 21.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Allow setting db with use_postgresql tests
If more than one test is using `use_postgresql` for it's tests and both need to run `db:create` those tests will fail often due to them running in parallel. I tried turning off parallelization for these tests and they still ran in separate processes. Ultimately though, turning off parallelization means we don't aren't testing parallelization. Instead I've added `Process.pid` to the database name so it creates a database specifically for that test. For cases where we need to set an explict name a name can be passed in. I also added an ensure to one of the tests so that the databases get cleaned up and not left behind. Fixes #45114 Fixes #45158
- Loading branch information
1 parent
1554cf1
commit 58f6710
Showing
3 changed files
with
24 additions
and
22 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters