sui icon indicating copy to clipboard operation
sui copied to clipboard

[sui-core] Add system transaction tests for dev-inspect and dry-run

Open tzakian opened this issue 1 year ago • 1 comments

Description

Add tests to make sure system transactions can't be run with dev-inpsect and dry-run.

Test Plan

Only tests


If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.

Type of Change (Check all that apply)

  • [ ] protocol change
  • [ ] user-visible impact
  • [ ] breaking change for a client SDKs
  • [ ] breaking change for FNs (FN binary must upgrade)
  • [ ] breaking change for validators or node operators (must upgrade binaries)
  • [ ] breaking change for on-chain data layout
  • [ ] necessitate either a data wipe or data migration

Release notes

tzakian avatar Dec 13 '23 21:12 tzakian

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
mysten-ui ✅ Ready (Inspect) Visit Preview Dec 13, 2023 9:54pm
sui-typescript-docs ✅ Ready (Inspect) Visit Preview Dec 13, 2023 9:54pm
3 Ignored Deployments
Name Status Preview Updated (UTC)
explorer ⬜️ Ignored (Inspect) Dec 13, 2023 9:54pm
multisig-toolkit ⬜️ Ignored (Inspect) Dec 13, 2023 9:54pm
sui-kiosk ⬜️ Ignored (Inspect) Dec 13, 2023 9:54pm

vercel[bot] avatar Dec 13 '23 21:12 vercel[bot]

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

github-actions[bot] avatar Feb 12 '24 01:02 github-actions[bot]

Is this waiting on something in particular?

tnowacki avatar Feb 13 '24 23:02 tnowacki

Is this waiting on something in particular?

Nope! I think I need to rebase this, but otherwise I think this should be good to go

tzakian avatar Feb 27 '24 22:02 tzakian