Expand CAP 33 integration tests and fix 500 error bug

* Add Sponsorhip update checks * Expand Claimable Balance test * Add Signer test * Fix type for RevokeSponsorship's OfferID field, which was causing a 500 error on `/operations` due to an unmarshalling error.
stellar · Sep 29, 2020 · 1f6928a · 1f6928a
1 parent e15f71e
commit 1f6928a
Show file tree

Hide file tree

Showing 6 changed files with 553 additions and 113 deletions.
diff --git a/protocols/horizon/effects/main.go b/protocols/horizon/effects/main.go
@@ -161,7 +161,7 @@ const (
 
 	// EffectClaimableBalanceSponsorshipRemoved occurs when the sponsorship of a claimable balance ledger entry
 	// is removed
-	EffectClaimableBalanceSponsorshipRemoved EffectType = 71 // from revoke_sponsorship
+	EffectClaimableBalanceSponsorshipRemoved EffectType = 71 // from claim_claimable_balance
 
 	// EffectSignerSponsorshipCreated occurs when the sponsorship of a signer is created
 	EffectSignerSponsorshipCreated EffectType = 72 // from set_options

diff --git a/protocols/horizon/operations/main.go b/protocols/horizon/operations/main.go
@@ -256,7 +256,7 @@ type RevokeSponsorship struct {
 	ClaimableBalanceID *string `json:"claimable_balance_id,omitempty"`
 	DataAccountID      *string `json:"data_account_id,omitempty"`
 	DataName           *string `json:"data_name,omitempty"`
-	OfferID            *string `json:"offer_id,omitempty"`
+	OfferID            *int64  `json:"offer_id,omitempty"`
 	TrustlineAccountID *string `json:"trustline_account_id,omitempty"`
 	TrustlineAsset     *string `json:"trustline_asset,omitempty"`
 	SignerAccountID    *string `json:"signer_account_id,omitempty"`

diff --git a/services/horizon/internal/db2/history/main.go b/services/horizon/internal/db2/history/main.go
@@ -163,7 +163,7 @@ const (
 
 	// EffectClaimableBalanceSponsorshipRemoved occurs when the sponsorship of a claimable balance ledger entry
 	// is removed
-	EffectClaimableBalanceSponsorshipRemoved EffectType = 71 // from revoke_sponsorship
+	EffectClaimableBalanceSponsorshipRemoved EffectType = 71 // from claim_claimable_balance
 
 	// EffectSignerSponsorshipCreated occurs when the sponsorship of a signer is created
 	EffectSignerSponsorshipCreated EffectType = 72 // from set_options