Closed rafaelcr closed 1 month ago
Attention: Patch coverage is 55.59567% with 369 lines in your changes missing coverage. Please review. |
Files with missing lines | Patch % | Lines |
---|---|---|---|
components/ordhook-core/src/db/ordinals.rs | 50.12% | 195 Missing :warning: | |
components/ordhook-core/src/db/blocks.rs | 51.26% | 58 Missing :warning: | |
components/ordhook-core/src/db/cursor.rs | 76.34% | 44 Missing :warning: | |
components/ordhook-cli/src/cli/mod.rs | 0.00% | 20 Missing :warning: | |
components/ordhook-core/src/service/mod.rs | 0.00% | 18 Missing :warning: | |
components/ordhook-core/src/core/mod.rs | 0.00% | 12 Missing :warning: | |
components/ordhook-core/src/db/mod.rs | 41.17% | 10 Missing :warning: | |
components/ordhook-core/src/utils/mod.rs | 77.27% | 5 Missing :warning: | |
...k-core/src/core/protocol/inscription_sequencing.rs | 86.66% | 2 Missing :warning: | |
components/ordhook-core/src/scan/bitcoin.rs | 0.00% | 2 Missing :warning: | |
... and 3 more |
:loudspeaker: Thoughts on this report? Let us know!
Adds a lot of unit tests. Also, refactors some of the code to make it easier to find and maintain:
Config
classIncreases test coverage from ~25% to ~50% 🚀