natefinch / lumberjack

lumberjack is a log rolling package for Go
MIT License
4.79k stars 591 forks source link

Fix test timing #64

Closed natefinch closed 6 years ago

natefinch commented 6 years ago

tests were failing in CI because the compression and writing to disk in a secondary goroutine was taking longer than expected in tests. The code is still accurate, it's just the test is too timing-specific. This bandaids the problem by giving the tests a longer wait period, but we should really change the test so it polls rather than just waits.