algorand / conduit

Algorand's data pipeline framework.
MIT License
37 stars 26 forks source link

Update metrics to include inner transactions. #105

Closed winder closed 1 year ago

winder commented 1 year ago

Summary

Update metrics to consider inner transactions.

Test Plan

New unit test.

codecov[bot] commented 1 year ago

Codecov Report

Merging #105 (94413ce) into master (442791a) will increase coverage by 2.27%. The diff coverage is 76.68%.

@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
+ Coverage   67.66%   69.93%   +2.27%     
==========================================
  Files          32       36       +4     
  Lines        1976     2428     +452     
==========================================
+ Hits         1337     1698     +361     
- Misses        570      641      +71     
- Partials       69       89      +20     
Impacted Files Coverage Δ
conduit/data/block_export_data.go 100.00% <ø> (+92.30%) :arrow_up:
conduit/metrics/metrics.go 100.00% <ø> (ø)
...duit/plugins/exporters/example/example_exporter.go 87.50% <ø> (-1.39%) :arrow_down:
...duit/plugins/exporters/filewriter/file_exporter.go 81.63% <ø> (-1.06%) :arrow_down:
conduit/plugins/exporters/noop/noop_exporter.go 73.68% <ø> (-3.59%) :arrow_down:
conduit/plugins/importers/algod/metrics.go 100.00% <ø> (ø)
...ins/processors/filterprocessor/filter_processor.go 83.82% <ø> (+3.54%) :arrow_up:
...plugins/processors/filterprocessor/gen/generate.go 34.28% <ø> (ø)
conduit/plugins/processors/noop/noop_processor.go 64.70% <ø> (+6.81%) :arrow_up:
pkg/cli/internal/list/list.go 20.75% <ø> (ø)
... and 11 more

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more