6 Sep
2022
6 Sep
'22
6:14 p.m.
On Tue Sep 6 12:08:53 2022 +0000, Huw Davies wrote:
Use `trace()` in tests. However I'm not convinced that this is particularly useful. It would be more interesting to add a test that `DeleteFile()` eventually succeeds, so that we'd notice if we weren't waiting long enough. On that note, I think sleeping for 100ms would be better. On my Win 7 VM it typically succeeds on the 2nd attempt with a 100ms wait.
In my machine and also in the testbot it succeeds after 7 attempts and sometimes it can't make it with 10 attempts when using 100ms, I think 200ms it's good.