[CDRIVER-3285] Remove hard-coded URIs and ports in transaction tests Created: 08/Aug/19 Updated: 09/Feb/23 |
|
| Status: | Backlog |
| Project: | C Driver |
| Component/s: | libmongoc |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Kevin Albertson | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Epic Link: | CDRIVER-2733 |
| Description |
|
The transactions tests with useMultipleMongoses: true currently is implemented in our test runner by hard coding the hosts "localhost:27017" and "localhost:27018". We should provide a way to override that with environment variables, to be consistent with the rest of the test runner. The transactions_test_before_test and transactions_test_after_test in test-mongoc-transactions.c also appears to hard-code URIs in their calls to _reset_server and _disable_failpoints, respectively. This is done regardless of whether useMultipleMongoses is set, so it also applies to RS environments. |
| Comments |
| Comment by Jeremy Mikola [ 07/Jun/21 ] |
|
Amending the title and description to refer to other hard-coded URIs/ports in the transaction tests. |