search

maxonrow / maxonrow-go

Maxonrow Blockchain
https://www.maxonrow.com/
5 stars 6 forks source link

multiSig NFTs testing - add new test-cases #89

Closed githubckgoh1439 closed 4 years ago

githubckgoh1439 commented 4 years ago

Hi, Mostafa

Need help to check on line-84, where below Error occur :

  1. I already double-check the signer value, is correct.
  2. Is that caused by I using only one signer, which should broadcast immediately ?

Please comment.

--- FAIL: TestTxs (13.71s)
    tx_test.go:92: 
            Error Trace:    tx_test.go:92
            Error:          Should be zero, but was 1
            Test:           TestTxs
            Messages:       test case 23(Create MultiSig Tx for NFTs [Approve-token] - Happy path -  commit.) check should not fail: {"codespace":"sdk","code":1,"message":"Internal transaction signature error....goh123 : ERROR:\nCodespace: sdk\nCode: 4\nMessage: \"signature verification failed; verify correct account sequence and chain-id{\\\"account_number\\\":\\\"2\\\",\\\"chain_id\\\":\\\"maxonrow-chain\\\",\\\"fee\\\":{\\\"amount\\\":[{\\\"amount\\\":\\\"100000000\\\",\\\"denom\\\":\\\"cin\\\"}],\\\"gas\\\":\\\"0\\\"},\\\"memo\\\":\\\"\\\",\\\"msgs\\\":[{\\\"type\\\":\\\"nonFungible/setNonFungibleTokenStatus\\\",\\\"value\\\":{\\\"owner\\\":\\\"mxw1ld3stcsk5l8xjngw2ucuazux895rk2hxve69gr\\\",\\\"payload\\\":{\\\"pub_key\\\":{\\\"type\\\":\\\"tendermint/PubKeySecp256k1\\\",\\\"value\\\":\\\"A0VBHXKgUEU2fttqh8Lhqp1G6+GzOxTXvCExzDLEdfD7\\\"},\\\"signature\\\":\\\"3JB3cIrLigVmyNXk5iBoUb2NPXFLDkqNVyJtF8Y0JhEzJWGk70FCrGSQ1Sa1nFzmfuqs0Wrj8inciCYYAFzofQ==\\\",\\\"token\\\":{\\\"burnable\\\":true,\\\"endorserList\\\":[\\\"mxw1f8r0k5p7s85kv7jatwvmpartyy2j0s20y0p0yk\\\",\\\"mxw1k9sxz0h3yeh0uzmxet2rmsj7xe5zg54eq7vhla\\\"],\\\"from\\\":\\\"mxw1f8r0k5p7s85kv7jatwvmpartyy2j0s20y0p0yk\\\",\\\"mintLimit\\\":\\\"2\\\",\\\"modifiable\\\":true,\\\"nonce\\\":\\\"0\\\",\\\"pub\\\":false,\\\"status\\\":\\\"APPROVE\\\",\\\"symbol\\\":\\\"TNFT-PUBLIC-FALSE-00\\\",\\\"tokenFees\\\":[{\\\"action\\\":\\\"transfer\\\",\\\"feeName\\\":\\\"default\\\"},{\\\"action\\\":\\\"mint\\\",\\\"feeName\\\":\\\"default\\\"},{\\\"action\\\":\\\"burn\\\",\\\"feeName\\\":\\\"default\\\"},{\\\"action\\\":\\\"transferOwnership\\\",\\\"feeName\\\":\\\"default\\\"},{\\\"action\\\":\\\"acceptOwnership\\\",\\\"feeName\\\":\\\"default\\\"}],\\\"transferLimit\\\":\\\"2\\\",\\\"transferable\\\":true}},\\\"signatures\\\":[{\\\"pub_key\\\":{\\\"type\\\":\\\"tendermint/PubKeySecp256k1\\\",\\\"value\\\":\\\"Ausyj7Gas2WkCjUpM8UasCcezXrzTMTRbPHqYx44GzLm\\\"},\\\"signature\\\":\\\"IatzT1omvjRGTcxUde8NmrxoWgYBwj4hmPodQsQWN0w6dxpQyqaa8E8WDtVEmjEfFvfZmf6fUznsETChfjVlXw==\\\"}]}}],\\\"sequence\\\":\\\"6\\\"}\"\n"}
FAIL

]
githubckgoh1439 commented 4 years ago

Hi, Mostafa n YK

We need discussion on this NFTs-multisig part later as I encounter above. Please let me know your available timing.

githubckgoh1439 commented 4 years ago

Dear all,

please refer to this latest update :

https://github.com/maxonrow/maxonrow-go/pull/97