We use the Sync lifecycle module to follow state changes and take
actions upon them. This is done via async methods which return
Promises resolving when a specific state is transitioned to.
However, in tests, we were modifying the state by hand so the
associated events would not be fired and the Promises waiting for them
would never resolve. This resulted in tests failing from time to time,
especially in the Windows and macOS CI builds.
We replaced all the manual state changes with the appropriate
transitions.
Please make sure the following boxes are checked:
[x] PR is not too big
[x] it improves UX & DX in some way
[ ] it includes unit tests matching the implementation changes
[ ] it includes scenarios matching a new behaviour or has been manually tested
We use the Sync lifecycle module to follow state changes and take actions upon them. This is done via async methods which return Promises resolving when a specific state is transitioned to.
However, in tests, we were modifying the state by hand so the associated events would not be fired and the Promises waiting for them would never resolve. This resulted in tests failing from time to time, especially in the Windows and macOS CI builds.
We replaced all the manual state changes with the appropriate transitions.
Please make sure the following boxes are checked: