Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

uncomment AssetsClaimed event assertions on asset-trap integration tests #114

Closed
bernardoaraujor opened this issue Jan 16, 2023 · 1 comment · Fixed by #121
Closed

uncomment AssetsClaimed event assertions on asset-trap integration tests #114

bernardoaraujor opened this issue Jan 16, 2023 · 1 comment · Fixed by #121
Assignees
@ayushmishra2005
Labels
enhancement New feature or request
Milestone
Trappist M1 / XCM v2

Comments

@bernardoaraujor
Copy link
Contributor

bernardoaraujor commented Jan 16, 2023 •
edited
Loading

follow up to #109

pallet-xcm introduced AssetsClaimed event only after v0.9.31

integration tests for TrappistDropAsset have assertions commented out until trappist dependencies have been updated accordingly.

trappist/integration-tests/asset-trap/1_trap_claim_native_asset.yml

Line 142 in 098634e

# ToDo: uncomment when trappist is above 0.9.31

trappist/integration-tests/asset-trap/2_trap_claim_reserved_asset.yml

Line 202 in 098634e

# ToDo: uncomment when trappist is above 0.9.31

trappist/integration-tests/asset-trap/3_trap_claim_native_and_reserved.yml

Line 236 in 098634e

# ToDo: uncomment when trappist is above 0.9.31

@bernardoaraujor bernardoaraujor added the enhancement New feature or request label Jan 16, 2023
@evilrobot-01
Copy link
Contributor

Reliant on #97

@stiiifff stiiifff added this to the Trappist M1 / XCM v2 milestone Jan 25, 2023
@ayushmishra2005 ayushmishra2005 mentioned this issue Jan 26, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ayushmishra2005 ayushmishra2005

Labels
enhancement New feature or request
Projects
None yet
Milestone
Trappist M1 / XCM v2
Development

Successfully merging a pull request may close this issue.

Uncommented AssetsClaimed event assertions on asset-trap integration tests paritytech/trappist
4 participants
@stiiifff @ayushmishra2005 @bernardoaraujor @evilrobot-01