application-research / autoretrieve

A server to make GraphSync data accessible on IPFS
22 stars 7 forks source link

fix: timestamp test resolution - allow for millisecond boundary cross #141

Closed rvagg closed 1 year ago

rvagg commented 1 year ago

Flake experienced @ https://github.com/application-research/autoretrieve/actions/runs/3451782684/jobs/5761244626

I think this gives us a half-millisecond resolution rather than risking hitting right on the millisecond boundary and failing.

codecov-commenter commented 1 year ago

Codecov Report

Base: 20.46% // Head: 20.43% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (988f535) compared to base (e32674d). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #141 +/- ## ========================================== - Coverage 20.46% 20.43% -0.03% ========================================== Files 16 16 Lines 2302 2354 +52 ========================================== + Hits 471 481 +10 - Misses 1801 1841 +40 - Partials 30 32 +2 ``` | [Impacted Files](https://codecov.io/gh/application-research/autoretrieve/pull/141?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research) | Coverage Δ | | |---|---|---| | [blocks/manager.go](https://codecov.io/gh/application-research/autoretrieve/pull/141/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-YmxvY2tzL21hbmFnZXIuZ28=) | `80.00% <0.00%> (-5.00%)` | :arrow_down: | | [filecoin/retriever.go](https://codecov.io/gh/application-research/autoretrieve/pull/141/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-ZmlsZWNvaW4vcmV0cmlldmVyLmdv) | `32.05% <0.00%> (-2.60%)` | :arrow_down: | | [filecoin/activeretrievals.go](https://codecov.io/gh/application-research/autoretrieve/pull/141/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research#diff-ZmlsZWNvaW4vYWN0aXZlcmV0cmlldmFscy5nbw==) | `66.02% <0.00%> (+2.71%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=application-research)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.