Merge new KAH E2E tests into master#435
Conversation
rockbmb
added
e2e tests
| }, | ||
| }, | ||
| }, | ||
| assetHubPaseo: { |
There was a problem hiding this comment.
Have to remove this, leftover from local testing.
There was a problem hiding this comment.
is it a continuation of my PR or something unrelated?
There was a problem hiding this comment.
Unrelated; this branch contains the changes required of KAH E2E test suites to run on the new post-AHM runtimes; in the past, I had used to test Paseo things, and forgot to remove some things related to it.
| Collator: 6, | ||
| } | ||
|
|
||
| export const AssetHubPaseoProxyTypes: ProxyTypeMap = { |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
KAH bounties E2E test snapshots will be added in a separate PR.
Will be left to follow-up PR.
There was a problem hiding this comment.
The changes introduce new E2E tests for Kusama Asset Hub and update existing test configurations. The review identified two issues in packages/shared/src/accounts.ts: a potentially incorrect hardcoded value for a reservation and a leftover commented-out debugging call.
|
|
||
| await updateCumulativeFees(client.api, cumulativeFees, testConfig.addressEncoding) | ||
|
|
||
| //await client.pause() |
There was a problem hiding this comment.
A commented-out call to client.pause() should be removed. This is likely a remnant of a debugging session and is not needed.
`proxy.ProxyExecuted` event is enough for analysis to make the test robust.
3 participants
Closes #431 .
With the AHM, Kusama AH gets the E2E test suites for staking, nomination pools, referenda, etc, while on Kusama relay, those should be disabled, as such calls are now filtered.