You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the task
The api/fixture/test/0003_add_bceid_users.py file should be updated so that each test user is assigned a distinct organization; currently, 2 distinct users can be assigned the same organization and if that happens, some credit transfer tests will fail.
Purpose
Makes it so that our unit tests don't fail intermittently.
Acceptance Criteria
The credit transfer unit tests pass.
Developer Checklist
(1) Update the code in the file mentioned above (0003_add_bceid_users.py) in the manner specified above.
(2) Update the test_list_transfer and test_transfer_pass tests in test_credit_transfers.py so that they don't use transaction.on_commit() (that is there because, earlier, that was Tim's suggested solution to the intermittent failure of these tests, but it actually makes it so that the assertions don't run because the transactions are never actually committed!).
The text was updated successfully, but these errors were encountered:
Describe the task
The api/fixture/test/0003_add_bceid_users.py file should be updated so that each test user is assigned a distinct organization; currently, 2 distinct users can be assigned the same organization and if that happens, some credit transfer tests will fail.
Purpose
Makes it so that our unit tests don't fail intermittently.
Acceptance Criteria
Developer Checklist
(1) Update the code in the file mentioned above (0003_add_bceid_users.py) in the manner specified above.
(2) Update the
test_list_transfer
andtest_transfer_pass
tests in test_credit_transfers.py so that they don't usetransaction.on_commit()
(that is there because, earlier, that was Tim's suggested solution to the intermittent failure of these tests, but it actually makes it so that the assertions don't run because the transactions are never actually committed!).The text was updated successfully, but these errors were encountered: