Tests - Additional sharding tests

[naveensrinivasan]

Tests - Additional sharding tests
Signed-off-by: naveensrinivasan 172697+naveensrinivasan@users.noreply.github.com

[codecov-commenter]

Codecov Report

Merging #1180 (9bbfda9) into main (3f39194) will increase coverage by 0.25%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #1180      +/-   ##
==========================================
+ Coverage   63.25%   63.51%   +0.25%     
==========================================
  Files          82       82              
  Lines        7863     7863              
==========================================
+ Hits         4974     4994      +20     
+ Misses       2258     2242      -16     
+ Partials      631      627       -4     

Flag	Coverage Δ
e2etests	46.76% <ø> (+0.02%)	⬆️
unittests	43.25% <ø> (+0.32%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
pkg/types/rekord/v0.0.1/entry.go	66.97% <0.00%> (-0.63%)	⬇️
pkg/types/alpine/v0.0.1/entry.go	72.90% <0.00%> (ø)
pkg/sharding/ranges.go	83.19% <0.00%> (+18.48%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[bobcallaway]

what about testing a case where wantErr == false?

[naveensrinivasan]

I don't know what I could pass to this func not to return an error.

if err := root.UnmarshalBinary(resp.SignedLogRoot.LogRoot); err != nil {
			return LogRange{}, err
		}

I am mocking the call.

[naveensrinivasan]

@bobcallaway Friendly reminder

[bobcallaway]

@bobcallaway Friendly reminder

The tests aren't passing, can you take a look? thanks!

[naveensrinivasan]

@bobcallaway Friendly reminder

The tests aren't passing, can you take a look? thanks!

Took care of it!