-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release v1.7.0 testing/verification #379
Comments
Client APIVerify grpc-web support is enabled and working ????To do this I checked and verified that config files were being updated correctly to set grpc-web to true. If there is a better way to test this that anyone knows about, please comment. Verify rosetta interface is enabled and functional
Also, if you haven't already you will need to install rosetta-cli and you can find instructions for that here: https://github.com/coinbase/rosetta-cli Verify test suite can pass against a chain that has had many markers created/minted/burned/etc post genesis - not completeI am currently waiting to get testnet data from devops for testnet-beta and then I am going to run a local node against that and verify that all the tests pass as I believe that testnet-beta would have many marker transactions in it. FeegrantI used multiple accounts to verify that I could do feegrants to various accounts which had 0 hash and that those accounts could then do transactions using the granters address to pay the fees. I tested both basic and periodic tests and was able to verify both the unlimited case and setting specific amounts. I also verified that the periodic tests updates the amount a user can have for fees correctly. I used various combinations of the following commands while running locally: Grant with spend limitprovenanced -t tx feegrant grant "$addrV" "$addr1" --spend-limit 100000000nhash --gas-prices="1905nhash" --gas=auto --gas-adjustment=1.5 --home=./build/run/provenanced --chain-id="testing" --yes Grant without spend limitprovenanced -t tx feegrant grant "$addr3" "$addr2" --gas-prices="1905nhash" --gas=auto --gas-adjustment=1.5 --home=./build/run/provenanced --chain-id="testing" --yes Grant with a periodprovenanced -t tx feegrant grant "$addrV" "$addr1" --gas-prices="1905nhash" --gas=auto --gas-adjustment=1.5 --home=./build/run/provenanced --chain-id="testing" --yes --period=30 --period-limit=200000000nhash Queryprovenanced -t q feegrant grant "$addrV" "$addr1" --home=./build/run/provenanced --chain-id="testing" Query for grantsprovenanced -t q feegrant grants "$addr1" --home=./build/run/provenanced --chain-id="testing" Revoke Grantprovenanced -t tx feegrant revoke "$addrV" "$addr1" --gas-prices="1905nhash" --gas=auto --gas-adjustment=1.5 --home=./build/run/provenanced --chain-id="testing" --yes Using grant funds for txsInclude the following at the end of a transaction to use the feegranter to pay for the fee rather than the account of the tx signer: --fee-account="$addrV" I also added a small pr to cosmos-sdk to fix a minor issue I uncovered while testing the Verify correct Rosetta support for IBC/marker minted tokens - FAILEDI looked through the issue and all the connected issues and was unable to find a resolution for this in coinbase or in cosmos. I tested it following the same steps as before for testing locally and then I created a new coin and gave some on one of the users in the network. I then ran the exact same rosetta tests as before and it failed on syncing up the blocks on the block where the new coin was minted at. I also used: #434 to test an alternate coin I created that I named |
To verify swagger API I did the following:
However, I did see the following output once the process was complete:
|
To verify
Results from from 3 & 4. Ascending order
Descending order:
|
To verify CLI pagination #332, I took the following steps:
Note that I had to use the following additional flags to successfully run my tests. Thank you @carlton for helping me figure this out. Other noteworthy flags: Before proceeding with the tests see step 3 above. Test results (TLDR)
In addition, Attribute module (PASSED)
Add
Add
Marker module (PASSED)
Name module (PASSED)
Metadata module (FAILED)To test pagination on the
List of all locators
|
Verify To verify
Check
Check
Retrieve output in JSON format using the
|
Summary
The 0.43 Cosmos SDK and associated v1.7.0 Provenance blockchain update contains more significant changes than the typical point release update. These changes require an indepth testing and verification process to be completed before the release can be deployed to
pio-testnet-1
andpio-mainnet-1
networks.Testing/Verification Areas
Client API
grpc-web
support is enabled and working (see gRPC-web proxy cosmos/cosmos-sdk#8077 ) @fkneeland-figureVerify test suite can pass against a chain that has had many markers created/minted/burned/etc post genesis @fkneeland-figure--reverse
pagination support (see Support reverse iteration on pagination cosmos/cosmos-sdk#8754 ) @egaxhaj-figureprovenanced query staking validators
provenanced query staking validators --reverse
secp256r1
account and checking that grpc and api methods properly return/handle longer address.make proto-swagger-gen
output (no errors, contains complete set of endpoints) @egaxhaj-figureGenesis, Migration @channa-figure
v1.7.0
and verify network start with resulting genesis file.1
) and perform correct migrations to2
as required duringv1.7.0
upgrade processSmart Contracts
Keychain and secp256r1 support
secp256r1
key/account for all operationssecp256r1
based account to thelocalnet
development configurationprovenanced keys add testkey --recover
operation for 24+1 word mnemonic and phrase without added passwordauthz
provenanced tx authz grant
: Granter can issue grants of normal & generic authorizations @channa-figureprovenanced tx authz revoke
: Granter can revoke authorizations & grantee can no longer execute msg's @channa-figureprovenanced tx authz exec
: Grantee can execute authorized msg's @channa-figureprovenanced q authz grants
: Querying of grants returns the correct response @channa-figureauthz
MsgExec emit events from all executed msgs. (see feat: make authz MsgExec emit events from all executed msgs cosmos/cosmos-sdk#9522 )marker transfer
integration support forauthz
Integrate authz module into provenance and use it to transfer restricted marker's #265 @channa-figurefeegrant
provenanced tx feegrant grant
: Granter can issue fee grants of basic & periodic allowances, and grantee can spend from allowance @fkneeland-figureprovenanced q feegrant grant
: Querying for a specific fee grant from a granter/grantee pair works as expected @fkneeland-figureprovenanced tx feegrant revoke
: Granter can revoke fee grant allowance, and grantee can no longer spend from allowance @fkneeland-figureprovenanced q feegrant grants
: Querying of all fee grants for a grantee works as expected @fkneeland-figureGas
IBC
Verify correct Rosetta support for IBC/marker minted tokens (see Cosmos minted denoms not picked up coinbase/mesh-cli#230 ) @fkneeland-figureFor Admin Use
The text was updated successfully, but these errors were encountered: