test: Add case for non-empty noop state change

Kava-Labs · Feb 22, 2024 · 29a4996 · 29a4996
1 parent ed9f446
commit 29a4996
Showing 1 changed file with 13 additions and 1 deletion.
diff --git a/x/evm/statedb/statedb_test.go b/x/evm/statedb/statedb_test.go
@@ -229,11 +229,23 @@ func (suite *StateDBTestSuite) TestState() {
 			db.SetState(address, key1, common.Hash{})
 		}, map[common.Hash]common.Hash{}},
 
-		{"noop state change", func(db *statedb.StateDB) {
+		{"noop state change - empty", func(db *statedb.StateDB) {
 			db.SetState(address, key1, value1)
 			db.SetState(address, key1, common.Hash{})
 		}, map[common.Hash]common.Hash{}},
 
+		{"noop state change - non-empty", func(db *statedb.StateDB) {
+			// Start with non-empty committed state
+			db.SetState(address, key1, value1)
+			suite.Require().NoError(db.Commit())
+
+			db.SetState(address, key1, common.Hash{})
+			db.SetState(address, key1, value1)
+		}, map[common.Hash]common.Hash{
+			// Shouldn't be modified - Commit() may still write it again though
+			key1: value1,
+		}},
+
 		{"set state", func(db *statedb.StateDB) {
 			// check empty initial state
 			suite.Require().Equal(common.Hash{}, db.GetState(address, key1))