mock icon indicating copy to clipboard operation
mock copied to clipboard

Improve test coverage for time.ts

Open KyleJune opened this issue 4 years ago • 1 comments

I removed the last 2 tests here because they relied on system time and would intermittently fail. Need to figure out a better way to test FakeTime.

https://github.com/udibo/mock/blob/a22be7fe275697da87fb534e0c92c398eade28d7/time_test.ts#L303-L360

KyleJune avatar Sep 27 '20 20:09 KyleJune

I attempted to resolve the issue but the test would still intermittently fail on MacOS on GitHub actions. The failures were always related to it iterating forward faster than dictated by my delays. See the commits from that PR for more information about what I tried.

Open to suggestions on how to get test coverage for this behavior to be reliable in CI.

KyleJune avatar Nov 14 '21 06:11 KyleJune