sui icon indicating copy to clipboard operation
sui copied to clipboard

doc: fixed test_sword_transactions arguments error

Open madeinfree opened this issue 1 year ago • 3 comments

I'm not very sure is correct, but I got the error and try to remove the first argument.

error[E04017]: too many arguments
   ┌─ ./sources/my_module.move:97:9
   │
97 │         sword_create(&mut forge, 42, 7, initial_owner, test_scenario::ctx(scenario));
   │         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   │         │           │
   │         │           Found 5 argument(s) here
   │         Invalid call of '(my_first_package=0x0)::my_module::sword_create'. The call expected 4 argument(s) but got 5

pass test.

madeinfree avatar Nov 08 '22 07:11 madeinfree

Are you sure you rebased on the latest main? I see your PR touches a lot of (sensitive) code in the authority's core

asonnino avatar Nov 09 '22 02:11 asonnino

@madeinfree is attempting to deploy a commit to the Mysten Labs Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Nov 09 '22 03:11 vercel[bot]

@asonnino thank you for your reply, that is my fault. I rebased on master branch and force to push branch again.

madeinfree avatar Nov 09 '22 03:11 madeinfree