Interact with contracts: upload, instantiate and call commands

[ascjones]

Adds upload, instantiate and call commands which allows interacting with contracts on live chains with a compatible pallet-contracts.

Upload

cargo contract upload --suri //Alice --dry-run

cargo contract upload --suri //Alice

Instantiate

cargo contract instantiate \
       --constructor new \
       --args false \
       --suri //Alice \
       --code-hash 0xbc1b42256696c8a4187ec3ed79fc602789fc11287c4c30926f5e31ed8169574e \
       --dry-run

cargo contract instantiate \
       --constructor new \
       --args false \
       --suri //Alice \
       --code-hash 0xbc1b42256696c8a4187ec3ed79fc602789fc11287c4c30926f5e31ed8169574e

Instantiate with code

Omitting --code-hash will call instantiate_with_code:

Call

cargo contract call \
       --contract 5FQNUFjuSxjB1o67CneHXieB1Hy6xAj2A4h4dUVLCvNkgRmD \
       --message flip \
       --suri //Alice \
       --dry-run

cargo contract call \
       --contract 5FQNUFjuSxjB1o67CneHXieB1Hy6xAj2A4h4dUVLCvNkgRmD \
       --message flip \
       --suri //Alice

cargo contract call \
       --contract 5FQNUFjuSxjB1o67CneHXieB1Hy6xAj2A4h4dUVLCvNkgRmD \
       --message get \
       --suri //Alice \
       --dry-run

Debug message

If the contract is build in debug mode then

Will result in...

Todo

• Display all events resulting from Call
• Optional 0x prefix for hex encoded code hash
• SS58 AccountIds
• Release and use pallet-contracts-primitives and sp-rpc from crates.io
• Documentation
  • CLI usage help
  • extrinsics.md
  • transcoding/scon code docs

Follow ups

These will be promoted to their own issue, or into separate issues for following up once this is merged.

• Signing: safer alternatives for value bearing chains than providing secret key on the command line. Possibly allow using encrypted file with private key, or integrating with external signers somehow.
• Dynamic DispatchError::Module errors, so that error details are displayed correctly across different chains Dynamic DispatchError::Module details paritytech/subxt#443
• Gas estimation: auto dry run first to use estimated gas as a value for the gas limit when running actual call.
• Add flag --wait-for-finalization. so that success/events are not displayed until the block is finalized
• Denominated units for Balances e.g. 100CAN, 1MCAN
• Bitvec encoding/decoding
• Improve error message when failing to encode contract message arg e40dc80
• Set up CI with substrate-contracts-node and unignore integration test: https://github.com/paritytech/cargo-contract/blob/60f65a2863be1266882bdd81861d836baeb06724/src/cmd/extrinsics/integration_tests.rs#L124
• Add support for remove_code extrinsic.

[ascjones]

Renamed because I have an embedded contract for testing purposes and ink! generates code with ::scale imports.

[athei]

I played around with it a bit and it is working as I imagined it. Some remarks:

Probably fix now

• Remove trailing new line from debug message. It looks awkward with the quote on the next line.
• Remove the decoding of the module error as long as this is buggy. It is massively confusing to see a wrong error there. Better to not decode.

Fix in follow up

• Add support for remove_code extrinsic.

[ascjones]

* Remove trailing new line from debug message. It looks awkward with the quote on the next line.

Changed this so that it will print successive lines indented:

Other issues addressed and added follow up to list in PR.

[cmichi]

This will probably trigger a dependabot PR for an upgrade to 2.0.0.

[ascjones]

I'll keep it as 1.0.0 for now since that is the same version as ink_metadata rc8 is using.