go/consensus/cometbft: Remove unused DecodeEvent method from context

Author: peternose

go/consensus/cometbft/api/context.go

@@ -370,17 +370,6 @@ func (c *Context) HasEvent(app string, kind events.TypedAttribute) bool {
 	return c.hasEvent(app, kind.EventKind())
 }
 
-// DecodeEvent decodes the given raw event as a specific typed event.
-func (c *Context) DecodeEvent(index int, ev events.TypedAttribute) error {
-	raw := c.events[index]
-	for _, pair := range raw.Attributes {
-		if events.IsAttributeKind(pair.GetKey(), ev) {
-			return events.DecodeValue(pair.GetValue(), ev)
-		}
-	}
-	return fmt.Errorf("incompatible event")
-}
-
 // ProvableEvents returns the emitted provable events.
 func (c *Context) ProvableEvents() []events.Provable {
 	return c.eventsProvable

go/consensus/cometbft/apps/staking/state/state_test.go

@@ -974,13 +974,6 @@ func TestTransfer(t *testing.T) {
 	require.EqualValues(*quantity.NewFromUint64(250), a2.General.Balance, "amount in destination account should be correct")
 	require.Len(ctx.GetEvents(), 1, "one event should be emitted")
 
-	var ev staking.TransferEvent
-	err = ctx.DecodeEvent(0, &ev)
-	require.NoError(err, "DecodeEvent")
-	require.EqualValues(addr1, ev.From, "event should have the correct source address")
-	require.EqualValues(addr2, ev.To, "event should have the correct destination address")
-	require.EqualValues(*quantity.NewFromUint64(50), ev.Amount, "event should have the correct amount")
-
 	// Test min transact balance checks.
 	err = s.Transfer(ctx, addr1, addr2, quantity.NewFromUint64(101))
 	require.ErrorIs(err, staking.ErrBalanceTooLow, "Transfer leaving below min transact balance in source")