Skip to content

Commit

Permalink
test: proto package may have more than 3 'parts'. Use len - 1 instead
Browse files Browse the repository at this point in the history
  • Loading branch information
Unique-Divine committed Oct 25, 2023
1 parent 6f4b577 commit a69b501
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions wasmbinding/stargate_query_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,13 +84,14 @@ func TestWasmAcceptedStargateQueries(t *testing.T) {
// Get proto message "package" from the response type
typeUrlMinusSuffix := splitResponse[0]
typeUrlPartsFromProtoMsg := strings.Split(typeUrlMinusSuffix, ".")
assert.GreaterOrEqual(t, len(typeUrlPartsFromProtoMsg), 4, typeUrlPartsFromProtoMsg)
protoMessagePackage := typeUrlPartsFromProtoMsg[:3]
lenOfParts := len(typeUrlPartsFromProtoMsg)
assert.GreaterOrEqual(t, lenOfParts, 4, typeUrlPartsFromProtoMsg)
protoMessagePackage := typeUrlPartsFromProtoMsg[:lenOfParts-1]

// Get proto message "package" from the query path
typeUrlPartsFromQueryPath := strings.Split(queryPath, ".")
assert.GreaterOrEqual(t, len(typeUrlPartsFromQueryPath), 4, typeUrlPartsFromQueryPath)
queryPathProtoPackage := typeUrlPartsFromQueryPath[:3]
queryPathProtoPackage := typeUrlPartsFromQueryPath[:lenOfParts-1]

// Verify that the packages match
assert.Equalf(t, queryPathProtoPackage, protoMessagePackage,
Expand Down

0 comments on commit a69b501

Please sign in to comment.