Apply suggestions from code review

Co-authored-by: Yurii Oleksyshyn <[email protected]>

Author: jordanschalm

consensus/hotstuff/signature/block_signer_decoder_test.go

@@ -78,7 +78,7 @@ func (s *blockSignerDecoderSuite) Test_CommitteeException() {
 		require.Empty(s.T(), ids)
 		require.NotErrorIs(s.T(), err, model.ErrViewForUnknownEpoch)
 		require.False(s.T(), signature.IsInvalidSignerIndicesError(err))
-		require.True(s.T(), errors.Is(err, exception))
+		require.ErrorIs(s.T(), err, exception)
 	})
 	s.Run("ByBlock exception", func() {
 		exception := errors.New("unexpected exception")
@@ -90,7 +90,7 @@ func (s *blockSignerDecoderSuite) Test_CommitteeException() {
 		require.Empty(s.T(), ids)
 		require.NotErrorIs(s.T(), err, model.ErrViewForUnknownEpoch)
 		require.False(s.T(), signature.IsInvalidSignerIndicesError(err))
-		require.True(s.T(), errors.Is(err, exception))
+		require.ErrorIs(s.T(), err, exception)
 	})
 }
 

storage/badger/operation/common_test.go

@@ -276,6 +276,7 @@ func TestRetrieveUnencodeable(t *testing.T) {
 	})
 }
 
+// TestExists verifies that `exists` returns correct results in different scenarios.
 func TestExists(t *testing.T) {
 	unittest.RunWithBadgerDB(t, func(db *badger.DB) {
 		t.Run("non-existent key", func(t *testing.T) {