- fixes after self review

SergeevDmitry · Apr 14, 2022 · 99cd142 · 99cd142
1 parent a652343
commit 99cd142
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 21 deletions.
diff --git a/state/accountsDB_test.go b/state/accountsDB_test.go
@@ -151,16 +151,6 @@ func TestNewAccountsDB(t *testing.T) {
 		assert.True(t, check.IfNil(adb))
 		assert.Equal(t, state.ErrNilHasher, err)
 	})
-	t.Run("nil hasher should error", func(t *testing.T) {
-		t.Parallel()
-
-		args := createMockAccountsDBArgs()
-		args.Hasher = nil
-
-		adb, err := state.NewAccountsDB(args)
-		assert.True(t, check.IfNil(adb))
-		assert.Equal(t, state.ErrNilHasher, err)
-	})
 	t.Run("nil marshaller should error", func(t *testing.T) {
 		t.Parallel()
 

diff --git a/state/export_test.go b/state/export_test.go
@@ -34,7 +34,7 @@ func (adb *AccountsDB) GetObsoleteHashes() map[string][][]byte {
 	return adb.obsoleteDataTrieHashes
 }
 
-// GetObsoleteHashes -
+// WaitForCompletionIfRunningInImportDB -
 func (adb *AccountsDB) WaitForCompletionIfRunningInImportDB(stats common.SnapshotStatisticsHandler) {
 	adb.waitForCompletionIfRunningInImportDB(stats)
 }

diff --git a/state/peerAccountsDB_test.go b/state/peerAccountsDB_test.go
@@ -40,16 +40,6 @@ func TestNewPeerAccountsDB(t *testing.T) {
 		assert.True(t, check.IfNil(adb))
 		assert.Equal(t, state.ErrNilHasher, err)
 	})
-	t.Run("nil hasher should error", func(t *testing.T) {
-		t.Parallel()
-
-		args := createMockAccountsDBArgs()
-		args.Hasher = nil
-
-		adb, err := state.NewPeerAccountsDB(args)
-		assert.True(t, check.IfNil(adb))
-		assert.Equal(t, state.ErrNilHasher, err)
-	})
 	t.Run("nil marshaller should error", func(t *testing.T) {
 		t.Parallel()