Add test coverage for ElementAccumulator.DecodeFrom

SiaFoundation / core

Core packages for the Sia project

MIT License

51 stars 7 forks source link

Add test coverage for ElementAccumulator.DecodeFrom #154

Closed new0nebit closed 2 months ago

new0nebit commented 3 months ago

This PR expands test coverage in merkle to 95.1% and overall consensus coverage to 76.2% Now that we have the coverage for DecodeFrom and EncodeTo I'm planning to add a roundtrip later, assuming you find such tests to be valuable.

lukechampine commented 3 months ago

Let's do a roundtrip test instead

new0nebit commented 3 months ago

Great, I'll give it a try 🧐

new0nebit commented 3 months ago

Done! (ง •̀‿•́ )ง

lukechampine commented 2 months ago

thanks!