[AccountController ] Change AC tests to be multi-threaded #3338
Conversation
There was a problem hiding this comment.
@pronebird reviewed 3 of 3 files at r1.
Reviewable status: 2 of 3 files reviewed, 1 unresolved discussion (waiting on @simonwicky)
There was a problem hiding this comment.
@pronebird reviewed 1 of 1 files at r2.
Reviewable status: all files reviewed (commit messages unreviewed), 1 unresolved discussion (waiting on @simonwicky)
There was a problem hiding this comment.
@pronebird reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @simonwicky)
simonwicky
force-pushed
the
simon/ac/multithread_tests
branch
from
bf0af36 to
5cb48d2
Compare
There was a problem hiding this comment.
@pronebird reviewed 2 of 2 files at r4.
Reviewable status: all files reviewed (commit messages unreviewed), 1 unresolved discussion (waiting on @simonwicky)
|
Superseded by #3460 |
Description
Some AccountController tests seem to fail sometimes.
This PR aims to fix that by making the tests multi threaded allowing the AccountController to be moor free in a separate thread
This change is