fix: allow mfx minting only #422
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #422 +/- ##
=======================================
Coverage 81.67% 81.67%
=======================================
Files 151 151
Lines 16837 16841 +4
=======================================
+ Hits 13751 13755 +4
Misses 3086 3086 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
| @@ -48,7 +48,11 @@ jobs: | |||
| steps: | |||
| - run: | |||
| name: install cbor-diag | |||
| command: cargo install --force cbor-diag-cli | |||
| command: | | |||
There was a problem hiding this comment.
cbor-diag-cli no longer builds using Rust < 1.81 because of some nested dependency issue.
I created a binary release and used it here.
| @@ -87,12 +87,12 @@ local tendermint(i, user) = { | |||
| ports: [ "" + (26600 + i) + ":26657" ], | |||
| }; | |||
|
|
|||
| function(nb_nodes=4, user=1000, id_with_balances="", allow_addrs=false, enable_migrations=false, abci_migrations=false) { | |||
| function(nb_nodes=4, user=1000, id_with_balances="", token="mqbfbahksdwaqeenayy2gxke32hgb7aq4ao4wt745lsfs6wiaaaaqnz", allow_addrs=false, enable_migrations=false, abci_migrations=false) { | |||
There was a problem hiding this comment.
Allow one to customize the token symbol for initial balances in tests.
|
|
||
| // Production network MFX address | ||
| pub static MFX: Lazy<Address> = Lazy::new(|| { | ||
| Address::from_str("mqbh742x4s356ddaryrxaowt4wxtlocekzpufodvowrirfrqaaaaa3l").unwrap() |
There was a problem hiding this comment.
The production network MFX address. Won't work if the MFX address is something else.
| let TokenMintArgs { | ||
| symbol, | ||
| distribution, | ||
| memo, | ||
| } = args; | ||
|
|
||
| if symbol != *MFX |
There was a problem hiding this comment.
Disable minting for everything except MFX.
| @@ -3,7 +3,7 @@ | |||
| GIT_ROOT="$BATS_TEST_DIRNAME/../../../" | |||
| MIGRATION_ROOT="$GIT_ROOT/staging/ledger_migrations.json" | |||
| MAKEFILE="Makefile.ledger" | |||
| MFX_ADDRESS=mqbfbahksdwaqeenayy2gxke32hgb7aq4ao4wt745lsfs6wiaaaaqnz | |||
| MFX_ADDRESS=mqbh742x4s356ddaryrxaowt4wxtlocekzpufodvowrirfrqaaaaa3l | |||
There was a problem hiding this comment.
Use the production MFX address in the resiliency test
Disable token minting for all BUT MFX.
This only works if the MFX address is
mqbh742x4s356ddaryrxaowt4wxtlocekzpufodvowrirfrqaaaaa3l, i.e., the production network MFX address. The resiliency test reflects this.