search

application-research / autoretrieve

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

Update to use new filclient event reporter #106

Closed rvagg closed 2 years ago

rvagg commented 2 years ago

Ref: https://github.com/application-research/filclient/pull/89 - depends on that branch's commit for filclient

This is really about cleaning up the "event" vs "state" thing now we've moved to a pure event model and not tracking state in our reporting, but it also adds "receivedCids" to the success case for reporting.

codecov-commenter commented 2 years ago

Codecov Report

Merging #106 (c0dc81b) into master (01a3307) will decrease coverage by 0.34%. The diff coverage is 10.38%.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
- Coverage   12.14%   11.79%   -0.35%     
==========================================
  Files          16       16              
  Lines        2116     2136      +20     
==========================================
- Hits          257      252       -5     
- Misses       1845     1869      +24     
- Partials       14       15       +1
Impacted Files Coverage Δ
filecoin/retriever.go 0.00% <0.00%> (ø)
filecoin/retrieverevents.go 0.00% <0.00%> (ø)
filecoin/eventrecorder/eventrecorder.go 72.09% <100.00%> (ø)
blocks/manager.go 80.00% <0.00%> (-5.00%) :arrow_down:

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 01a3307...c0dc81b. Read the comment docs.