protocols/horizon: Make the ClawbackClaimableBalanceOp ID a required …

…string (#3540)
stellar · Apr 15, 2021 · fcd228f · fcd228f
1 parent 23dc177
commit fcd228f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/protocols/horizon/operations/main.go b/protocols/horizon/operations/main.go
@@ -280,7 +280,7 @@ type Clawback struct {
 // ClawbackClaimableBalance.
 type ClawbackClaimableBalance struct {
 	Base
-	ClaimableBalanceID *string `json:"balance_id,omitempty"`
+	BalanceID string `json:"balance_id"`
 }
 
 // SetTrustLineFlags is the json resource representing a single operation whose type is

diff --git a/services/horizon/internal/integration/protocol16_test.go b/services/horizon/internal/integration/protocol16_test.go
@@ -381,7 +381,7 @@ func TestHappyClawbackClaimableBalance(t *testing.T) {
 	tt.NoError(err)
 	if tt.Len(opDetailsResponse.Embedded.Records, 1) {
 		clawbackOp := opDetailsResponse.Embedded.Records[0].(operations.ClawbackClaimableBalance)
-		tt.Equal(cbID, *clawbackOp.ClaimableBalanceID)
+		tt.Equal(cbID, clawbackOp.BalanceID)
 	}
 
 	// Check the operation effects