Add 1 MS deplay to CleanUpDb to allow time for test{guid}.db to fully…

[SteveBush]

Pull request for bug #2940

Added code to CleanUpDb to wait for one millisecond to allow test{guid}.db to fully close. In the process made CleanUpDb async. I decided not to rename CleanUpDbAsync which I would normally have done.

[CLAassistant]

All committers have signed the CLA.

[iancooper]

Thanks for this.

[iancooper]

@SteveBush At this point we are committing to the v9 branch and then re-basing main so that fixes go into both versions whilst preserving history. I can handle this for you in this case.

[SteveBush]

@SteveBush At this point we are committing to the v9 branch and then re-basing main so that fixes go into both versions whilst preserving history. I can handle this for you in this case.

Sounds great. You might also consider adding an Async to CleanUpDbAsync depending on your method naming style as I added an async modifier. I’m new to your project so I’m still learning your coding style. Lot’s of great work.

[iancooper]

You might also consider adding an Async to CleanUpDbAsync

yeah that tends to be our style

[iancooper]

@SteveBush This change should have been applied via V9. Do you want to check, and then I will close this PR?

[SteveBush]

@SteveBush This change should have been applied via V9. Do you want to check, and then I will close this PR?

Yes. Thank you. I have verified that 47d5c90 fixes the issue.