Skip to content

Conversation

@rockbmb
Copy link
Collaborator

@rockbmb rockbmb commented Oct 8, 2025

Closes #436 .

@rockbmb rockbmb added this to the Kusama Migration milestone Oct 8, 2025
@rockbmb rockbmb self-assigned this Oct 8, 2025
@rockbmb rockbmb added bug Something isn't working e2e tests Related to end-to-end tests labels Oct 8, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

No issues found

@xlc
Copy link
Member

xlc commented Oct 8, 2025

now transfer KSM with AH as reserve means there won't be UMP but HRMP instead. so we need to update related tests

@rockbmb
Copy link
Collaborator Author

rockbmb commented Oct 8, 2025

@xlc can you confirm that

  1. tests involving assetHubKusama will go from using HRMP to using UMP/DMP
  2. tests with kusama will go from UMP/DMP to HRMP

@xlc
Copy link
Member

xlc commented Oct 9, 2025

the core change is the reserve chain of KSM is moved from Kusama to KAH. This means two parachains reserved transferring KSM should use HRMP instead of UMP/DMP

@rockbmb
Copy link
Collaborator Author

rockbmb commented Oct 9, 2025

@xlc kusama.shiden is failing with RpcError: 1010: {"invalid":{"exhaustsResources":null}}.

Do the Kusama relay tests also need an update, or is this unrelated?

@xlc
Copy link
Member

xlc commented Oct 9, 2025

I will say ignore failing one and put them into an issue and then check them later one by one.
KSM transfer between relay to para should work but it is not wrong if it stops working if the parachain decides to stop support such case

With the AHM, proxy types on AHM are now expanded to include the relay's
proxy types.
Changes to other proxy tests in this commit are due to the renaming of
the proxy removal action for clarity.
Some are disabled pending a GH issue and subsequent PR.
@rockbmb rockbmb requested a review from xlc October 10, 2025 02:26
@rockbmb
Copy link
Collaborator Author

rockbmb commented Oct 10, 2025

Failing XCM tests disabled for now; all that's left is to disable the two burn tests that uncovered paritytech/polkadot-sdk#9986

@xlc ready for review

@github-actions
Copy link
Contributor

The pull request has two issues. A TODO was found in packages/kusama/src/kusama.proxy.e2e.test.ts that needs to be addressed. The script scripts/check-proxy-coverage.ts is using incorrect proxy types for assetHubKusama.


Review Suggestions

packages/kusama/src/kusama.proxy.e2e.test.ts:82-83

There is a TODO comment indicating that the configuration for the Auction proxy type is incorrect. This should be addressed, or a ticket should be created to track this issue.


scripts/check-proxy-coverage.ts:83

The assetHubKusama network is configured to use AssetHubPolkadotProxyTypes for checking proxy coverage. It should be using AssetHubKusamaProxyTypes as defined in packages/shared/src/helpers/proxyTypes.ts.

@rockbmb rockbmb merged commit 7b565f0 into master Oct 10, 2025
160 of 163 checks passed
@rockbmb rockbmb deleted the remove-kusama-relay-tests branch October 10, 2025 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working e2e tests Related to end-to-end tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Disable obsolete Kusama relay chain E2E test suites after AHM

3 participants