tools icon indicating copy to clipboard operation
tools copied to clipboard

Refactor tests to use `MemoryFileSystem` for tests that depend on time

Open eliasyishak opened this issue 1 year ago • 0 comments

Currently, we are using the MemoryFileSystem.test() constructor for all tests. This uses a fake clock internally which works great for tests that may fail due to race conditions.

However, all tests don't require a fake clock, as pointed out here, some may actually benefit from using the real clock.

We should instead use the appropriate constructor of MemoryFileSystem on a test-by-test case instead of globally setting it up in setUp

eliasyishak avatar Mar 18 '24 17:03 eliasyishak