Dao integration test and upgrade mpm #95

baichuan3 · 2022-08-24T14:17:21Z

Pull request type

Please check the type of change your PR introduces:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

支持了checkpoint集成测试
支持从集成测试中获取raw proofs和access path
除二阶段升级外，DAOSpace整体的集成测试已跑通，包括DAO的创建、Proposal的创建、投票、Action的执行等

Other information

baichuan3 · 2022-08-25T02:42:47Z

mpm新增的一些方法，导致ci过不去，这种如何处理？是先把新加的test case注释掉，等mpm更新后再放出来？还是更新ci中mpm到最新版本？ @jolestar @0xpause

jolestar · 2022-08-25T05:07:06Z

mpm新增的一些方法，导致ci过不去，这种如何处理？是先把新加的test case注释掉，等mpm更新后再放出来？还是更新ci中mpm到最新版本？ @jolestar @0xpause

等发个新 beta 版本，那个 panic 的问题是需要修复吗？ @0xpause

integration-tests/daospace/daospace_proposal.move

pause125 · 2022-08-25T08:09:04Z

mpm新增的一些方法，导致ci过不去，这种如何处理？是先把新加的test case注释掉，等mpm更新后再放出来？还是更新ci中mpm到最新版本？ @jolestar @0xpause

等发个新 beta 版本，那个 panic 的问题是需要修复吗？ @0xpause

@jolestar mpm 应该没有问题。可能是以前的测试代码不完善，以这个 PR 为准。

Move.toml

sources/daospace/SnapshotUtil.move

pause125 · 2022-08-25T13:24:18Z

sources/daospace/SnapshotUtil.move

+        assert!(value == 0, 1);
+        Vector::reverse(&mut buffer);
+        buffer
+    }


to_hex_string_without_prefix 的实现不用 while 和 Vector::reverse 会不会省点 gas？

我试了下换成实现：

Vector::push_back(&mut buffer, *Vector::borrow(&HEX_SYMBOLS, ((value & 0xf0) >> 4 as u64))); Vector::push_back(&mut buffer, *Vector::borrow(&HEX_SYMBOLS, (value & 0x0f as u64)));

调用一次 address_to_hex_string 函数， gas 变化： 556779 -> 478587 。

如果是u8的话，是可以把while优化掉的。此处暂时保留了u64和u128等类型扩展的余地。

pause125 · 2022-08-25T14:09:16Z

integration-tests/daospace/daospace_proposal.move

@@ -198,7 +218,7 @@ script{
        IdentifierNFT::accept<DAOMember<X>, DAOMemberBody<X>>(&sender);

        let user_add = Signer::address_of(&sender);
-        DAOHelper::member_join<X>(user_add, 1000u128);
+        DAOHelper::member_join<X>(user_add, 10000u128);
    }
 }


这个 case 看起来是任何人可以自己 join member，容易给开发者误导，集成测试本身也是给开发者的一个很好的教程。也许 alice 先 IdentifierNFT::accept ，然后 creator 来 join member 更好一点。

dao_proposal.move的测试更偏dao底层代码的测试，类似member和grant等插件的测试更偏具体的场景，开发者参考那个会更好一些。

pause125 · 2022-08-25T14:34:29Z

integration-tests/daospace/daospace_proposal.move

+////# call-api state.list_resource ["$.faucet[2].txn.raw_txn.decoded_payload.ScriptFunction.args[0]"]
+//# call-api state.list_resource ["0xb5d577dc9ce59725e29886632e69ecdf",{"decode":true}]
+
+//# call-api state.get_resource ["0xb5d577dc9ce59725e29886632e69ecdf","0x00000000000000000000000000000001::IdentifierNFT::IdentifierNFT<0x00000000000000000000000000000001::DAOSpace::DAOMember<0x662ba5a1a1da0f1c70a9762c7eeb7aaf::DAOHelper::X>,0x00000000000000000000000000000001::DAOSpace::DAOMemberBody<0x662ba5a1a1da0f1c70a9762c7eeb7aaf::DAOHelper::X>>",{"decode":true}]


这两处 call-api 似乎没有用？

这两个api为了排查问题，正常集成测试不会用到，先保留着

jolestar · 2022-08-26T14:03:59Z

可以把 mpm 的版本升级到 v1.12.2-beta 试试，修改 dev_setup.sh 就可以。

pause125 · 2022-08-26T14:45:56Z

可以把 mpm 的版本升级到 v1.12.2-beta 试试，修改 dev_setup.sh 就可以。

这是不是要把其他测试的 exp 文件一并改了才行？

baichuan3 · 2022-08-27T05:14:26Z

用新版本的mpm更新了所有integration test baseline。

mpm integration-test --ub

test result: FAILED. 47 passed; 93 failed; 0 filtered out

有几个testcase在fork模式下会失败，异常会中断后续的集成测试，通过注释掉出问题case的方式，定位到所有出问题的tesecase为：

#1
thread 'integration-test::shared_ed25519_public_key/shared_key.move' panicked at 'TransactionStatus at here must been KeptVMStatus: INVALID_AUTH_KEY', vm/starcoin-transactional-test-harness/src/fork_chain.rs:149:14

#2
thread 'integration-test::check/gas_check.move' panicked at 'TransactionStatus at here must been KeptVMStatus: MAX_GAS_UNITS_BELOW_MIN_TRANSACTION_GAS_UNITS', vm/starcoin-transactional-test-harness/src/fork_chain.rs:149:14

其中mpm版本

mpm --version
move-package-manager 1.12.1

需要 @0xpause @jolestar 修复下。

jolestar · 2022-08-27T05:23:16Z

应该是新的 faucet 命令回 hack 账号的 auth key 导致的

jolestar · 2022-08-27T05:24:34Z

2.应该是 fork 模式下，使用的是链上的 gas config 配置，和测试环境有差异导致的

pause125 · 2022-08-28T08:55:51Z

这两个问题都是因为有 Discard 的交易，触发了 bug，从而崩溃。

这个 Bug 是尝试将 Discard txn 添加到 txn_accumulator 中，而 txn_accumulator 只接受 KeptVMStatus 的 txn，因此在 check 时遇到了 Discard 的 txn，从而发生 panic 。

问题已修复， PR

jolestar · 2022-08-28T11:12:33Z

prepare release v1.12.3 starcoinorg/starcoin#3692

jolestar · 2022-08-29T06:15:49Z

我先合并了，其他的 move 相关的修复可以单独 PR。

This reverts commit 82032cc.

* Revert "fix stake plugin for compatibility (#218)" This reverts commit e618173. * Revert "add some test_functions (#216)" This reverts commit 87859a9. * Revert "Add WithdrawPlugin (#214)" This reverts commit a66f0f3. * Revert "fix StakeToSBTPlugin install event (#215)" This reverts commit 77a968a. * Revert "add install plugin proposal for TreasuryPlugin (#213)" This reverts commit a98de9e. * Revert "Fix v12 (#210)" This reverts commit bfd8fa6. * Revert "release v12 (#205)" This reverts commit 94be70e. * Revert "Fix some entry function and do some clean (#203)" This reverts commit 884f648. * Revert "prepare v12 release (#200)" This reverts commit 3bae7fd. * Revert "Refactor root cap (#198)" This reverts commit b30fdc9. * Revert "feat: add entry for plugin marketplace (#199)" This reverts commit 459ce12. * Revert "Upgrade MerkleNFTDistributor::register to v2 (#196)" This reverts commit 40178bb. * Revert "[daospace] Refactor daospace functions name (#195)" This reverts commit 0dd5c68. * Revert "Accept Offer and Accept NFT (#192)" This reverts commit 19b7d1d. * Revert "Fix sign flag for SignedInteger64 zero (#191)" This reverts commit 722a298. * Revert "fix cmp (#190)" This reverts commit fce8707. * Revert "Fix (#189)" This reverts commit 64d754d. * Revert "feat: remove sender of update_plugin (#187)" This reverts commit cb21b0a. * Revert "Custom proposal's quorum_votes for plugin (#184)" This reverts commit 45eb5ca. * Revert "[Feature] Proposal add title and introduction (#183)" This reverts commit 4ebe51b. * Revert "[DAOSpace] Plugin marketplace remove NFT (#186)" This reverts commit f186ec8. * Revert "Migrate Genesis initialize to new DAOSpace (#181)" This reverts commit 57e859e. * Revert "Update GasOracle module" This reverts commit f0270c3. * Revert "Fix prologue gas token error" This reverts commit 3c58061. * Revert "[Feature] Simple map (#182)" This reverts commit daa971f. * Revert "add some function in Compare (#180)" This reverts commit e5a3108. * Revert "fix to_bytes need store (#179)" This reverts commit 31792d4. * Revert "[Feature] Member Grant Offer (#174)" This reverts commit 0234cfc. * Revert "migrate TreasuryWithdrawDaoProposal to DAOSpace TreasuryPlugin (#175)" This reverts commit f44fd34. * Revert "refactor plugin initialize (#176)" This reverts commit e4c7bf8. * Revert "update old links in README (#173)" This reverts commit cb26bbd. * Revert "[Bugfix]Fix ERR_EXPECT_NOT_MEMBER (#172)" This reverts commit dca2274. * Revert "Fix acount.txn_epilogue_v2 incompatible (#168)" This reverts commit ad81d03. * Revert "[Feature] Add dao create test (#167)" This reverts commit dfbf3d7. * Revert "Remove proposal after executed or rejected (#145)" This reverts commit 1e17512. * Revert "EasyGas support (#92)" This reverts commit e142883. * Revert "Reformat code and decrease_member_sbt when unstake (#166)" This reverts commit 25148a4. * Revert "upgrade mpm to 1.12.5 and fix integration tests (#163)" This reverts commit 0fb8810. * Revert "[Feature] DAOSpace add join_member_with_root_cap (#164)" This reverts commit 1518644. * Revert "[Feature] Add ASCII module and TODO: ascii DAO name (#161)" This reverts commit d3eb047. * Revert "[Feature]DAOAccount Todo : add exists_upgrade_plan_cap (#160)" This reverts commit 1fb3b8a. * Revert "fix eventutil (#146)" This reverts commit d2c4f5c. * Revert "[Feature] Make StarcoinDAO inherit Dao's config (#144)" This reverts commit 8675779. * Revert "migrate script to entry function (#142)" This reverts commit 971a1a2. * Revert "[Feature] Multi Offer (#135)" This reverts commit 4ec8c2f. * Revert "add upgrade plan event (#128)" This reverts commit e82c1d3. * Revert "merge upgrade entry v12 and v12_1 (#126)" This reverts commit 47b38ad. * Revert "[Feature] NFTGallery Add Errors Assert (#138)" This reverts commit 61392b0. * Revert "Add event util (#134)" This reverts commit 545f191. * Revert "#131 Add only_new_module (#132) " This reverts commit f1363a3. * Revert "[Feature] DAO incompatible module upgrade test (#133)" This reverts commit 022ca1a. * Revert "update_module_upgrade_strategy support set min_time_limit (#127)" This reverts commit c8c9ba7. * Revert "Daospace support plugin marketplace (#116)" This reverts commit 6cccb73. * Revert "check StakeList existence (#124)" This reverts commit dee9a0b. * Revert "[ Feature ] do deposit when proposal state is DEFEATED (#122)" This reverts commit 3274821. * Revert "[daospace] Remove proposal info and add get function for proposal (#121)" This reverts commit 7b44ee7. * Revert "[integration-test] StarcoinDAO integration-test (#112)" This reverts commit 45f017a. * Revert "add assertion in stake to `StakeToSBTPlugin::stake` while the lock ti… (#117)" This reverts commit 08928d4. * Revert "Add Script function entry (#111)" This reverts commit 4c32fda. * Revert "Fix DAO Pulgin Event (#118)" This reverts commit 3cda388. * Revert "Plugin Event (#97)" This reverts commit 31b9a9c. * Revert "fix SalaryGovPlugin and migrate to test (#115)" This reverts commit 3e95aef. * Revert "[account] Retry when creating delegate account (#110)" This reverts commit 7a38b4b. * Revert "Add upgrade plugin And add StarcoinDAO (#94)" This reverts commit 85418d7. * Revert "able to update DAOExt (#108)" This reverts commit fc7ac4a. * Revert "[Feature] Public Native name_of function (#107)" This reverts commit 7263b0a. * Revert "MintProposalPlugin (#105)" This reverts commit ff2d802. * Revert "add daospace module upgrade test (#98)" This reverts commit 70429e2. * Revert "Add DAO nft image and member image (#96)" This reverts commit 2c89bbc. * Revert "Dao integration test and upgrade mpm (#95)" This reverts commit 82032cc. * Revert "add integration-test for refund_grant (#93)" This reverts commit 52fb86d. * Revert "signature.move add secp256k1_verify function (#86)" This reverts commit 9ea61ce. * Revert "check DAOAccountCap (#89)" This reverts commit 0404a53. * Revert "Add DAOSpace description (#85)" This reverts commit a981f67. * Revert "Remove decrease SBT when unstake item (#81)" This reverts commit d401c37. * Revert "[signature] Fix EVMAddress padding and crop bug (#84)" This reverts commit 41ad861. * Revert "fix dao snapshot deserialize when some option field is empty (#82)" This reverts commit b272a6f. * Revert "Complete DAOSpace (#80)" This reverts commit 98e96e6. * Revert "[dao] Implement DAOSpace (#32)" This reverts commit 0d37c89. * Revert "Update README.md (#68)" This reverts commit d3bb99f. * Revert "Update .gitattributes (#63)" This reverts commit 41ed34b. * Revert "support installing mpm from source (#56)" This reverts commit 3f46c6c. * Revert "[WIP] add Block state root (#39) (#41)" This reverts commit 67027f9. * Revert "Add multiple types of bcs skip and test (#52)" This reverts commit a5dc012. * Revert "upgrade rust toolchain 1.62.0 (#51)" This reverts commit 81c1e51. * Revert "implement Ring module (#44)" This reverts commit c9582a6. * Revert "support starcoin snapshot proof and verify (#45)" This reverts commit 63b05ec. * Revert "add bcs function , byte to basic type (#43)" This reverts commit e39dfeb. * Revert "account events (#42)" This reverts commit bfbe32c. * Revert "[scripts] fix dev_setup package manager detection and mpm version on ubuntu18.04 (#37)" This reverts commit 796fd3f. * Revert "Upgrade mpm to v1.11.11 (#36)" This reverts commit 92fb22a. * Revert "[Feature] Identifier nft add borrow function (#34)" This reverts commit 3e24ac4. * Revert "[account] Add create_delegate_account function (#31)" This reverts commit a225d41. * Revert "STRATEGY_FREEZE use invalid_argument (#29)" This reverts commit 51828cd. * Revert "remove zero Balance or empty NFTGallery (#28)" This reverts commit 0303214. * Revert "update rust 1.61 (#27)" This reverts commit 6da0321. * Revert "[ci] Upgrade mpm version to v1.11.9 (#26)" This reverts commit c2475ea. * Revert "Improve spec of several modules (#25)" This reverts commit 8f8e5a0. * Revert "Update README.md" This reverts commit 8fc83b6. * Revert "Upgrade mpm to v1.11.5-alpha and rename `spectest` to `integrationt-test` (#24)" This reverts commit a35c3f7. * Revert "Update stdlib spec, remove redundant spec funcs and improve coding style (#23)" This reverts commit 9c9922d. * Revert "[doc] Add a readme template for generate docs (#22)" This reverts commit 2c53e4f. * Revert "[test] Add a signature test for verify message signature from starcoin cli. (#17)" This reverts commit f88c277. * Revert "Add a script to generate docs and commit docs to git (#21)" This reverts commit 60c8002. * Revert "#13 [Feature Request] Account::deposit(address, token) auto create a… (#14)" This reverts commit 88061e9. * Revert "[ci] fix spectest (#15)" This reverts commit 0b866e1. * Revert "config Move highlighting (#12)" This reverts commit 01c8419. * Revert "simplify dev setup scripts (#2)" This reverts commit d2a2fc9. * Revert "[ci & doc] Add more document and setup ci github action workflow. (#1)" This reverts commit 9fd1e52. * add Cargo.lock

* Revert "fix stake plugin for compatibility (#218)" This reverts commit e618173. * Revert "add some test_functions (#216)" This reverts commit 87859a9. * Revert "Add WithdrawPlugin (#214)" This reverts commit a66f0f3. * Revert "fix StakeToSBTPlugin install event (#215)" This reverts commit 77a968a. * Revert "add install plugin proposal for TreasuryPlugin (#213)" This reverts commit a98de9e. * Revert "Fix v12 (#210)" This reverts commit bfd8fa6. * Revert "release v12 (#205)" This reverts commit 94be70e. * Revert "Fix some entry function and do some clean (#203)" This reverts commit 884f648. * Revert "prepare v12 release (#200)" This reverts commit 3bae7fd. * Revert "Refactor root cap (#198)" This reverts commit b30fdc9. * Revert "feat: add entry for plugin marketplace (#199)" This reverts commit 459ce12. * Revert "Upgrade MerkleNFTDistributor::register to v2 (#196)" This reverts commit 40178bb. * Revert "[daospace] Refactor daospace functions name (#195)" This reverts commit 0dd5c68. * Revert "Accept Offer and Accept NFT (#192)" This reverts commit 19b7d1d. * Revert "Fix sign flag for SignedInteger64 zero (#191)" This reverts commit 722a298. * Revert "fix cmp (#190)" This reverts commit fce8707. * Revert "Fix (#189)" This reverts commit 64d754d. * Revert "feat: remove sender of update_plugin (#187)" This reverts commit cb21b0a. * Revert "Custom proposal's quorum_votes for plugin (#184)" This reverts commit 45eb5ca. * Revert "[Feature] Proposal add title and introduction (#183)" This reverts commit 4ebe51b. * Revert "[DAOSpace] Plugin marketplace remove NFT (#186)" This reverts commit f186ec8. * Revert "Migrate Genesis initialize to new DAOSpace (#181)" This reverts commit 57e859e. * Revert "Update GasOracle module" This reverts commit f0270c3. * Revert "Fix prologue gas token error" This reverts commit 3c58061. * Revert "[Feature] Simple map (#182)" This reverts commit daa971f. * Revert "add some function in Compare (#180)" This reverts commit e5a3108. * Revert "fix to_bytes need store (#179)" This reverts commit 31792d4. * Revert "[Feature] Member Grant Offer (#174)" This reverts commit 0234cfc. * Revert "migrate TreasuryWithdrawDaoProposal to DAOSpace TreasuryPlugin (#175)" This reverts commit f44fd34. * Revert "refactor plugin initialize (#176)" This reverts commit e4c7bf8. * Revert "update old links in README (#173)" This reverts commit cb26bbd. * Revert "[Bugfix]Fix ERR_EXPECT_NOT_MEMBER (#172)" This reverts commit dca2274. * Revert "Fix acount.txn_epilogue_v2 incompatible (#168)" This reverts commit ad81d03. * Revert "[Feature] Add dao create test (#167)" This reverts commit dfbf3d7. * Revert "Remove proposal after executed or rejected (#145)" This reverts commit 1e17512. * Revert "EasyGas support (#92)" This reverts commit e142883. * Revert "Reformat code and decrease_member_sbt when unstake (#166)" This reverts commit 25148a4. * Revert "upgrade mpm to 1.12.5 and fix integration tests (#163)" This reverts commit 0fb8810. * Revert "[Feature] DAOSpace add join_member_with_root_cap (#164)" This reverts commit 1518644. * Revert "[Feature] Add ASCII module and TODO: ascii DAO name (#161)" This reverts commit d3eb047. * Revert "[Feature]DAOAccount Todo : add exists_upgrade_plan_cap (#160)" This reverts commit 1fb3b8a. * Revert "fix eventutil (#146)" This reverts commit d2c4f5c. * Revert "[Feature] Make StarcoinDAO inherit Dao's config (#144)" This reverts commit 8675779. * Revert "migrate script to entry function (#142)" This reverts commit 971a1a2. * Revert "[Feature] Multi Offer (#135)" This reverts commit 4ec8c2f. * Revert "add upgrade plan event (#128)" This reverts commit e82c1d3. * Revert "merge upgrade entry v12 and v12_1 (#126)" This reverts commit 47b38ad. * Revert "[Feature] NFTGallery Add Errors Assert (#138)" This reverts commit 61392b0. * Revert "Add event util (#134)" This reverts commit 545f191. * Revert "#131 Add only_new_module (#132) " This reverts commit f1363a3. * Revert "[Feature] DAO incompatible module upgrade test (#133)" This reverts commit 022ca1a. * Revert "update_module_upgrade_strategy support set min_time_limit (#127)" This reverts commit c8c9ba7. * Revert "Daospace support plugin marketplace (#116)" This reverts commit 6cccb73. * Revert "check StakeList existence (#124)" This reverts commit dee9a0b. * Revert "[ Feature ] do deposit when proposal state is DEFEATED (#122)" This reverts commit 3274821. * Revert "[daospace] Remove proposal info and add get function for proposal (#121)" This reverts commit 7b44ee7. * Revert "[integration-test] StarcoinDAO integration-test (#112)" This reverts commit 45f017a. * Revert "add assertion in stake to `StakeToSBTPlugin::stake` while the lock ti… (#117)" This reverts commit 08928d4. * Revert "Add Script function entry (#111)" This reverts commit 4c32fda. * Revert "Fix DAO Pulgin Event (#118)" This reverts commit 3cda388. * Revert "Plugin Event (#97)" This reverts commit 31b9a9c. * Revert "fix SalaryGovPlugin and migrate to test (#115)" This reverts commit 3e95aef. * Revert "[account] Retry when creating delegate account (#110)" This reverts commit 7a38b4b. * Revert "Add upgrade plugin And add StarcoinDAO (#94)" This reverts commit 85418d7. * Revert "able to update DAOExt (#108)" This reverts commit fc7ac4a. * Revert "[Feature] Public Native name_of function (#107)" This reverts commit 7263b0a. * Revert "MintProposalPlugin (#105)" This reverts commit ff2d802. * Revert "add daospace module upgrade test (#98)" This reverts commit 70429e2. * Revert "Add DAO nft image and member image (#96)" This reverts commit 2c89bbc. * Revert "Dao integration test and upgrade mpm (#95)" This reverts commit 82032cc. * Revert "add integration-test for refund_grant (#93)" This reverts commit 52fb86d. * Revert "signature.move add secp256k1_verify function (#86)" This reverts commit 9ea61ce. * Revert "check DAOAccountCap (#89)" This reverts commit 0404a53. * Revert "Add DAOSpace description (#85)" This reverts commit a981f67. * Revert "Remove decrease SBT when unstake item (#81)" This reverts commit d401c37. * Revert "[signature] Fix EVMAddress padding and crop bug (#84)" This reverts commit 41ad861. * Revert "fix dao snapshot deserialize when some option field is empty (#82)" This reverts commit b272a6f. * Revert "Complete DAOSpace (#80)" This reverts commit 98e96e6. * Revert "[dao] Implement DAOSpace (#32)" This reverts commit 0d37c89. * Revert "Update README.md (#68)" This reverts commit d3bb99f. * Revert "Update .gitattributes (#63)" This reverts commit 41ed34b. * Revert "support installing mpm from source (#56)" This reverts commit 3f46c6c. * Revert "[WIP] add Block state root (#39) (#41)" This reverts commit 67027f9. * Revert "Add multiple types of bcs skip and test (#52)" This reverts commit a5dc012. * Revert "upgrade rust toolchain 1.62.0 (#51)" This reverts commit 81c1e51. * Revert "implement Ring module (#44)" This reverts commit c9582a6. * Revert "support starcoin snapshot proof and verify (#45)" This reverts commit 63b05ec. * Revert "add bcs function , byte to basic type (#43)" This reverts commit e39dfeb.

fix Revert daospace (#226) commit msg * Revert "fix stake plugin for compatibility (#218)" This reverts commit e618173. * Revert "add some test_functions (#216)" This reverts commit 87859a9. * Revert "Add WithdrawPlugin (#214)" This reverts commit a66f0f3. * Revert "fix StakeToSBTPlugin install event (#215)" This reverts commit 77a968a. * Revert "add install plugin proposal for TreasuryPlugin (#213)" This reverts commit a98de9e. * Revert "Fix v12 (#210)" This reverts commit bfd8fa6. * Revert "release v12 (#205)" This reverts commit 94be70e. * Revert "Fix some entry function and do some clean (#203)" This reverts commit 884f648. * Revert "prepare v12 release (#200)" This reverts commit 3bae7fd. * Revert "Refactor root cap (#198)" This reverts commit b30fdc9. * Revert "feat: add entry for plugin marketplace (#199)" This reverts commit 459ce12. * Revert "Upgrade MerkleNFTDistributor::register to v2 (#196)" This reverts commit 40178bb. * Revert "[daospace] Refactor daospace functions name (#195)" This reverts commit 0dd5c68. * Revert "Accept Offer and Accept NFT (#192)" This reverts commit 19b7d1d. * Revert "Fix sign flag for SignedInteger64 zero (#191)" This reverts commit 722a298. * Revert "fix cmp (#190)" This reverts commit fce8707. * Revert "Fix (#189)" This reverts commit 64d754d. * Revert "feat: remove sender of update_plugin (#187)" This reverts commit cb21b0a. * Revert "Custom proposal's quorum_votes for plugin (#184)" This reverts commit 45eb5ca. * Revert "[Feature] Proposal add title and introduction (#183)" This reverts commit 4ebe51b. * Revert "[DAOSpace] Plugin marketplace remove NFT (#186)" This reverts commit f186ec8. * Revert "Migrate Genesis initialize to new DAOSpace (#181)" This reverts commit 57e859e. * Revert "Update GasOracle module" This reverts commit f0270c3. * Revert "Fix prologue gas token error" This reverts commit 3c58061. * Revert "[Feature] Simple map (#182)" This reverts commit daa971f. * Revert "add some function in Compare (#180)" This reverts commit e5a3108. * Revert "fix to_bytes need store (#179)" This reverts commit 31792d4. * Revert "[Feature] Member Grant Offer (#174)" This reverts commit 0234cfc. * Revert "migrate TreasuryWithdrawDaoProposal to DAOSpace TreasuryPlugin (#175)" This reverts commit f44fd34. * Revert "refactor plugin initialize (#176)" This reverts commit e4c7bf8. * Revert "update old links in README (#173)" This reverts commit cb26bbd. * Revert "[Bugfix]Fix ERR_EXPECT_NOT_MEMBER (#172)" This reverts commit dca2274. * Revert "Fix acount.txn_epilogue_v2 incompatible (#168)" This reverts commit ad81d03. * Revert "[Feature] Add dao create test (#167)" This reverts commit dfbf3d7. * Revert "Remove proposal after executed or rejected (#145)" This reverts commit 1e17512. * Revert "EasyGas support (#92)" This reverts commit e142883. * Revert "Reformat code and decrease_member_sbt when unstake (#166)" This reverts commit 25148a4. * Revert "upgrade mpm to 1.12.5 and fix integration tests (#163)" This reverts commit 0fb8810. * Revert "[Feature] DAOSpace add join_member_with_root_cap (#164)" This reverts commit 1518644. * Revert "[Feature] Add ASCII module and TODO: ascii DAO name (#161)" This reverts commit d3eb047. * Revert "[Feature]DAOAccount Todo : add exists_upgrade_plan_cap (#160)" This reverts commit 1fb3b8a. * Revert "fix eventutil (#146)" This reverts commit d2c4f5c. * Revert "[Feature] Make StarcoinDAO inherit Dao's config (#144)" This reverts commit 8675779. * Revert "migrate script to entry function (#142)" This reverts commit 971a1a2. * Revert "[Feature] Multi Offer (#135)" This reverts commit 4ec8c2f. * Revert "add upgrade plan event (#128)" This reverts commit e82c1d3. * Revert "merge upgrade entry v12 and v12_1 (#126)" This reverts commit 47b38ad. * Revert "[Feature] NFTGallery Add Errors Assert (#138)" This reverts commit 61392b0. * Revert "Add event util (#134)" This reverts commit 545f191. * Revert "#131 Add only_new_module (#132) " This reverts commit f1363a3. * Revert "[Feature] DAO incompatible module upgrade test (#133)" This reverts commit 022ca1a. * Revert "update_module_upgrade_strategy support set min_time_limit (#127)" This reverts commit c8c9ba7. * Revert "Daospace support plugin marketplace (#116)" This reverts commit 6cccb73. * Revert "check StakeList existence (#124)" This reverts commit dee9a0b. * Revert "[ Feature ] do deposit when proposal state is DEFEATED (#122)" This reverts commit 3274821. * Revert "[daospace] Remove proposal info and add get function for proposal (#121)" This reverts commit 7b44ee7. * Revert "[integration-test] StarcoinDAO integration-test (#112)" This reverts commit 45f017a. * Revert "add assertion in stake to `StakeToSBTPlugin::stake` while the lock ti… (#117)" This reverts commit 08928d4. * Revert "Add Script function entry (#111)" This reverts commit 4c32fda. * Revert "Fix DAO Pulgin Event (#118)" This reverts commit 3cda388. * Revert "Plugin Event (#97)" This reverts commit 31b9a9c. * Revert "fix SalaryGovPlugin and migrate to test (#115)" This reverts commit 3e95aef. * Revert "[account] Retry when creating delegate account (#110)" This reverts commit 7a38b4b. * Revert "Add upgrade plugin And add StarcoinDAO (#94)" This reverts commit 85418d7. * Revert "able to update DAOExt (#108)" This reverts commit fc7ac4a. * Revert "[Feature] Public Native name_of function (#107)" This reverts commit 7263b0a. * Revert "MintProposalPlugin (#105)" This reverts commit ff2d802. * Revert "add daospace module upgrade test (#98)" This reverts commit 70429e2. * Revert "Add DAO nft image and member image (#96)" This reverts commit 2c89bbc. * Revert "Dao integration test and upgrade mpm (#95)" This reverts commit 82032cc. * Revert "add integration-test for refund_grant (#93)" This reverts commit 52fb86d. * Revert "signature.move add secp256k1_verify function (#86)" This reverts commit 9ea61ce. * Revert "check DAOAccountCap (#89)" This reverts commit 0404a53. * Revert "Add DAOSpace description (#85)" This reverts commit a981f67. * Revert "Remove decrease SBT when unstake item (#81)" This reverts commit d401c37. * Revert "[signature] Fix EVMAddress padding and crop bug (#84)" This reverts commit 41ad861. * Revert "fix dao snapshot deserialize when some option field is empty (#82)" This reverts commit b272a6f. * Revert "Complete DAOSpace (#80)" This reverts commit 98e96e6. * Revert "[dao] Implement DAOSpace (#32)" This reverts commit 0d37c89. * Revert "Update README.md (#68)" This reverts commit d3bb99f. * Revert "Update .gitattributes (#63)" This reverts commit 41ed34b. * Revert "support installing mpm from source (#56)" This reverts commit 3f46c6c. * Revert "[WIP] add Block state root (#39) (#41)" This reverts commit 67027f9. * Revert "Add multiple types of bcs skip and test (#52)" This reverts commit a5dc012. * Revert "upgrade rust toolchain 1.62.0 (#51)" This reverts commit 81c1e51. * Revert "implement Ring module (#44)" This reverts commit c9582a6. * Revert "support starcoin snapshot proof and verify (#45)" This reverts commit 63b05ec. * Revert "add bcs function , byte to basic type (#43)" This reverts commit e39dfeb.

baichuan3 added 8 commits August 15, 2022 11:03

daospace integration test testing

ce5da9a

call-pai multi call test case

037035a

integration test hang test case when call api in halley network

c68deb0

checkpoint integration test case

7ad8790

checkpoint integration test case update

b305976

checkpoint test can't not work

4315286

complete check point and vote test

bc20572

fixed dao space integration test

47a97a6

baichuan3 requested review from jolestar, WGB5445 and pause125 August 24, 2022 14:17

baichuan3 added 2 commits August 25, 2022 10:25

fix conflict by merge from main

ca3da83

restoresome test case exp

9f7f48d

pause125 reviewed Aug 25, 2022

View reviewed changes

integration-tests/daospace/daospace_proposal.move Show resolved Hide resolved

jolestar reviewed Aug 25, 2022

View reviewed changes

Move.toml Outdated Show resolved Hide resolved

remove SFT account in toml

3291cb4

pause125 reviewed Aug 25, 2022

View reviewed changes

fix vector borrow code

0e0a2a5

upgrade mpm version in ci and update all test case baseline

9d1ceff

pause125 mentioned this pull request Aug 28, 2022

mpm integration-test: fix discarded txn starcoinorg/starcoin#3691

Merged

7 tasks

fix some abort tests

2bd214c

jolestar approved these changes Aug 29, 2022

View reviewed changes

jolestar changed the title ~~Dao integration test~~ Dao integration test and upgrade mpm Aug 29, 2022

jolestar merged commit 82032cc into main Aug 29, 2022

jolestar deleted the dao_integration_test branch August 29, 2022 02:37

nkysg added a commit that referenced this pull request Mar 2, 2023

Revert "Dao integration test and upgrade mpm (#95)"

1f236bb

This reverts commit 82032cc.

nkysg added a commit that referenced this pull request Mar 2, 2023

Revert "Dao integration test and upgrade mpm (#95)"

1f709d9

This reverts commit 82032cc.

nkysg added a commit that referenced this pull request Mar 3, 2023

Revert "Dao integration test and upgrade mpm (#95)"

4c52bd0

This reverts commit 82032cc.

nkysg mentioned this pull request Mar 3, 2023

fix Revert daospace commit msg #227

Merged

7 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Dao integration test and upgrade mpm #95

Dao integration test and upgrade mpm #95

baichuan3 commented Aug 24, 2022

baichuan3 commented Aug 25, 2022

jolestar commented Aug 25, 2022

pause125 commented Aug 25, 2022

pause125 Aug 25, 2022

baichuan3 Aug 26, 2022

pause125 Aug 25, 2022

baichuan3 Aug 26, 2022

pause125 Aug 25, 2022

baichuan3 Aug 26, 2022

jolestar commented Aug 26, 2022

pause125 commented Aug 26, 2022

baichuan3 commented Aug 27, 2022

jolestar commented Aug 27, 2022

jolestar commented Aug 27, 2022

pause125 commented Aug 28, 2022

jolestar commented Aug 28, 2022

jolestar commented Aug 29, 2022

Dao integration test and upgrade mpm #95

Dao integration test and upgrade mpm #95

Conversation

baichuan3 commented Aug 24, 2022

Pull request type

What is the current behavior?

What is the new behavior?

Other information

baichuan3 commented Aug 25, 2022

jolestar commented Aug 25, 2022

pause125 commented Aug 25, 2022

pause125 Aug 25, 2022

Choose a reason for hiding this comment

baichuan3 Aug 26, 2022

Choose a reason for hiding this comment

pause125 Aug 25, 2022

Choose a reason for hiding this comment

baichuan3 Aug 26, 2022

Choose a reason for hiding this comment

pause125 Aug 25, 2022

Choose a reason for hiding this comment

baichuan3 Aug 26, 2022

Choose a reason for hiding this comment

jolestar commented Aug 26, 2022

pause125 commented Aug 26, 2022

baichuan3 commented Aug 27, 2022

jolestar commented Aug 27, 2022

jolestar commented Aug 27, 2022

pause125 commented Aug 28, 2022

jolestar commented Aug 28, 2022

jolestar commented Aug 29, 2022