Increase sleep time during logs tests

[TylerLoewen]

These changes should hopefully fix this reoccurring CI test failure issue.

[codecov]

Codecov Report

Merging #39 (211978b) into master (1819fca) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
- Coverage   94.28%   94.21%   -0.07%     
==========================================
  Files           5        5              
  Lines         175      173       -2     
==========================================
- Hits          165      163       -2     
  Misses         10       10              

Impacted Files	Coverage Δ
src/handler.jl	92.68% <0.00%> (-0.35%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1819fca...211978b. Read the comment docs.

[mattBrzezinski]

There are a few other sleeps, maybe set a variable for the amount and use it within all of those?

[TylerLoewen]

There are a few other sleeps, maybe set a variable for the amount and use it within all of those?

That is an option but I'm not sure how well that would work since there are sleep values that range from 1, 5, or 10 seconds. If I make them all use the same variable they'd all be 10 seconds which is probably sleeping more than we need for most cases.

[mattBrzezinski]

There are a few other sleeps, maybe set a variable for the amount and use it within all of those?

That is an option but I'm not sure how well that would work since there are sleep values that range from 1, 5, or 10 seconds. If I make them all use the same variable they'd all be 10 seconds which is probably sleeping more than we need for most cases.

Yeah that's fair. Tbh these online tests, and testing in general needs to be re-worked as it's just awkward to get stuff to work together when you're creating/deleting resources really quickly.

[TylerLoewen]

There are a few other sleeps, maybe set a variable for the amount and use it within all of those?

That is an option but I'm not sure how well that would work since there are sleep values that range from 1, 5, or 10 seconds. If I make them all use the same variable they'd all be 10 seconds which is probably sleeping more than we need for most cases.

Yeah that's fair. Tbh these online tests, and testing in general needs to be re-worked as it's just awkward to get stuff to work together when you're creating/deleting resources really quickly.

Ya this is definitely not a permanent solution but hopefully this will make the CI less noisy for the time being.