Flaky tests for removal of items

[abulvenz]

Hi there,

Thanks for sharing your code kata. The structure looks really nice.
However, when running the tests they sometimes fail. On my machine for bills and products.

Maybe its located around this lines, because only the retrieval of items is waited for, but not for the removal.

api-with-vanilla/src/services/bills.js

Lines 9 to 10 in 9fda40c

	const billList = await models.Bill.findAll();
	billList.forEach(bill => bill.destroy());

Since it's a racing condition maybe you didn't encounter it.

Best
Andi

[Falinor]

Hi Andi,

Thanks for noticing this issue and sorry for the late answer. We've all been pretty much busy.
I ran the tests hundreds of times to finally catch a failing one. Machines are so fast nowadays that we struggle to find this kind of bug.
I am going to propose a fix.

Best regards