Skip to content

E2e update#2481

Open
shamil-gadelshin wants to merge 9 commits intodevnet-readyfrom
e2e-update
Open

E2e update#2481
shamil-gadelshin wants to merge 9 commits intodevnet-readyfrom
e2e-update

Conversation

@shamil-gadelshin
Copy link
Collaborator

This PR adds staking package to e2e workflow (using fast-runtime binary). It covers happy paths for the extrinsics:

  • add_stake
  • add_stake_limit
  • remove_stake
  • remove_stake_limit
  • remove_stake_full_limit
  • unstake_all
  • unstake_all_alpha
  • move_stake
  • transfer_stake
  • swap_stake
  • swap_stake_limit
  • root_claim (all RootClaimType)
  • set_root_claim_type
  • sudo_set_num_root_claims
  • sudo_set_root_claim_threshold

@shamil-gadelshin shamil-gadelshin added the skip-cargo-audit This PR fails cargo audit but needs to be merged anyway label Mar 4, 2026
l0r1s
l0r1s previously approved these changes Mar 4, 2026
Copy link
Collaborator

@l0r1s l0r1s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would recommend using expect instead of assert because assert is too generic while expect will allow vitest (the test runner) to capture the values and give more explicit failure messages (https://vitest.dev/api/expect.html)

assert.ok(a > b, "Some message")

AssertionError [ERR_ASSERTION]: Balance should increase after stake move

expect(a, "Some message").toBeGreaterThan(b)

Balance should increase after stake move

expect(received).toBeGreaterThan(expected)

Expected: 500n
Received: 300n

@shamil-gadelshin shamil-gadelshin requested a review from l0r1s March 6, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-cargo-audit This PR fails cargo audit but needs to be merged anyway

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants