Neptune-Crypto / twenty-first / 15699701258

98%

Build Type

push

github

Committed by jan-ferdinand

Commit Message

refactor!: Remove feature `mock`

The so-called “MockMMR” is a historic artifact that is unused in direct
dependencies. Turn it into a type that is accessible only from unit
tests and remove the feature `mock` to slim down the public API.

Replace `hashbrown::HashSet` with `std::collections::HashSet`, which
amounts to the same thing but allows removing the direct dependency on
`hashbrown`.

Other changes, only relevant for developers of twenty-first:
- rename the “MockMMR” to the more canonical “ArchivalMMR”.
- reflect the internals more accurately by renaming the collection of
  all nodes from “leafs” to “nodes”
- remove helper struct `MyVec<_>`
- replace some helper functions with sane initialize functions
- reduce visibility of some internal helper functions
- rework documentation in some places

Run Details

301 of 303 new or added lines in 3 files covered. (99.34%)

2 existing lines in 2 files now uncovered.

12772 of 13047 relevant lines covered (97.89%)

38049850.82 hits per line