Add ParseProposalIDFromEvents helper function (#4942)

* add ParseProposalIDFromEvents

* update with damian suggest

testing/endpoint.go

@@ -2,7 +2,6 @@ package ibctesting
 
 import (
 	"fmt"
-	"strconv"
 	"strings"
 
 	"github.com/stretchr/testify/require"
@@ -619,15 +618,8 @@ func (endpoint *Endpoint) ChanUpgradeInit() error {
 		return err
 	}
 
-	events := res.Events
-	for _, event := range events {
-		for _, attribute := range event.Attributes {
-			if attribute.Key == "proposal_id" {
-				proposalID, err = strconv.ParseUint(attribute.Value, 10, 64)
-				require.NoError(endpoint.Chain.TB, err)
-			}
-		}
-	}
+	proposalID, err = ParseProposalIDFromEvents(res.Events)
+	require.NoError(endpoint.Chain.TB, err)
 
 	return VoteAndCheckProposalStatus(endpoint, proposalID)
 }

testing/events.go

@@ -135,6 +135,19 @@ func ParseAckFromEvents(events []abci.Event) ([]byte, error) {
 	return nil, fmt.Errorf("acknowledgement event attribute not found")
 }
 
+// ParseProposalIDFromEvents parses events emitted from MsgSubmitProposal and returns proposalID
+func ParseProposalIDFromEvents(events []abci.Event) (uint64, error) {
+	for _, event := range events {
+		for _, attribute := range event.Attributes {
+			if attribute.Key == "proposal_id" {
+				return strconv.ParseUint(attribute.Value, 10, 64)
+			}
+		}
+	}
+
+	return 0, fmt.Errorf("proposalID event attribute not found")
+}
+
 // AssertEventsLegacy asserts that expected events are present in the actual events.
 // Expected map needs to be a subset of actual events to pass.
 func AssertEventsLegacy(