msupply-foundation / open-msupply

Open mSupply represents our most recent advancement in the Logistics Management Information System (LMIS), expanding on more than two decades of development inherited from the well-established legacy of the original mSupply.
https://msupply.foundation/open-msupply/
Other
20 stars 12 forks source link

Intermittent test failure: approved quantity transfer #4608

Open lache-melvin opened 4 weeks ago

lache-melvin commented 4 weeks ago

What went wrong? 😲

There's some kind of race condition/dependency in the tests - is it an actual bug?

Expected behaviour 🤔

Tests consistently passss

How to Reproduce 🔨

Steps to reproduce the behaviour:

  1. Run backend tests
  2. Maybe run many time?
  3. Requisition transfer test will fail, approved quantity didn't get transferred?

Your environment 🌱

andreievg commented 3 weeks ago

There has been some fixes for this in this PR, but that was for postgres, do we have issues with sqlite ?

https://github.com/msupply-foundation/open-msupply/pull/3904

lache-melvin commented 1 week ago

This is now fairly consistently failing our CI build - what changed? Can we bump severity here, I think being able to rely on CI checks should be an important part of our workflow?