Replaced deprecated SetFullFundraiserPath with SetCoinType and SetPurpose #1037
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nullpointer0x00
approved these changes
Aug 23, 2022
arnabmitra
reviewed
Aug 23, 2022
Codecov Report
@@ Coverage Diff @@
## dwedul/995-bump-cosmos-to-0.46.0-pio-1-rc2 #1037 +/- ##
==============================================================================
+ Coverage 57.25% 57.34% +0.08%
==============================================================================
Files 178 178
Lines 22107 22109 +2
==============================================================================
+ Hits 12658 12678 +20
+ Misses 8482 8464 -18
Partials 967 967
|
added 2 commits
August 24, 2022 09:28
…aram. This was mainly to help with testing.
…github.com/provenance-io/provenance into taztingo/1011-replace-deprecated-setfullfundraiser-path
Taztingo
deleted the
taztingo/1011-replace-deprecated-setfullfundraiser-path
branch
4 tasks
SpicyLemon
added a commit
that referenced
this pull request
Sep 20, 2022
* [995]: Bump cosmos to v0.46.0-pio-1-rc1 (from v0.45.5-pio-1). * [995]: Add changelog line. * [995]: Remove call to authrest.RegisterTxRoutes since it was deprecated and now no longer exists. * [995]: Replace import of github.com/cosmos/cosmos-sdk/x/auth/legacy/legacytx with github.com/cosmos/cosmos-sdk/x/auth/migrations/legacytx since it was moved. * [995]: Remove imports of github.com/cosmos/cosmos-sdk/client/rest which was deprecated and no longer exists. Also remove all the RegisterRESTRoutes functions and calls. * [995]: Update the name client's call to govclient.NewProposalHandler and delete the name client's rest.go since nothing in there is used anymore and github.com/cosmos/cosmos-sdk/x/gov/client/rest has been removed. * [995]: Remove all uses of github.com/cosmos/cosmos-sdk/types/rest which was deprecated, and has now been removed. * [995]: Update ibc-go to v5.0.0-beta1 since that's the first one that supports cosmos v0.46. * [995]: Remove google.golang.org/grpc v1.33.2 replace line. It was added when we bumped cosmos to v0.41.0 (from v0.40.1). But now some stuff in the newer version (v1.48.0) is needed by cosmos. * [995]: Update the cosmos-sdk version in the require section (even though it's being replaced below). * [995]: Add a replace line for wasmd to use a local verison. There's a draft PR to update wasmd to v0.46: CosmWasm/wasmd#930 That's what I've got locally and will be referencing. Not yet sure what I'll do if I need to push this up before they've got a tagged version. * [995]: go mod tidy (finally, it works). * [995]: Add v0.46 to the TODO in go.mod to help identify it as something that needs to be done as part of the v0.46 update. * [995]: Fix compliation issues in app.go. * [995]: Fix compilation issues in root.go. * [995]: Switch to using storetypes.StoreKey (from sdk.StoreKey). * [995]: Fix uses of coins.SafeSub since it's now vararg. * [995]: Fix the gov stuff that moved to v1beta1. * [995]: Add the FeeGasMeter.GasRemaining function that was added to the interface. * [995]: Update the antewrapper handler to use a couple new functions that changed. * [995]: Fix a couple more govtypes moves. * [995]: change a coins.Sub call to use varargs. * [995]: Provide the expiration time as a reference to authz.NewMsgGrant since they seem to have changed it to a reference. * [995]: Fix uses of the renamed GenSignedMockTx (was GenTx) and SimDeliver (was Deliver) sim functions. * [995]: update call to AddressAndDenomFromBalancesStore (used to be AddressFromBalancesStore). * [995]: Fix call to GetPubKey that now also returns an error (instead of panic). * [995]: Switch to a local version of cosmos so I can test the authz fix. * [995]: Switch to GetAuthorization (from GetCleanAuthorization). * [995]: Replace the rest of the GenTx calls with GenSignedMockTx. * [995]: Fix uses of simapp.FundAccount and standardize all of them to just use the bank module's testutil.FundAccount. * [995]: Fix the rest of the uses of SimDeliver (was previously named Deliver). * [995]: Remove a bunch of test helpers that aren't used but are available from the SDK. * [995]: Delete our version of EmptyAppOptions which was just a copy of the cosmos version, and switch to using the cosmos version. * [995]: Standardize the import of github.com/cosmos/cosmos-sdk/simapp to be aliased to sdksim. * [995]: Fix uses of AccAddressFromHexUnsafe (used to be named AccAddressFromHex). * [995]: Fix uses of testnet.New. * [995]: Remove the use of MaxWasmCodeSize and set the MaxWasmSize instead. * [995]: Fix simulation call to wasm.NewAppModule, which requires some new arguments. * [995]: Remove use of UpgradeInfo and replace it with Plan. * [995]: Switch to .Codec (from JSONCodec) since the latter was removed from the context and we're supposed to use .Codec now. * [995]: Provide a codec to all calls to kerying.New. * [995]: Update uses of keyring.Record.GetAddress to account for the additional error return value. * [995]: gov module's GenesisState is now in v1beta1 and the GenerateSaveCoinKey moved to testutil. * [995]: switch to keyring.Record (used be called Info). Add handling of the error return value from GetAddress (newly added). * [995]: account for the added error return value from GetPubKey. Call BuildUnsignedTx directly from the factory since it was moved there. * [995]: gov module's GenesisState was moved to v1beta1. * [995]: Fix several complaints about the receivers in the dbmigrator migrator tests. * [995]: Fix several complaints about the ReflectorTestSuit method receivers having a lock but being concrete. * [995]: Correct uses of the encoding config's Codec (used to be called Marshaller). * [995]: Update a use of NewDeductFeeDecorator (used to be called NewMempoolFeeDecorator). * [995]: Fix some complaints about variable names conflicting with import names. * [995]: Comment out a couple more uses of SetMsgServiceRouter (with a TODO above them). Fix the expiration arg in all calls to SaveGrant and replace the last few uses of GetCleanAuthorization. * [995]: Switch to pruning/types for the pruning options (was moved from store/types). * [995]: Make the marker HandlerTestSuite methods have a pointer receiver since it does some lock stuff and Intellij complains. * [995]: The InitialStake field now requires an sdk Int. * [995]: The sdksim.Setup function now also takes in the Testing variable. * [995]: The NewTextProposal function moved to gov types v1beta1. * [995]: Use the SetMsgServiceRouter again. * [995]: Fix some errorcheck lint complaints. The functions that it's now complaining about all are hard-coded to return nil. I thought about just putting a nolint comment on them stating such, but eh, this protects us from future changes to those SDK functions. * [995]: The clientCtx.Codec returns an interface now, so there's no need to type-assert/convert it anymore. This fixes some gocritic linter complaints. * [995]: Tweak the makefile a bit. Don't output errors from go mod list when getting the TM_VERSION; those will be output later if that value is needed. Echo the commands from several targets that used to have them suppressed; just makes it easier to troubleshoot. Use the GO variable for all go executions. Do a go.sum before running tests. * [995]: Rip out all the stuff added to facilitate having a kafka dependency. * [995]: Switch to cosmos version v0.46.0-pio-1-rc2. * [995]: Update app/test_helpers.go to bring it in line with Cosmos' simapp/test_helpers.go. Also stop using DefaultNodeHome for unit tests. * [995]: Put the marker authz_test in the types_test package due to circular import cycle with /app. * [995]: Bring prepForZeroHeightGenesis inline with the updated simapp version and clean up the TestSimAppExportAndBlockedAddrs to be closer to the cosmos version too (and use the new stuff). * [995]: Remove the config index-events workaround since it was fixed in v0.46. * [995]: Tweak some fee antewrapper errors to make them match Cosmos' similar ones. Also, the addresses were being converted to hex instead of bech32. So this is better now. * [995]: Fix the TestEnsureMempoolFees test because Cosmos now checks for fund availability. Also clean up the antewrapper tests so that they only run once (they were running thrice). And update the TestDeductFeesNoDelegation to be more robust and make sure we're getting the expectd errors instead of just any error. * [995]: Segment TestMsgService since it's failing and need to slog through it. Bump the gas used in there since it's no longer enough again, and remove the check function and all it's uses, replaced with require NoError. * [995]: Fix unit test that was failing because TextProposal moved to v1beta1 (from types) and is in an error string check. * [995]: In the metadata cli test, set the suite's config value before generating the keyring stuff so that it can use the config as intended (not sure how it didn't panic before). Also change a couple expected error strings because some errors changed. * [995]: Fix the marker cli tests. Need more gas for the gov proposals. Need to set the suite's cfg earlier (same as what metadate cli tests did). And now expect a 0x9 instead of 0x5 from the withdraw proposal. * [995]: Fix the marker handler test, invalid proposal since the package changed for TextProposal (to v1beta1 from types). * [995]: Correct an incorrectly updated comment (that might be useful when fixing stuff in a bit). * [995]: Fix the marker operations test that changed due to added use of the random number generator by the SDK. Also added a note about that and a list of valid options that things might change to in the future. But I left it looking for one specific name since that shouldn't change unless making changes to the SDK or operations. * [995]: Fix the Holding query since the data is stored differently now. key is <prefix><account><denom>, value is <amount>. Used to be key is <prefix><account>, value is <coin>. * [995]: Refactor the marker Holding query to use the sdk's DenomOwners query (should have better performance). * [995]: lint fixes. * [995]: Undo a lot of the formatting/division changes that I had made to TestMsgService since we're going to fix that in a separte PR. * [995]: Skip several MsgService tests, to be fixed in a separate PR. * [995]: Switch the TODO comment in msg_fees_decorator to reference that it's required for 1.12. * [995]: Enhance the changelog with more info about what's being done here. * [995]: Switch wasm to a version tagged on our own fork so that we can keep moving forward. * [995]: Add new upgrade handler that runs migrations. Add a couple 1.12.x TODO entries. * [995]: Remove unused deprecated Conditions field. * [995]: Replace deprecated use of strings.Title. * [995]: Replace use of a couple deprecated gov cli flags, and just make local versions of them to use instead. * [995]: Correct the changelog entry about the DenomOwners query. * [995]: Undo a comment change that will be handy during a followup. * [995]: Delete a couple missed references to sims-state-listening.mk. * [1010]: Replace use of the deprecated sdkResponse.Data field. The documentation says it's still usable, but something changed because it made the sim tests all start failing due to index out of range (asking for 0, but no entries). That indicates that stuff was missing from the Data field, and so we can't use it anymore. * [995]: Fix the runsim sims. I accidentally got rid of the -tags part that allows the other dbs to be used. Also simplified it further since there doesn't need to be differences between the runsim and go test versions. * [1010]: Tweak the .Data fix to use the GetCachedValue off the response rather than reunmarshaling it from the bytes. * [995]: Add unit tests on the ExportAppStateAndValidators function. * [995]: Fix the casting of the MsgInstantiateContractResponse. * [995]: Update the TestAppImportExport simulation to bring it in line with it's cosmos-sdk counterpart. Fail if the db can't close. Handle the panic when there are no validators. Provide some keys to the authzkeeper checking. * [995]: Fix a couple marker tests that started failing because I fixed the account numbers thing. * [995]: Bump the sdk to v0.46.0-pio-1-rc3 (from v0.46.0-pio-1-rc2). * [995]: Bump cosmos-sdk to v0.46.0-pio-1-rc4 (from v0.46.0-pio-1-rc3). * [995]: Add ochre-rc1 upgrade handler. * [995]: Fix the changelog so that this new stuff isn't listed in the 1.12 section. * [995]: Update the TODO: Required comments to reference v1.13.x (instead of v1.12.x). * [995]: Fix the name of the TestHandlerTestSuite (was TestAnteTestSuite, which is a different thing). * [995]: Undo a change to the msg_fee_invoker that I made at some point, but now don't want since it's in an area to be fixed anyway. * Updated to remove deprecated sdk.Int. (#1036) * [995]: Disable the sims for the same reason I needed to disable some of the msg service router tests. Added TODO comments about removing the skips too. * [995]: Add issue links to the TODOs needed to be todone with issue 1006 (the fees stuff). * [995]: Add issue links to the other new TODOs. * [995]: Bump cosmos to v0.46.0-pio-1-rc5 (from v0.46.0-pio-1-rc4). * Replaced deprecated SetFullFundraiserPath with SetCoinType and SetPurpose (#1037) * Removed SetFullFundraiserPath and replaced it with SetCoinType and SetPurpose. * Added prefix_test and updated SetConfig to have a configurable seal param. This was mainly to help with testing. * [995]: Lint fixes. * [995]: Add some lookups to the TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations test for some extra debug info. * [995]: Fix import order. * [995]: Refactor TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations a bit. Make it so that the votes are coming from an account that isn't a validator. * [995]: Remove the nickel upgrade handler and combine it with my ochre one. * [995]: Add missing comment about how the tests are initiated. * [995]: Lint fixes. * [995]: Fix the TestFullBIP44Path test to actually test what we're trying to test. * [995]: Use the FundModuleAccount function instead of minting and sending. * [995]: Bump Cosmos-SDK to v0.46.1-pio-1-rc1 (from v0.46.0-pio-1-rc5). It's exactly the same, just named more accurately. * [995]: lazy-load the govVoteMsgUrls messages, just to be safe. * [995]: Update the isGovernanceMessage func to account for the new version of governance messages. * [995]: Fix the gov msg count. * [995]: Lint fixes. I really hate that the Go convention for acronyms in variable names is to capitalize all or none of them. * [995]: Switch msgURL setting logic to just look for an empty slice instead of slice of specific length. Makes it a little more future-safe. * [995]: Switch those checks to a nil check and add a comment about it. Co-authored-by: Matt Witkowski <[email protected]>
SpicyLemon
added a commit
that referenced
this pull request
Sep 22, 2022
…1042) * [995]: Bump cosmos to v0.46.0-pio-1-rc1 (from v0.45.5-pio-1). * [995]: Add changelog line. * [995]: Remove call to authrest.RegisterTxRoutes since it was deprecated and now no longer exists. * [995]: Replace import of github.com/cosmos/cosmos-sdk/x/auth/legacy/legacytx with github.com/cosmos/cosmos-sdk/x/auth/migrations/legacytx since it was moved. * [995]: Remove imports of github.com/cosmos/cosmos-sdk/client/rest which was deprecated and no longer exists. Also remove all the RegisterRESTRoutes functions and calls. * [995]: Update the name client's call to govclient.NewProposalHandler and delete the name client's rest.go since nothing in there is used anymore and github.com/cosmos/cosmos-sdk/x/gov/client/rest has been removed. * [995]: Remove all uses of github.com/cosmos/cosmos-sdk/types/rest which was deprecated, and has now been removed. * [995]: Update ibc-go to v5.0.0-beta1 since that's the first one that supports cosmos v0.46. * [995]: Remove google.golang.org/grpc v1.33.2 replace line. It was added when we bumped cosmos to v0.41.0 (from v0.40.1). But now some stuff in the newer version (v1.48.0) is needed by cosmos. * [995]: Update the cosmos-sdk version in the require section (even though it's being replaced below). * [995]: Add a replace line for wasmd to use a local verison. There's a draft PR to update wasmd to v0.46: CosmWasm/wasmd#930 That's what I've got locally and will be referencing. Not yet sure what I'll do if I need to push this up before they've got a tagged version. * [995]: go mod tidy (finally, it works). * [995]: Add v0.46 to the TODO in go.mod to help identify it as something that needs to be done as part of the v0.46 update. * [995]: Fix compliation issues in app.go. * [995]: Fix compilation issues in root.go. * [995]: Switch to using storetypes.StoreKey (from sdk.StoreKey). * [995]: Fix uses of coins.SafeSub since it's now vararg. * [995]: Fix the gov stuff that moved to v1beta1. * [995]: Add the FeeGasMeter.GasRemaining function that was added to the interface. * [995]: Update the antewrapper handler to use a couple new functions that changed. * [995]: Fix a couple more govtypes moves. * [995]: change a coins.Sub call to use varargs. * [995]: Provide the expiration time as a reference to authz.NewMsgGrant since they seem to have changed it to a reference. * [995]: Fix uses of the renamed GenSignedMockTx (was GenTx) and SimDeliver (was Deliver) sim functions. * [995]: update call to AddressAndDenomFromBalancesStore (used to be AddressFromBalancesStore). * [995]: Fix call to GetPubKey that now also returns an error (instead of panic). * [995]: Switch to a local version of cosmos so I can test the authz fix. * [995]: Switch to GetAuthorization (from GetCleanAuthorization). * [995]: Replace the rest of the GenTx calls with GenSignedMockTx. * [995]: Fix uses of simapp.FundAccount and standardize all of them to just use the bank module's testutil.FundAccount. * [995]: Fix the rest of the uses of SimDeliver (was previously named Deliver). * [995]: Remove a bunch of test helpers that aren't used but are available from the SDK. * [995]: Delete our version of EmptyAppOptions which was just a copy of the cosmos version, and switch to using the cosmos version. * [995]: Standardize the import of github.com/cosmos/cosmos-sdk/simapp to be aliased to sdksim. * [995]: Fix uses of AccAddressFromHexUnsafe (used to be named AccAddressFromHex). * [995]: Fix uses of testnet.New. * [995]: Remove the use of MaxWasmCodeSize and set the MaxWasmSize instead. * [995]: Fix simulation call to wasm.NewAppModule, which requires some new arguments. * [995]: Remove use of UpgradeInfo and replace it with Plan. * [995]: Switch to .Codec (from JSONCodec) since the latter was removed from the context and we're supposed to use .Codec now. * [995]: Provide a codec to all calls to kerying.New. * [995]: Update uses of keyring.Record.GetAddress to account for the additional error return value. * [995]: gov module's GenesisState is now in v1beta1 and the GenerateSaveCoinKey moved to testutil. * [995]: switch to keyring.Record (used be called Info). Add handling of the error return value from GetAddress (newly added). * [995]: account for the added error return value from GetPubKey. Call BuildUnsignedTx directly from the factory since it was moved there. * [995]: gov module's GenesisState was moved to v1beta1. * [995]: Fix several complaints about the receivers in the dbmigrator migrator tests. * [995]: Fix several complaints about the ReflectorTestSuit method receivers having a lock but being concrete. * [995]: Correct uses of the encoding config's Codec (used to be called Marshaller). * [995]: Update a use of NewDeductFeeDecorator (used to be called NewMempoolFeeDecorator). * [995]: Fix some complaints about variable names conflicting with import names. * [995]: Comment out a couple more uses of SetMsgServiceRouter (with a TODO above them). Fix the expiration arg in all calls to SaveGrant and replace the last few uses of GetCleanAuthorization. * [995]: Switch to pruning/types for the pruning options (was moved from store/types). * [995]: Make the marker HandlerTestSuite methods have a pointer receiver since it does some lock stuff and Intellij complains. * [995]: The InitialStake field now requires an sdk Int. * [995]: The sdksim.Setup function now also takes in the Testing variable. * [995]: The NewTextProposal function moved to gov types v1beta1. * [995]: Use the SetMsgServiceRouter again. * [995]: Fix some errorcheck lint complaints. The functions that it's now complaining about all are hard-coded to return nil. I thought about just putting a nolint comment on them stating such, but eh, this protects us from future changes to those SDK functions. * [995]: The clientCtx.Codec returns an interface now, so there's no need to type-assert/convert it anymore. This fixes some gocritic linter complaints. * [995]: Tweak the makefile a bit. Don't output errors from go mod list when getting the TM_VERSION; those will be output later if that value is needed. Echo the commands from several targets that used to have them suppressed; just makes it easier to troubleshoot. Use the GO variable for all go executions. Do a go.sum before running tests. * [995]: Rip out all the stuff added to facilitate having a kafka dependency. * [995]: Switch to cosmos version v0.46.0-pio-1-rc2. * [995]: Update app/test_helpers.go to bring it in line with Cosmos' simapp/test_helpers.go. Also stop using DefaultNodeHome for unit tests. * [995]: Put the marker authz_test in the types_test package due to circular import cycle with /app. * [995]: Bring prepForZeroHeightGenesis inline with the updated simapp version and clean up the TestSimAppExportAndBlockedAddrs to be closer to the cosmos version too (and use the new stuff). * [995]: Remove the config index-events workaround since it was fixed in v0.46. * [995]: Tweak some fee antewrapper errors to make them match Cosmos' similar ones. Also, the addresses were being converted to hex instead of bech32. So this is better now. * [995]: Fix the TestEnsureMempoolFees test because Cosmos now checks for fund availability. Also clean up the antewrapper tests so that they only run once (they were running thrice). And update the TestDeductFeesNoDelegation to be more robust and make sure we're getting the expectd errors instead of just any error. * [995]: Segment TestMsgService since it's failing and need to slog through it. Bump the gas used in there since it's no longer enough again, and remove the check function and all it's uses, replaced with require NoError. * [995]: Fix unit test that was failing because TextProposal moved to v1beta1 (from types) and is in an error string check. * [995]: In the metadata cli test, set the suite's config value before generating the keyring stuff so that it can use the config as intended (not sure how it didn't panic before). Also change a couple expected error strings because some errors changed. * [995]: Fix the marker cli tests. Need more gas for the gov proposals. Need to set the suite's cfg earlier (same as what metadate cli tests did). And now expect a 0x9 instead of 0x5 from the withdraw proposal. * [995]: Fix the marker handler test, invalid proposal since the package changed for TextProposal (to v1beta1 from types). * [995]: Correct an incorrectly updated comment (that might be useful when fixing stuff in a bit). * [995]: Fix the marker operations test that changed due to added use of the random number generator by the SDK. Also added a note about that and a list of valid options that things might change to in the future. But I left it looking for one specific name since that shouldn't change unless making changes to the SDK or operations. * [995]: Fix the Holding query since the data is stored differently now. key is <prefix><account><denom>, value is <amount>. Used to be key is <prefix><account>, value is <coin>. * [995]: Refactor the marker Holding query to use the sdk's DenomOwners query (should have better performance). * [995]: lint fixes. * [995]: Undo a lot of the formatting/division changes that I had made to TestMsgService since we're going to fix that in a separte PR. * [995]: Skip several MsgService tests, to be fixed in a separate PR. * [995]: Switch the TODO comment in msg_fees_decorator to reference that it's required for 1.12. * [995]: Enhance the changelog with more info about what's being done here. * [995]: Switch wasm to a version tagged on our own fork so that we can keep moving forward. * [995]: Add new upgrade handler that runs migrations. Add a couple 1.12.x TODO entries. * [995]: Remove unused deprecated Conditions field. * [995]: Replace deprecated use of strings.Title. * [995]: Replace use of a couple deprecated gov cli flags, and just make local versions of them to use instead. * [995]: Correct the changelog entry about the DenomOwners query. * [995]: Undo a comment change that will be handy during a followup. * [995]: Delete a couple missed references to sims-state-listening.mk. * [1010]: Replace use of the deprecated sdkResponse.Data field. The documentation says it's still usable, but something changed because it made the sim tests all start failing due to index out of range (asking for 0, but no entries). That indicates that stuff was missing from the Data field, and so we can't use it anymore. * [995]: Fix the runsim sims. I accidentally got rid of the -tags part that allows the other dbs to be used. Also simplified it further since there doesn't need to be differences between the runsim and go test versions. * [1010]: Tweak the .Data fix to use the GetCachedValue off the response rather than reunmarshaling it from the bytes. * [995]: Add unit tests on the ExportAppStateAndValidators function. * [995]: Fix the casting of the MsgInstantiateContractResponse. * [995]: Update the TestAppImportExport simulation to bring it in line with it's cosmos-sdk counterpart. Fail if the db can't close. Handle the panic when there are no validators. Provide some keys to the authzkeeper checking. * [995]: Fix a couple marker tests that started failing because I fixed the account numbers thing. * [995]: Bump the sdk to v0.46.0-pio-1-rc3 (from v0.46.0-pio-1-rc2). * [995]: Bump cosmos-sdk to v0.46.0-pio-1-rc4 (from v0.46.0-pio-1-rc3). * [995]: Add ochre-rc1 upgrade handler. * [995]: Fix the changelog so that this new stuff isn't listed in the 1.12 section. * [995]: Update the TODO: Required comments to reference v1.13.x (instead of v1.12.x). * [995]: Fix the name of the TestHandlerTestSuite (was TestAnteTestSuite, which is a different thing). * [995]: Undo a change to the msg_fee_invoker that I made at some point, but now don't want since it's in an area to be fixed anyway. * Updated to remove deprecated sdk.Int. (#1036) * [995]: Disable the sims for the same reason I needed to disable some of the msg service router tests. Added TODO comments about removing the skips too. * [995]: Add issue links to the TODOs needed to be todone with issue 1006 (the fees stuff). * [995]: Add issue links to the other new TODOs. * [995]: Bump cosmos to v0.46.0-pio-1-rc5 (from v0.46.0-pio-1-rc4). * Replaced deprecated SetFullFundraiserPath with SetCoinType and SetPurpose (#1037) * Removed SetFullFundraiserPath and replaced it with SetCoinType and SetPurpose. * Added prefix_test and updated SetConfig to have a configurable seal param. This was mainly to help with testing. * [1013]: Update uses under internal/antewrapper. * [1013]: Update the rest of the uses under internal: handlers and provwasm. * [1013]: Fix uses in x/attribute. * [1013]: Fix uses in x/marker. * [1013]: Fix uses in x/metadata. * [1013]: Fix uses in x/msgfees. * [1013]: Fix uses in x/name. * [1013]: Add changelog entry. * [995]: Lint fixes. * [995]: Add some lookups to the TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations test for some extra debug info. * [995]: Fix import order. * [995]: Refactor TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations a bit. Make it so that the votes are coming from an account that isn't a validator. * [995]: Remove the nickel upgrade handler and combine it with my ochre one. * [995]: Add missing comment about how the tests are initiated. * [995]: Lint fixes. * [1013]: Fix uses of deprecated sdkerrors stuff in the rewards stuff (and stuff). * [995]: Fix the TestFullBIP44Path test to actually test what we're trying to test. * [995]: Use the FundModuleAccount function instead of minting and sending. * [995]: Bump Cosmos-SDK to v0.46.1-pio-1-rc1 (from v0.46.0-pio-1-rc5). It's exactly the same, just named more accurately. * [995]: lazy-load the govVoteMsgUrls messages, just to be safe. * [995]: Update the isGovernanceMessage func to account for the new version of governance messages. * [995]: Fix the gov msg count. * [995]: Lint fixes. I really hate that the Go convention for acronyms in variable names is to capitalize all or none of them. * [995]: Switch msgURL setting logic to just look for an empty slice instead of slice of specific length. Makes it a little more future-safe. * [995]: Switch those checks to a nil check and add a comment about it. * [1013]: Fix a couple unit tests that broke after the merge. Co-authored-by: Matt Witkowski <[email protected]>
SpicyLemon
added a commit
that referenced
this pull request
Sep 22, 2022
* [995]: Bump cosmos to v0.46.0-pio-1-rc1 (from v0.45.5-pio-1). * [995]: Add changelog line. * [995]: Remove call to authrest.RegisterTxRoutes since it was deprecated and now no longer exists. * [995]: Replace import of github.com/cosmos/cosmos-sdk/x/auth/legacy/legacytx with github.com/cosmos/cosmos-sdk/x/auth/migrations/legacytx since it was moved. * [995]: Remove imports of github.com/cosmos/cosmos-sdk/client/rest which was deprecated and no longer exists. Also remove all the RegisterRESTRoutes functions and calls. * [995]: Update the name client's call to govclient.NewProposalHandler and delete the name client's rest.go since nothing in there is used anymore and github.com/cosmos/cosmos-sdk/x/gov/client/rest has been removed. * [995]: Remove all uses of github.com/cosmos/cosmos-sdk/types/rest which was deprecated, and has now been removed. * [995]: Update ibc-go to v5.0.0-beta1 since that's the first one that supports cosmos v0.46. * [995]: Remove google.golang.org/grpc v1.33.2 replace line. It was added when we bumped cosmos to v0.41.0 (from v0.40.1). But now some stuff in the newer version (v1.48.0) is needed by cosmos. * [995]: Update the cosmos-sdk version in the require section (even though it's being replaced below). * [995]: Add a replace line for wasmd to use a local verison. There's a draft PR to update wasmd to v0.46: CosmWasm/wasmd#930 That's what I've got locally and will be referencing. Not yet sure what I'll do if I need to push this up before they've got a tagged version. * [995]: go mod tidy (finally, it works). * [995]: Add v0.46 to the TODO in go.mod to help identify it as something that needs to be done as part of the v0.46 update. * [995]: Fix compliation issues in app.go. * [995]: Fix compilation issues in root.go. * [995]: Switch to using storetypes.StoreKey (from sdk.StoreKey). * [995]: Fix uses of coins.SafeSub since it's now vararg. * [995]: Fix the gov stuff that moved to v1beta1. * [995]: Add the FeeGasMeter.GasRemaining function that was added to the interface. * [995]: Update the antewrapper handler to use a couple new functions that changed. * [995]: Fix a couple more govtypes moves. * [995]: change a coins.Sub call to use varargs. * [995]: Provide the expiration time as a reference to authz.NewMsgGrant since they seem to have changed it to a reference. * [995]: Fix uses of the renamed GenSignedMockTx (was GenTx) and SimDeliver (was Deliver) sim functions. * [995]: update call to AddressAndDenomFromBalancesStore (used to be AddressFromBalancesStore). * [995]: Fix call to GetPubKey that now also returns an error (instead of panic). * [995]: Switch to a local version of cosmos so I can test the authz fix. * [995]: Switch to GetAuthorization (from GetCleanAuthorization). * [995]: Replace the rest of the GenTx calls with GenSignedMockTx. * [995]: Fix uses of simapp.FundAccount and standardize all of them to just use the bank module's testutil.FundAccount. * [995]: Fix the rest of the uses of SimDeliver (was previously named Deliver). * [995]: Remove a bunch of test helpers that aren't used but are available from the SDK. * [995]: Delete our version of EmptyAppOptions which was just a copy of the cosmos version, and switch to using the cosmos version. * [995]: Standardize the import of github.com/cosmos/cosmos-sdk/simapp to be aliased to sdksim. * [995]: Fix uses of AccAddressFromHexUnsafe (used to be named AccAddressFromHex). * [995]: Fix uses of testnet.New. * [995]: Remove the use of MaxWasmCodeSize and set the MaxWasmSize instead. * [995]: Fix simulation call to wasm.NewAppModule, which requires some new arguments. * [995]: Remove use of UpgradeInfo and replace it with Plan. * [995]: Switch to .Codec (from JSONCodec) since the latter was removed from the context and we're supposed to use .Codec now. * [995]: Provide a codec to all calls to kerying.New. * [995]: Update uses of keyring.Record.GetAddress to account for the additional error return value. * [995]: gov module's GenesisState is now in v1beta1 and the GenerateSaveCoinKey moved to testutil. * [995]: switch to keyring.Record (used be called Info). Add handling of the error return value from GetAddress (newly added). * [995]: account for the added error return value from GetPubKey. Call BuildUnsignedTx directly from the factory since it was moved there. * [995]: gov module's GenesisState was moved to v1beta1. * [995]: Fix several complaints about the receivers in the dbmigrator migrator tests. * [995]: Fix several complaints about the ReflectorTestSuit method receivers having a lock but being concrete. * [995]: Correct uses of the encoding config's Codec (used to be called Marshaller). * [995]: Update a use of NewDeductFeeDecorator (used to be called NewMempoolFeeDecorator). * [995]: Fix some complaints about variable names conflicting with import names. * [995]: Comment out a couple more uses of SetMsgServiceRouter (with a TODO above them). Fix the expiration arg in all calls to SaveGrant and replace the last few uses of GetCleanAuthorization. * [995]: Switch to pruning/types for the pruning options (was moved from store/types). * [995]: Make the marker HandlerTestSuite methods have a pointer receiver since it does some lock stuff and Intellij complains. * [995]: The InitialStake field now requires an sdk Int. * [995]: The sdksim.Setup function now also takes in the Testing variable. * [995]: The NewTextProposal function moved to gov types v1beta1. * [995]: Use the SetMsgServiceRouter again. * [995]: Fix some errorcheck lint complaints. The functions that it's now complaining about all are hard-coded to return nil. I thought about just putting a nolint comment on them stating such, but eh, this protects us from future changes to those SDK functions. * [995]: The clientCtx.Codec returns an interface now, so there's no need to type-assert/convert it anymore. This fixes some gocritic linter complaints. * [995]: Tweak the makefile a bit. Don't output errors from go mod list when getting the TM_VERSION; those will be output later if that value is needed. Echo the commands from several targets that used to have them suppressed; just makes it easier to troubleshoot. Use the GO variable for all go executions. Do a go.sum before running tests. * [995]: Rip out all the stuff added to facilitate having a kafka dependency. * [995]: Switch to cosmos version v0.46.0-pio-1-rc2. * [995]: Update app/test_helpers.go to bring it in line with Cosmos' simapp/test_helpers.go. Also stop using DefaultNodeHome for unit tests. * [995]: Put the marker authz_test in the types_test package due to circular import cycle with /app. * [995]: Bring prepForZeroHeightGenesis inline with the updated simapp version and clean up the TestSimAppExportAndBlockedAddrs to be closer to the cosmos version too (and use the new stuff). * [995]: Remove the config index-events workaround since it was fixed in v0.46. * [995]: Tweak some fee antewrapper errors to make them match Cosmos' similar ones. Also, the addresses were being converted to hex instead of bech32. So this is better now. * [995]: Fix the TestEnsureMempoolFees test because Cosmos now checks for fund availability. Also clean up the antewrapper tests so that they only run once (they were running thrice). And update the TestDeductFeesNoDelegation to be more robust and make sure we're getting the expectd errors instead of just any error. * [995]: Segment TestMsgService since it's failing and need to slog through it. Bump the gas used in there since it's no longer enough again, and remove the check function and all it's uses, replaced with require NoError. * [995]: Fix unit test that was failing because TextProposal moved to v1beta1 (from types) and is in an error string check. * [995]: In the metadata cli test, set the suite's config value before generating the keyring stuff so that it can use the config as intended (not sure how it didn't panic before). Also change a couple expected error strings because some errors changed. * [995]: Fix the marker cli tests. Need more gas for the gov proposals. Need to set the suite's cfg earlier (same as what metadate cli tests did). And now expect a 0x9 instead of 0x5 from the withdraw proposal. * [995]: Fix the marker handler test, invalid proposal since the package changed for TextProposal (to v1beta1 from types). * [995]: Correct an incorrectly updated comment (that might be useful when fixing stuff in a bit). * [995]: Fix the marker operations test that changed due to added use of the random number generator by the SDK. Also added a note about that and a list of valid options that things might change to in the future. But I left it looking for one specific name since that shouldn't change unless making changes to the SDK or operations. * [995]: Fix the Holding query since the data is stored differently now. key is <prefix><account><denom>, value is <amount>. Used to be key is <prefix><account>, value is <coin>. * [995]: Refactor the marker Holding query to use the sdk's DenomOwners query (should have better performance). * [995]: lint fixes. * [995]: Undo a lot of the formatting/division changes that I had made to TestMsgService since we're going to fix that in a separte PR. * [995]: Skip several MsgService tests, to be fixed in a separate PR. * [995]: Switch the TODO comment in msg_fees_decorator to reference that it's required for 1.12. * [995]: Enhance the changelog with more info about what's being done here. * [995]: Switch wasm to a version tagged on our own fork so that we can keep moving forward. * [995]: Add new upgrade handler that runs migrations. Add a couple 1.12.x TODO entries. * [995]: Remove unused deprecated Conditions field. * [995]: Replace deprecated use of strings.Title. * [995]: Replace use of a couple deprecated gov cli flags, and just make local versions of them to use instead. * [995]: Correct the changelog entry about the DenomOwners query. * [995]: Undo a comment change that will be handy during a followup. * [995]: Delete a couple missed references to sims-state-listening.mk. * [1010]: Replace use of the deprecated sdkResponse.Data field. The documentation says it's still usable, but something changed because it made the sim tests all start failing due to index out of range (asking for 0, but no entries). That indicates that stuff was missing from the Data field, and so we can't use it anymore. * [995]: Fix the runsim sims. I accidentally got rid of the -tags part that allows the other dbs to be used. Also simplified it further since there doesn't need to be differences between the runsim and go test versions. * [1010]: Tweak the .Data fix to use the GetCachedValue off the response rather than reunmarshaling it from the bytes. * [995]: Add unit tests on the ExportAppStateAndValidators function. * [995]: Fix the casting of the MsgInstantiateContractResponse. * [995]: Update the TestAppImportExport simulation to bring it in line with it's cosmos-sdk counterpart. Fail if the db can't close. Handle the panic when there are no validators. Provide some keys to the authzkeeper checking. * [995]: Fix a couple marker tests that started failing because I fixed the account numbers thing. * [995]: Bump the sdk to v0.46.0-pio-1-rc3 (from v0.46.0-pio-1-rc2). * [995]: Bump cosmos-sdk to v0.46.0-pio-1-rc4 (from v0.46.0-pio-1-rc3). * [995]: Add ochre-rc1 upgrade handler. * [995]: Fix the changelog so that this new stuff isn't listed in the 1.12 section. * [995]: Update the TODO: Required comments to reference v1.13.x (instead of v1.12.x). * [995]: Fix the name of the TestHandlerTestSuite (was TestAnteTestSuite, which is a different thing). * [995]: Undo a change to the msg_fee_invoker that I made at some point, but now don't want since it's in an area to be fixed anyway. * Updated to remove deprecated sdk.Int. (#1036) * [995]: Disable the sims for the same reason I needed to disable some of the msg service router tests. Added TODO comments about removing the skips too. * [995]: Add issue links to the TODOs needed to be todone with issue 1006 (the fees stuff). * [995]: Add issue links to the other new TODOs. * [995]: Bump cosmos to v0.46.0-pio-1-rc5 (from v0.46.0-pio-1-rc4). * Replaced deprecated SetFullFundraiserPath with SetCoinType and SetPurpose (#1037) * Removed SetFullFundraiserPath and replaced it with SetCoinType and SetPurpose. * Added prefix_test and updated SetConfig to have a configurable seal param. This was mainly to help with testing. * [995]: Lint fixes. * [1007]: Add the groups module to the app. Remove the mango-rc4 upgrade since testnet was just upgraded to neoncarrot-rc1. Clean up some of the upgrades stuff too, and add a log message if at a skip height. In the upgrades map, move the version comment to the same line as the upgrade name because that's the assuciation we usually want. * [995]: Add some lookups to the TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations test for some extra debug info. * [995]: Fix import order. * [995]: Refactor TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations a bit. Make it so that the votes are coming from an account that isn't a validator. * [995]: Remove the nickel upgrade handler and combine it with my ochre one. * [995]: Add missing comment about how the tests are initiated. * [995]: Lint fixes. * [1007]: Remove the mango upgrade (since both mainnet and testnet are on neoncarrot. * [1007]: Remove TODO that is TODONE. Co-authored-by: Matt Witkowski <[email protected]>
SpicyLemon
added a commit
that referenced
this pull request
Sep 22, 2022
* [995]: Bump cosmos to v0.46.0-pio-1-rc1 (from v0.45.5-pio-1). * [995]: Add changelog line. * [995]: Remove call to authrest.RegisterTxRoutes since it was deprecated and now no longer exists. * [995]: Replace import of github.com/cosmos/cosmos-sdk/x/auth/legacy/legacytx with github.com/cosmos/cosmos-sdk/x/auth/migrations/legacytx since it was moved. * [995]: Remove imports of github.com/cosmos/cosmos-sdk/client/rest which was deprecated and no longer exists. Also remove all the RegisterRESTRoutes functions and calls. * [995]: Update the name client's call to govclient.NewProposalHandler and delete the name client's rest.go since nothing in there is used anymore and github.com/cosmos/cosmos-sdk/x/gov/client/rest has been removed. * [995]: Remove all uses of github.com/cosmos/cosmos-sdk/types/rest which was deprecated, and has now been removed. * [995]: Update ibc-go to v5.0.0-beta1 since that's the first one that supports cosmos v0.46. * [995]: Remove google.golang.org/grpc v1.33.2 replace line. It was added when we bumped cosmos to v0.41.0 (from v0.40.1). But now some stuff in the newer version (v1.48.0) is needed by cosmos. * [995]: Update the cosmos-sdk version in the require section (even though it's being replaced below). * [995]: Add a replace line for wasmd to use a local verison. There's a draft PR to update wasmd to v0.46: CosmWasm/wasmd#930 That's what I've got locally and will be referencing. Not yet sure what I'll do if I need to push this up before they've got a tagged version. * [995]: go mod tidy (finally, it works). * [995]: Add v0.46 to the TODO in go.mod to help identify it as something that needs to be done as part of the v0.46 update. * [995]: Fix compliation issues in app.go. * [995]: Fix compilation issues in root.go. * [995]: Switch to using storetypes.StoreKey (from sdk.StoreKey). * [995]: Fix uses of coins.SafeSub since it's now vararg. * [995]: Fix the gov stuff that moved to v1beta1. * [995]: Add the FeeGasMeter.GasRemaining function that was added to the interface. * [995]: Update the antewrapper handler to use a couple new functions that changed. * [995]: Fix a couple more govtypes moves. * [995]: change a coins.Sub call to use varargs. * [995]: Provide the expiration time as a reference to authz.NewMsgGrant since they seem to have changed it to a reference. * [995]: Fix uses of the renamed GenSignedMockTx (was GenTx) and SimDeliver (was Deliver) sim functions. * [995]: update call to AddressAndDenomFromBalancesStore (used to be AddressFromBalancesStore). * [995]: Fix call to GetPubKey that now also returns an error (instead of panic). * [995]: Switch to a local version of cosmos so I can test the authz fix. * [995]: Switch to GetAuthorization (from GetCleanAuthorization). * [995]: Replace the rest of the GenTx calls with GenSignedMockTx. * [995]: Fix uses of simapp.FundAccount and standardize all of them to just use the bank module's testutil.FundAccount. * [995]: Fix the rest of the uses of SimDeliver (was previously named Deliver). * [995]: Remove a bunch of test helpers that aren't used but are available from the SDK. * [995]: Delete our version of EmptyAppOptions which was just a copy of the cosmos version, and switch to using the cosmos version. * [995]: Standardize the import of github.com/cosmos/cosmos-sdk/simapp to be aliased to sdksim. * [995]: Fix uses of AccAddressFromHexUnsafe (used to be named AccAddressFromHex). * [995]: Fix uses of testnet.New. * [995]: Remove the use of MaxWasmCodeSize and set the MaxWasmSize instead. * [995]: Fix simulation call to wasm.NewAppModule, which requires some new arguments. * [995]: Remove use of UpgradeInfo and replace it with Plan. * [995]: Switch to .Codec (from JSONCodec) since the latter was removed from the context and we're supposed to use .Codec now. * [995]: Provide a codec to all calls to kerying.New. * [995]: Update uses of keyring.Record.GetAddress to account for the additional error return value. * [995]: gov module's GenesisState is now in v1beta1 and the GenerateSaveCoinKey moved to testutil. * [995]: switch to keyring.Record (used be called Info). Add handling of the error return value from GetAddress (newly added). * [995]: account for the added error return value from GetPubKey. Call BuildUnsignedTx directly from the factory since it was moved there. * [995]: gov module's GenesisState was moved to v1beta1. * [995]: Fix several complaints about the receivers in the dbmigrator migrator tests. * [995]: Fix several complaints about the ReflectorTestSuit method receivers having a lock but being concrete. * [995]: Correct uses of the encoding config's Codec (used to be called Marshaller). * [995]: Update a use of NewDeductFeeDecorator (used to be called NewMempoolFeeDecorator). * [995]: Fix some complaints about variable names conflicting with import names. * [995]: Comment out a couple more uses of SetMsgServiceRouter (with a TODO above them). Fix the expiration arg in all calls to SaveGrant and replace the last few uses of GetCleanAuthorization. * [995]: Switch to pruning/types for the pruning options (was moved from store/types). * [995]: Make the marker HandlerTestSuite methods have a pointer receiver since it does some lock stuff and Intellij complains. * [995]: The InitialStake field now requires an sdk Int. * [995]: The sdksim.Setup function now also takes in the Testing variable. * [995]: The NewTextProposal function moved to gov types v1beta1. * [995]: Use the SetMsgServiceRouter again. * [995]: Fix some errorcheck lint complaints. The functions that it's now complaining about all are hard-coded to return nil. I thought about just putting a nolint comment on them stating such, but eh, this protects us from future changes to those SDK functions. * [995]: The clientCtx.Codec returns an interface now, so there's no need to type-assert/convert it anymore. This fixes some gocritic linter complaints. * [995]: Tweak the makefile a bit. Don't output errors from go mod list when getting the TM_VERSION; those will be output later if that value is needed. Echo the commands from several targets that used to have them suppressed; just makes it easier to troubleshoot. Use the GO variable for all go executions. Do a go.sum before running tests. * [995]: Rip out all the stuff added to facilitate having a kafka dependency. * [995]: Switch to cosmos version v0.46.0-pio-1-rc2. * [995]: Update app/test_helpers.go to bring it in line with Cosmos' simapp/test_helpers.go. Also stop using DefaultNodeHome for unit tests. * [995]: Put the marker authz_test in the types_test package due to circular import cycle with /app. * [995]: Bring prepForZeroHeightGenesis inline with the updated simapp version and clean up the TestSimAppExportAndBlockedAddrs to be closer to the cosmos version too (and use the new stuff). * [995]: Remove the config index-events workaround since it was fixed in v0.46. * [995]: Tweak some fee antewrapper errors to make them match Cosmos' similar ones. Also, the addresses were being converted to hex instead of bech32. So this is better now. * [995]: Fix the TestEnsureMempoolFees test because Cosmos now checks for fund availability. Also clean up the antewrapper tests so that they only run once (they were running thrice). And update the TestDeductFeesNoDelegation to be more robust and make sure we're getting the expectd errors instead of just any error. * [995]: Segment TestMsgService since it's failing and need to slog through it. Bump the gas used in there since it's no longer enough again, and remove the check function and all it's uses, replaced with require NoError. * [995]: Fix unit test that was failing because TextProposal moved to v1beta1 (from types) and is in an error string check. * [995]: In the metadata cli test, set the suite's config value before generating the keyring stuff so that it can use the config as intended (not sure how it didn't panic before). Also change a couple expected error strings because some errors changed. * [995]: Fix the marker cli tests. Need more gas for the gov proposals. Need to set the suite's cfg earlier (same as what metadate cli tests did). And now expect a 0x9 instead of 0x5 from the withdraw proposal. * [995]: Fix the marker handler test, invalid proposal since the package changed for TextProposal (to v1beta1 from types). * [995]: Correct an incorrectly updated comment (that might be useful when fixing stuff in a bit). * [995]: Fix the marker operations test that changed due to added use of the random number generator by the SDK. Also added a note about that and a list of valid options that things might change to in the future. But I left it looking for one specific name since that shouldn't change unless making changes to the SDK or operations. * [995]: Fix the Holding query since the data is stored differently now. key is <prefix><account><denom>, value is <amount>. Used to be key is <prefix><account>, value is <coin>. * [995]: Refactor the marker Holding query to use the sdk's DenomOwners query (should have better performance). * [995]: lint fixes. * [995]: Undo a lot of the formatting/division changes that I had made to TestMsgService since we're going to fix that in a separte PR. * [995]: Skip several MsgService tests, to be fixed in a separate PR. * [995]: Switch the TODO comment in msg_fees_decorator to reference that it's required for 1.12. * [995]: Enhance the changelog with more info about what's being done here. * [995]: Switch wasm to a version tagged on our own fork so that we can keep moving forward. * [995]: Add new upgrade handler that runs migrations. Add a couple 1.12.x TODO entries. * [995]: Remove unused deprecated Conditions field. * [995]: Replace deprecated use of strings.Title. * [995]: Replace use of a couple deprecated gov cli flags, and just make local versions of them to use instead. * [995]: Correct the changelog entry about the DenomOwners query. * [995]: Undo a comment change that will be handy during a followup. * [995]: Delete a couple missed references to sims-state-listening.mk. * [1010]: Replace use of the deprecated sdkResponse.Data field. The documentation says it's still usable, but something changed because it made the sim tests all start failing due to index out of range (asking for 0, but no entries). That indicates that stuff was missing from the Data field, and so we can't use it anymore. * [995]: Fix the runsim sims. I accidentally got rid of the -tags part that allows the other dbs to be used. Also simplified it further since there doesn't need to be differences between the runsim and go test versions. * [1010]: Tweak the .Data fix to use the GetCachedValue off the response rather than reunmarshaling it from the bytes. * [995]: Add unit tests on the ExportAppStateAndValidators function. * [995]: Fix the casting of the MsgInstantiateContractResponse. * [995]: Update the TestAppImportExport simulation to bring it in line with it's cosmos-sdk counterpart. Fail if the db can't close. Handle the panic when there are no validators. Provide some keys to the authzkeeper checking. * [995]: Fix a couple marker tests that started failing because I fixed the account numbers thing. * [995]: Bump the sdk to v0.46.0-pio-1-rc3 (from v0.46.0-pio-1-rc2). * [1017]: Delete the third part proto stuff that gets updated by the script. * [1017]: Run make proto-update-deps. * [1017]: Copy freshly generated swagger from our cosmos fork. * [1017]: Properly quote things. Delete the directories right before we download the new stuff for them. Get the cosmos version of cosmos-proto instead of regen's. Move the proofs.proto head command into the rest of the creation of that file so there's just a single write of it. * [1017]: Provide the full desired destination path to the update deps script. * [1017]: Refactor the proto-update-check.sh script. Exit early when it's identified that there aren't any changes. Extract from the update-deps script, the list of libraries to that might have changes of interest. Differences will only be output once. Library differences will be listed (if checked and there are any). Pay attention to a FORCE env var (same as providing the --force flag, but can be done using the make target). * [1017]: Get the updated version of cosmos_proto/cosmos.proto and move the proofs.proto into the confio dir. * [1017]: Allow for providing the branch to compare to in the proto-update-check script and do so with the github action. * [1017]: make proto-swagger-gen. * [1017]: Add changelog entry. * [1017]: Add the ..origin/ to the github base ref for the proto-update-check. * [1017]: Just use a space delimited string instead of an array for the libs since the github actions shell doesn't support arrays (maybe?). * [1017]: Output branch info since it doesn't seem to be working right. * [1017]: Rename the BRANCH env var to BASE_BRANCH since github seems to set BRANCH to HEAD. * [1017]: Add a comment after the tendermint entry in go.mod to trick the third-party check into downloading and comparing. * [1017]: Output the breaking results in the github action. * [1017]: Try again with the outputting of failure results. * [1017]: It still didn't run the third-party breaking results step, so try harder. * [1017]: Okay. it's running the results step now, but it didn't print anything. * [1017]: Bump the buf-breaking action to 1.1.0 (from 1) because 1 doesn't have the results yet. * [1017]: One more tweak to try to get the results in the action instead of only as annotations. * [1017]: one more try (I lied before) and also bump the lint action to 1.0.1 (from 1). * [1017]: Add missing colon to output that doesn't seem to be there. * [1017]: Try the toJSON function. * [1017]: that at least printed something, but the outputs was listed as {}. So maybe it's lazy laaded and needs to be specified. * [1017]: Remove temporary go.mod change (for testing github actions). * [1017]: Add a make target for proto-check-breaking-third-party. * [1017]: Move the proofs.proto file back to the root (out of the confio dir) since the tendermint files that use it can't find it in the confio dir. Also, when updating deps, when downloading a single proto file, only delete that specific file. * [1017]: When updating deps, grab the now-deleted ibc core v1 query proto too. * [995]: Bump cosmos-sdk to v0.46.0-pio-1-rc4 (from v0.46.0-pio-1-rc3). * [995]: Add ochre-rc1 upgrade handler. * [995]: Fix the changelog so that this new stuff isn't listed in the 1.12 section. * [995]: Update the TODO: Required comments to reference v1.13.x (instead of v1.12.x). * [995]: Fix the name of the TestHandlerTestSuite (was TestAnteTestSuite, which is a different thing). * [995]: Undo a change to the msg_fee_invoker that I made at some point, but now don't want since it's in an area to be fixed anyway. * [1017]: See if I can have the buf breaking gh action checks run against the PRs target branch. * Updated to remove deprecated sdk.Int. (#1036) * [1017]: The proto gh action only runs on PRs, so it's safe to run the checks agains the target branch. So try that. If we ever make it run on push also, we'll need something else to figure out what to check against. * [1017]: output the base branch in the setup step. Also provide the base branch to make proto-update-check. * [1017]: Update statik from updated swagger files. * [995]: Disable the sims for the same reason I needed to disable some of the msg service router tests. Added TODO comments about removing the skips too. * [995]: Add issue links to the TODOs needed to be todone with issue 1006 (the fees stuff). * [995]: Add issue links to the other new TODOs. * [995]: Bump cosmos to v0.46.0-pio-1-rc5 (from v0.46.0-pio-1-rc4). * Replaced deprecated SetFullFundraiserPath with SetCoinType and SetPurpose (#1037) * Removed SetFullFundraiserPath and replaced it with SetCoinType and SetPurpose. * Added prefix_test and updated SetConfig to have a configurable seal param. This was mainly to help with testing. * [995]: Lint fixes. * [1017]: Update version string to 1.13.0 since we're way beyond 1.3.1 anymore. * [1017]: Regenerate swagger stuff to get the version string change. * [995]: Add some lookups to the TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations test for some extra debug info. * [995]: Fix import order. * [995]: Refactor TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations a bit. Make it so that the votes are coming from an account that isn't a validator. * [995]: Remove the nickel upgrade handler and combine it with my ochre one. * [995]: Add missing comment about how the tests are initiated. * [995]: Lint fixes. * [1017]: Regenerate swagger docs after merge. * [995]: Fix the TestFullBIP44Path test to actually test what we're trying to test. * [995]: Use the FundModuleAccount function instead of minting and sending. * [995]: Bump Cosmos-SDK to v0.46.1-pio-1-rc1 (from v0.46.0-pio-1-rc5). It's exactly the same, just named more accurately. * [995]: lazy-load the govVoteMsgUrls messages, just to be safe. * [995]: Update the isGovernanceMessage func to account for the new version of governance messages. * [995]: Fix the gov msg count. * [995]: Lint fixes. I really hate that the Go convention for acronyms in variable names is to capitalize all or none of them. * [995]: Switch msgURL setting logic to just look for an empty slice instead of slice of specific length. Makes it a little more future-safe. * [995]: Switch those checks to a nil check and add a comment about it. * [1017]: Add the force flag to the github action third-party proto check. * [1017]: Update the proto-update-check.sh script as follows: 1) Fix argument processing to not be an infinate loop. 2) When skipping the check because no changes were found in go.mod, output a message about it and how to bypass it. 3) Do a superficial diff of go.mod to skip out even earlier if there aren't any go.mod changes at all. 4) Add argument/env vars to suggested commands. * [1017]: make proto-update-deps. * [1017]: Remove the temporary FORCE=1 from the proto github action. Co-authored-by: Matt Witkowski <[email protected]>
Taztingo
added a commit
that referenced
this pull request
Sep 28, 2022
* [995]: Bump cosmos to v0.46.0-pio-1-rc1 (from v0.45.5-pio-1). * [995]: Add changelog line. * [995]: Remove call to authrest.RegisterTxRoutes since it was deprecated and now no longer exists. * [995]: Replace import of github.com/cosmos/cosmos-sdk/x/auth/legacy/legacytx with github.com/cosmos/cosmos-sdk/x/auth/migrations/legacytx since it was moved. * [995]: Remove imports of github.com/cosmos/cosmos-sdk/client/rest which was deprecated and no longer exists. Also remove all the RegisterRESTRoutes functions and calls. * [995]: Update the name client's call to govclient.NewProposalHandler and delete the name client's rest.go since nothing in there is used anymore and github.com/cosmos/cosmos-sdk/x/gov/client/rest has been removed. * [995]: Remove all uses of github.com/cosmos/cosmos-sdk/types/rest which was deprecated, and has now been removed. * [995]: Update ibc-go to v5.0.0-beta1 since that's the first one that supports cosmos v0.46. * [995]: Remove google.golang.org/grpc v1.33.2 replace line. It was added when we bumped cosmos to v0.41.0 (from v0.40.1). But now some stuff in the newer version (v1.48.0) is needed by cosmos. * [995]: Update the cosmos-sdk version in the require section (even though it's being replaced below). * [995]: Add a replace line for wasmd to use a local verison. There's a draft PR to update wasmd to v0.46: CosmWasm/wasmd#930 That's what I've got locally and will be referencing. Not yet sure what I'll do if I need to push this up before they've got a tagged version. * [995]: go mod tidy (finally, it works). * [995]: Add v0.46 to the TODO in go.mod to help identify it as something that needs to be done as part of the v0.46 update. * [995]: Fix compliation issues in app.go. * [995]: Fix compilation issues in root.go. * [995]: Switch to using storetypes.StoreKey (from sdk.StoreKey). * [995]: Fix uses of coins.SafeSub since it's now vararg. * [995]: Fix the gov stuff that moved to v1beta1. * [995]: Add the FeeGasMeter.GasRemaining function that was added to the interface. * [995]: Update the antewrapper handler to use a couple new functions that changed. * [995]: Fix a couple more govtypes moves. * [995]: change a coins.Sub call to use varargs. * [995]: Provide the expiration time as a reference to authz.NewMsgGrant since they seem to have changed it to a reference. * [995]: Fix uses of the renamed GenSignedMockTx (was GenTx) and SimDeliver (was Deliver) sim functions. * [995]: update call to AddressAndDenomFromBalancesStore (used to be AddressFromBalancesStore). * [995]: Fix call to GetPubKey that now also returns an error (instead of panic). * [995]: Switch to a local version of cosmos so I can test the authz fix. * [995]: Switch to GetAuthorization (from GetCleanAuthorization). * [995]: Replace the rest of the GenTx calls with GenSignedMockTx. * [995]: Fix uses of simapp.FundAccount and standardize all of them to just use the bank module's testutil.FundAccount. * [995]: Fix the rest of the uses of SimDeliver (was previously named Deliver). * [995]: Remove a bunch of test helpers that aren't used but are available from the SDK. * [995]: Delete our version of EmptyAppOptions which was just a copy of the cosmos version, and switch to using the cosmos version. * [995]: Standardize the import of github.com/cosmos/cosmos-sdk/simapp to be aliased to sdksim. * [995]: Fix uses of AccAddressFromHexUnsafe (used to be named AccAddressFromHex). * [995]: Fix uses of testnet.New. * [995]: Remove the use of MaxWasmCodeSize and set the MaxWasmSize instead. * [995]: Fix simulation call to wasm.NewAppModule, which requires some new arguments. * [995]: Remove use of UpgradeInfo and replace it with Plan. * [995]: Switch to .Codec (from JSONCodec) since the latter was removed from the context and we're supposed to use .Codec now. * [995]: Provide a codec to all calls to kerying.New. * [995]: Update uses of keyring.Record.GetAddress to account for the additional error return value. * [995]: gov module's GenesisState is now in v1beta1 and the GenerateSaveCoinKey moved to testutil. * [995]: switch to keyring.Record (used be called Info). Add handling of the error return value from GetAddress (newly added). * [995]: account for the added error return value from GetPubKey. Call BuildUnsignedTx directly from the factory since it was moved there. * [995]: gov module's GenesisState was moved to v1beta1. * [995]: Fix several complaints about the receivers in the dbmigrator migrator tests. * [995]: Fix several complaints about the ReflectorTestSuit method receivers having a lock but being concrete. * [995]: Correct uses of the encoding config's Codec (used to be called Marshaller). * [995]: Update a use of NewDeductFeeDecorator (used to be called NewMempoolFeeDecorator). * [995]: Fix some complaints about variable names conflicting with import names. * [995]: Comment out a couple more uses of SetMsgServiceRouter (with a TODO above them). Fix the expiration arg in all calls to SaveGrant and replace the last few uses of GetCleanAuthorization. * [995]: Switch to pruning/types for the pruning options (was moved from store/types). * [995]: Make the marker HandlerTestSuite methods have a pointer receiver since it does some lock stuff and Intellij complains. * [995]: The InitialStake field now requires an sdk Int. * [995]: The sdksim.Setup function now also takes in the Testing variable. * [995]: The NewTextProposal function moved to gov types v1beta1. * [995]: Use the SetMsgServiceRouter again. * [995]: Fix some errorcheck lint complaints. The functions that it's now complaining about all are hard-coded to return nil. I thought about just putting a nolint comment on them stating such, but eh, this protects us from future changes to those SDK functions. * [995]: The clientCtx.Codec returns an interface now, so there's no need to type-assert/convert it anymore. This fixes some gocritic linter complaints. * [995]: Tweak the makefile a bit. Don't output errors from go mod list when getting the TM_VERSION; those will be output later if that value is needed. Echo the commands from several targets that used to have them suppressed; just makes it easier to troubleshoot. Use the GO variable for all go executions. Do a go.sum before running tests. * [995]: Rip out all the stuff added to facilitate having a kafka dependency. * [995]: Switch to cosmos version v0.46.0-pio-1-rc2. * [995]: Update app/test_helpers.go to bring it in line with Cosmos' simapp/test_helpers.go. Also stop using DefaultNodeHome for unit tests. * [995]: Put the marker authz_test in the types_test package due to circular import cycle with /app. * [995]: Bring prepForZeroHeightGenesis inline with the updated simapp version and clean up the TestSimAppExportAndBlockedAddrs to be closer to the cosmos version too (and use the new stuff). * [995]: Remove the config index-events workaround since it was fixed in v0.46. * [995]: Tweak some fee antewrapper errors to make them match Cosmos' similar ones. Also, the addresses were being converted to hex instead of bech32. So this is better now. * [995]: Fix the TestEnsureMempoolFees test because Cosmos now checks for fund availability. Also clean up the antewrapper tests so that they only run once (they were running thrice). And update the TestDeductFeesNoDelegation to be more robust and make sure we're getting the expectd errors instead of just any error. * [995]: Segment TestMsgService since it's failing and need to slog through it. Bump the gas used in there since it's no longer enough again, and remove the check function and all it's uses, replaced with require NoError. * [995]: Fix unit test that was failing because TextProposal moved to v1beta1 (from types) and is in an error string check. * [995]: In the metadata cli test, set the suite's config value before generating the keyring stuff so that it can use the config as intended (not sure how it didn't panic before). Also change a couple expected error strings because some errors changed. * [995]: Fix the marker cli tests. Need more gas for the gov proposals. Need to set the suite's cfg earlier (same as what metadate cli tests did). And now expect a 0x9 instead of 0x5 from the withdraw proposal. * [995]: Fix the marker handler test, invalid proposal since the package changed for TextProposal (to v1beta1 from types). * [995]: Correct an incorrectly updated comment (that might be useful when fixing stuff in a bit). * [995]: Fix the marker operations test that changed due to added use of the random number generator by the SDK. Also added a note about that and a list of valid options that things might change to in the future. But I left it looking for one specific name since that shouldn't change unless making changes to the SDK or operations. * [995]: Fix the Holding query since the data is stored differently now. key is <prefix><account><denom>, value is <amount>. Used to be key is <prefix><account>, value is <coin>. * [995]: Refactor the marker Holding query to use the sdk's DenomOwners query (should have better performance). * [995]: lint fixes. * [995]: Undo a lot of the formatting/division changes that I had made to TestMsgService since we're going to fix that in a separte PR. * [995]: Skip several MsgService tests, to be fixed in a separate PR. * [995]: Switch the TODO comment in msg_fees_decorator to reference that it's required for 1.12. * [995]: Enhance the changelog with more info about what's being done here. * [995]: Switch wasm to a version tagged on our own fork so that we can keep moving forward. * [995]: Add new upgrade handler that runs migrations. Add a couple 1.12.x TODO entries. * [995]: Remove unused deprecated Conditions field. * [995]: Replace deprecated use of strings.Title. * [995]: Replace use of a couple deprecated gov cli flags, and just make local versions of them to use instead. * [995]: Correct the changelog entry about the DenomOwners query. * [995]: Undo a comment change that will be handy during a followup. * [995]: Delete a couple missed references to sims-state-listening.mk. * [1010]: Replace use of the deprecated sdkResponse.Data field. The documentation says it's still usable, but something changed because it made the sim tests all start failing due to index out of range (asking for 0, but no entries). That indicates that stuff was missing from the Data field, and so we can't use it anymore. * [995]: Fix the runsim sims. I accidentally got rid of the -tags part that allows the other dbs to be used. Also simplified it further since there doesn't need to be differences between the runsim and go test versions. * Added ibcv2->ibcv3 migration. * [1010]: Tweak the .Data fix to use the GetCachedValue off the response rather than reunmarshaling it from the bytes. * Renamed to UpgradeICA because we are configuring ICA. IBC should upgrade automatically from RunMigrations. * Updated app.go to include the ica module. * Disabled ica controller capabilities until we have a use case for it. An auth module will need to be created if we want it. * [995]: Add unit tests on the ExportAppStateAndValidators function. * [995]: Fix the casting of the MsgInstantiateContractResponse. * [995]: Update the TestAppImportExport simulation to bring it in line with it's cosmos-sdk counterpart. Fail if the db can't close. Handle the panic when there are no validators. Provide some keys to the authzkeeper checking. * [995]: Fix a couple marker tests that started failing because I fixed the account numbers thing. * [995]: Bump the sdk to v0.46.0-pio-1-rc3 (from v0.46.0-pio-1-rc2). * Added ica module to migrations. Updated to use our own tag until they add our fix. * Updated function name to make it private, fixed logging statements. * Fixed typo that was accidentally committed. * Added test to verify ica initialization function. * Added setup before test. * [995]: Bump cosmos-sdk to v0.46.0-pio-1-rc4 (from v0.46.0-pio-1-rc3). * [995]: Add ochre-rc1 upgrade handler. * [995]: Fix the changelog so that this new stuff isn't listed in the 1.12 section. * [995]: Update the TODO: Required comments to reference v1.13.x (instead of v1.12.x). * [995]: Fix the name of the TestHandlerTestSuite (was TestAnteTestSuite, which is a different thing). * [995]: Undo a change to the msg_fee_invoker that I made at some point, but now don't want since it's in an area to be fixed anyway. * Updated to remove deprecated sdk.Int. (#1036) * [995]: Disable the sims for the same reason I needed to disable some of the msg service router tests. Added TODO comments about removing the skips too. * [995]: Add issue links to the TODOs needed to be todone with issue 1006 (the fees stuff). * [995]: Add issue links to the other new TODOs. * [995]: Bump cosmos to v0.46.0-pio-1-rc5 (from v0.46.0-pio-1-rc4). * Replaced deprecated SetFullFundraiserPath with SetCoinType and SetPurpose (#1037) * Removed SetFullFundraiserPath and replaced it with SetCoinType and SetPurpose. * Added prefix_test and updated SetConfig to have a configurable seal param. This was mainly to help with testing. * Updated dependencies to point to official tag that included our change. * [995]: Lint fixes. * Fixed missing initialization of scope keeper. * [995]: Add some lookups to the TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations test for some extra debug info. * [995]: Fix import order. * [995]: Refactor TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations a bit. Make it so that the votes are coming from an account that isn't a validator. * [995]: Remove the nickel upgrade handler and combine it with my ochre one. * [995]: Add missing comment about how the tests are initiated. * [995]: Lint fixes. * Added fix for ibc client proposal. It had the wrong key. * Updated changelog. * Moved ibc 5.0.0 to new features section. * Updated ibc-go to rc-2. * Added back group module. Was accidentally deleted during merge. * Forgot to save changes from go.mod * Fixed comment that was accidentally added back in from merge. * Updated to v5.0.0 * Updated changelog. * Updated the ICA Host Keeper to include an argument for the new parameter. * Updated protos. Co-authored-by: Daniel Wedul <[email protected]> Co-authored-by: Ira Miller <[email protected]>
SpicyLemon
added a commit
that referenced
this pull request
Sep 28, 2022
* [995]: Bump cosmos to v0.46.0-pio-1-rc1 (from v0.45.5-pio-1). * [995]: Add changelog line. * [995]: Remove call to authrest.RegisterTxRoutes since it was deprecated and now no longer exists. * [995]: Replace import of github.com/cosmos/cosmos-sdk/x/auth/legacy/legacytx with github.com/cosmos/cosmos-sdk/x/auth/migrations/legacytx since it was moved. * [995]: Remove imports of github.com/cosmos/cosmos-sdk/client/rest which was deprecated and no longer exists. Also remove all the RegisterRESTRoutes functions and calls. * [995]: Update the name client's call to govclient.NewProposalHandler and delete the name client's rest.go since nothing in there is used anymore and github.com/cosmos/cosmos-sdk/x/gov/client/rest has been removed. * [995]: Remove all uses of github.com/cosmos/cosmos-sdk/types/rest which was deprecated, and has now been removed. * [995]: Update ibc-go to v5.0.0-beta1 since that's the first one that supports cosmos v0.46. * [995]: Remove google.golang.org/grpc v1.33.2 replace line. It was added when we bumped cosmos to v0.41.0 (from v0.40.1). But now some stuff in the newer version (v1.48.0) is needed by cosmos. * [995]: Update the cosmos-sdk version in the require section (even though it's being replaced below). * [995]: Add a replace line for wasmd to use a local verison. There's a draft PR to update wasmd to v0.46: CosmWasm/wasmd#930 That's what I've got locally and will be referencing. Not yet sure what I'll do if I need to push this up before they've got a tagged version. * [995]: go mod tidy (finally, it works). * [995]: Add v0.46 to the TODO in go.mod to help identify it as something that needs to be done as part of the v0.46 update. * [995]: Fix compliation issues in app.go. * [995]: Fix compilation issues in root.go. * [995]: Switch to using storetypes.StoreKey (from sdk.StoreKey). * [995]: Fix uses of coins.SafeSub since it's now vararg. * [995]: Fix the gov stuff that moved to v1beta1. * [995]: Add the FeeGasMeter.GasRemaining function that was added to the interface. * [995]: Update the antewrapper handler to use a couple new functions that changed. * [995]: Fix a couple more govtypes moves. * [995]: change a coins.Sub call to use varargs. * [995]: Provide the expiration time as a reference to authz.NewMsgGrant since they seem to have changed it to a reference. * [995]: Fix uses of the renamed GenSignedMockTx (was GenTx) and SimDeliver (was Deliver) sim functions. * [995]: update call to AddressAndDenomFromBalancesStore (used to be AddressFromBalancesStore). * [995]: Fix call to GetPubKey that now also returns an error (instead of panic). * [995]: Switch to a local version of cosmos so I can test the authz fix. * [995]: Switch to GetAuthorization (from GetCleanAuthorization). * [995]: Replace the rest of the GenTx calls with GenSignedMockTx. * [995]: Fix uses of simapp.FundAccount and standardize all of them to just use the bank module's testutil.FundAccount. * [995]: Fix the rest of the uses of SimDeliver (was previously named Deliver). * [995]: Remove a bunch of test helpers that aren't used but are available from the SDK. * [995]: Delete our version of EmptyAppOptions which was just a copy of the cosmos version, and switch to using the cosmos version. * [995]: Standardize the import of github.com/cosmos/cosmos-sdk/simapp to be aliased to sdksim. * [995]: Fix uses of AccAddressFromHexUnsafe (used to be named AccAddressFromHex). * [995]: Fix uses of testnet.New. * [995]: Remove the use of MaxWasmCodeSize and set the MaxWasmSize instead. * [995]: Fix simulation call to wasm.NewAppModule, which requires some new arguments. * [995]: Remove use of UpgradeInfo and replace it with Plan. * [995]: Switch to .Codec (from JSONCodec) since the latter was removed from the context and we're supposed to use .Codec now. * [995]: Provide a codec to all calls to kerying.New. * [995]: Update uses of keyring.Record.GetAddress to account for the additional error return value. * [995]: gov module's GenesisState is now in v1beta1 and the GenerateSaveCoinKey moved to testutil. * [995]: switch to keyring.Record (used be called Info). Add handling of the error return value from GetAddress (newly added). * [995]: account for the added error return value from GetPubKey. Call BuildUnsignedTx directly from the factory since it was moved there. * [995]: gov module's GenesisState was moved to v1beta1. * [995]: Fix several complaints about the receivers in the dbmigrator migrator tests. * [995]: Fix several complaints about the ReflectorTestSuit method receivers having a lock but being concrete. * [995]: Correct uses of the encoding config's Codec (used to be called Marshaller). * [995]: Update a use of NewDeductFeeDecorator (used to be called NewMempoolFeeDecorator). * [995]: Fix some complaints about variable names conflicting with import names. * [995]: Comment out a couple more uses of SetMsgServiceRouter (with a TODO above them). Fix the expiration arg in all calls to SaveGrant and replace the last few uses of GetCleanAuthorization. * [995]: Switch to pruning/types for the pruning options (was moved from store/types). * [995]: Make the marker HandlerTestSuite methods have a pointer receiver since it does some lock stuff and Intellij complains. * [995]: The InitialStake field now requires an sdk Int. * [995]: The sdksim.Setup function now also takes in the Testing variable. * [995]: The NewTextProposal function moved to gov types v1beta1. * [995]: Use the SetMsgServiceRouter again. * [995]: Fix some errorcheck lint complaints. The functions that it's now complaining about all are hard-coded to return nil. I thought about just putting a nolint comment on them stating such, but eh, this protects us from future changes to those SDK functions. * [995]: The clientCtx.Codec returns an interface now, so there's no need to type-assert/convert it anymore. This fixes some gocritic linter complaints. * [995]: Tweak the makefile a bit. Don't output errors from go mod list when getting the TM_VERSION; those will be output later if that value is needed. Echo the commands from several targets that used to have them suppressed; just makes it easier to troubleshoot. Use the GO variable for all go executions. Do a go.sum before running tests. * [995]: Rip out all the stuff added to facilitate having a kafka dependency. * [995]: Switch to cosmos version v0.46.0-pio-1-rc2. * [995]: Update app/test_helpers.go to bring it in line with Cosmos' simapp/test_helpers.go. Also stop using DefaultNodeHome for unit tests. * [995]: Put the marker authz_test in the types_test package due to circular import cycle with /app. * [995]: Bring prepForZeroHeightGenesis inline with the updated simapp version and clean up the TestSimAppExportAndBlockedAddrs to be closer to the cosmos version too (and use the new stuff). * [995]: Remove the config index-events workaround since it was fixed in v0.46. * [995]: Tweak some fee antewrapper errors to make them match Cosmos' similar ones. Also, the addresses were being converted to hex instead of bech32. So this is better now. * [995]: Fix the TestEnsureMempoolFees test because Cosmos now checks for fund availability. Also clean up the antewrapper tests so that they only run once (they were running thrice). And update the TestDeductFeesNoDelegation to be more robust and make sure we're getting the expectd errors instead of just any error. * [995]: Segment TestMsgService since it's failing and need to slog through it. Bump the gas used in there since it's no longer enough again, and remove the check function and all it's uses, replaced with require NoError. * [995]: Fix unit test that was failing because TextProposal moved to v1beta1 (from types) and is in an error string check. * [995]: In the metadata cli test, set the suite's config value before generating the keyring stuff so that it can use the config as intended (not sure how it didn't panic before). Also change a couple expected error strings because some errors changed. * [995]: Fix the marker cli tests. Need more gas for the gov proposals. Need to set the suite's cfg earlier (same as what metadate cli tests did). And now expect a 0x9 instead of 0x5 from the withdraw proposal. * [995]: Fix the marker handler test, invalid proposal since the package changed for TextProposal (to v1beta1 from types). * [995]: Correct an incorrectly updated comment (that might be useful when fixing stuff in a bit). * [995]: Fix the marker operations test that changed due to added use of the random number generator by the SDK. Also added a note about that and a list of valid options that things might change to in the future. But I left it looking for one specific name since that shouldn't change unless making changes to the SDK or operations. * [995]: Fix the Holding query since the data is stored differently now. key is <prefix><account><denom>, value is <amount>. Used to be key is <prefix><account>, value is <coin>. * [995]: Refactor the marker Holding query to use the sdk's DenomOwners query (should have better performance). * [995]: lint fixes. * [995]: Undo a lot of the formatting/division changes that I had made to TestMsgService since we're going to fix that in a separte PR. * [995]: Skip several MsgService tests, to be fixed in a separate PR. * [995]: Switch the TODO comment in msg_fees_decorator to reference that it's required for 1.12. * [995]: Enhance the changelog with more info about what's being done here. * [995]: Switch wasm to a version tagged on our own fork so that we can keep moving forward. * [995]: Add new upgrade handler that runs migrations. Add a couple 1.12.x TODO entries. * [995]: Remove unused deprecated Conditions field. * [995]: Replace deprecated use of strings.Title. * [995]: Replace use of a couple deprecated gov cli flags, and just make local versions of them to use instead. * [995]: Correct the changelog entry about the DenomOwners query. * [995]: Undo a comment change that will be handy during a followup. * [995]: Delete a couple missed references to sims-state-listening.mk. * [1010]: Replace use of the deprecated sdkResponse.Data field. The documentation says it's still usable, but something changed because it made the sim tests all start failing due to index out of range (asking for 0, but no entries). That indicates that stuff was missing from the Data field, and so we can't use it anymore. * [995]: Fix the runsim sims. I accidentally got rid of the -tags part that allows the other dbs to be used. Also simplified it further since there doesn't need to be differences between the runsim and go test versions. * [1010]: Tweak the .Data fix to use the GetCachedValue off the response rather than reunmarshaling it from the bytes. * [995]: Add unit tests on the ExportAppStateAndValidators function. * [995]: Fix the casting of the MsgInstantiateContractResponse. * [995]: Update the TestAppImportExport simulation to bring it in line with it's cosmos-sdk counterpart. Fail if the db can't close. Handle the panic when there are no validators. Provide some keys to the authzkeeper checking. * [995]: Fix a couple marker tests that started failing because I fixed the account numbers thing. * [995]: Bump the sdk to v0.46.0-pio-1-rc3 (from v0.46.0-pio-1-rc2). * [995]: Bump cosmos-sdk to v0.46.0-pio-1-rc4 (from v0.46.0-pio-1-rc3). * [995]: Add ochre-rc1 upgrade handler. * [995]: Fix the changelog so that this new stuff isn't listed in the 1.12 section. * [995]: Update the TODO: Required comments to reference v1.13.x (instead of v1.12.x). * [995]: Fix the name of the TestHandlerTestSuite (was TestAnteTestSuite, which is a different thing). * [995]: Undo a change to the msg_fee_invoker that I made at some point, but now don't want since it's in an area to be fixed anyway. * Updated to remove deprecated sdk.Int. (#1036) * [995]: Disable the sims for the same reason I needed to disable some of the msg service router tests. Added TODO comments about removing the skips too. * [995]: Add issue links to the TODOs needed to be todone with issue 1006 (the fees stuff). * [995]: Add issue links to the other new TODOs. * [995]: Bump cosmos to v0.46.0-pio-1-rc5 (from v0.46.0-pio-1-rc4). * Replaced deprecated SetFullFundraiserPath with SetCoinType and SetPurpose (#1037) * Removed SetFullFundraiserPath and replaced it with SetCoinType and SetPurpose. * Added prefix_test and updated SetConfig to have a configurable seal param. This was mainly to help with testing. * [995]: Lint fixes. * [1006]: Create a public GetFeeTx that converts a tx to a fee tx or returns a standard error (since that was copied all over). * [1006]: Refactor the TxGasLimitDecorator. Only bypass if ALL msgs are gov (instead of just at least one). Include both old and new governance messages. * [1006]: Use our own antehandler in SetupTest. * [1006]: Reorder the ante handlers to more closely match the SDK's default one. * [1006]: A little cleanup on the MsgFeeDecorator. * [1006]: Start refactoring the ProvenanceDeductFeeDecorator to more closely match the SDK's DeductFeeDecorator. Still have stuff to work out here though. * Revert "[1006]: Start refactoring the ProvenanceDeductFeeDecorator to more closely match the SDK's DeductFeeDecorator. Still have stuff to work out here though." This reverts commit b9a7a3c. * [1006]: Remove all the test skips. * [1006]: Get rid of the cosmosante.NewDeductFeeDecorator since we have our own already added (this was the primary source of problems). * [1006]: Used named attributes in NewMsgFeesDecorator. * [1006]: Include the MsgFeesKeeper when creating the AnteHandler for tests. * [1006]: Bump the NewTestGasLimit because the TestDeductFeesNoDelegation didn't have enough. * [1006]: Add a replacement for the MempoolFeeDecorator since that functionality got moved into the DeductFeeDecorator and we're using our own for that. * [1006]: Create helper for getting the fee gas meter to standardize on the error. * [1006]: consume coins instead of just a coin. This makes a lot of parts more robust (e.g. no longer assuming the coins are the same). * [1006]: Remove some named return variables that don't need to be named. * [1006]: Move CalculateAdditionalFeesToBePaid to the msgfees keeper since it's all msgfee lookup stuff anyway. * [1006]: Pull in the cosmossdk.io/errors package. * [1006]: Add the chainid back into test app setup since some of the fee stuff behaves differently when there's no chain id, and it was breaking tests. * [1006]: Clean up the msg fee invoker and add the payer to the events. * [1006]: Overhaul the msg service router. Extract the Provenance-specific stuff into it's own function so that the main function is nearly identical to what the SDK has. Really simplify what it's doing too. * [1006]: Overhaul the ProvenanceDeductFeeDecorator. Make it closer to the DeductFeesDecorator in the SDK. Emit two events, one for the whole fee and one for the base fee (which is deducted in here). In here, check that the payer account has enough funds to cover the fees. * [1006]: Overhaul the NewMsgFeesDecorator to only check that enough fees have been provided to cover the base fee plus any message based fees. * [1006]: Update the antewrapper handler to call the new NewMinGasPricesDecorator and move the NewMsgFeesDecorator to right after that (since they're similar). * [1006]: Remove the TestCalculateAdditionalFeesToBePaid tests from the antewrapper stuff since it moved elsewhere (along with the tests). Call antewrapper.NewMsgFeesDecorator with the correct new args. * [1006]: Change the AnteTestSuite receivers to s (from suite). * [1006]: Fix the prov feegrant tests that started to fail because of a change in the error messages. * [1006]: Fix where the base fee wasn't being properly defined. * [1006]: Fix and clean up a bunch of the antewrapper tests. Some were broken, some not doing what they said, and some were redundant. Also, functionality in those changed so they needed updating to reflect those changes. * [1006]: Delete the TestEnsureMempoolFees test since a) the ProvenanceDeductFeeDecorator doesn't do that anymore and b) it was testing the SDK's NewDeductFeeDecorato. * [1006]: Delete a bunch of tests that are no longer relevant and functionality is tested where it was moved. * [1006]: a little cleanup and added robustnest. * [1006]: small text tweak. * [1006]: Lint fixes. * [995]: Add some lookups to the TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations test for some extra debug info. * [995]: Fix import order. * [995]: Refactor TestRewardsProgramStartPerformQualifyingActions_Vote_InvalidDelegations a bit. Make it so that the votes are coming from an account that isn't a validator. * [995]: Remove the nickel upgrade handler and combine it with my ochre one. * [995]: Add missing comment about how the tests are initiated. * [995]: Lint fixes. * [1006]: Simplify and fix MsgFeesDistribution.Increase. I originally had the logic backwards for adding to recipients. Also, go no longer complains when the map entry is null and you still call .Add on it. * [1006]: Add some more tests on CalculateAdditionalFeesToBePaid to cover the different splits now. * [1006]: Fix some test compilation issues from after the merge. * [1006]: Rename suite to s. * [1006]: Fix a couple unit tests now looking at the wrong denom. * [1006]: Fix a unit test. * [1006]: Re-enable TestRewardsProgramStart and bring it in-line with the other tests. * [1006]: Re-enable the TestMsgServiceMsgFeeWithRecipient test and simplify all the msg_fees attribut json stuff. * [1006]: Clean up and re-enable TestRewardsProgramStartPerformQualifyingActions. * [1006]: Fix a bunch of the asserts and make the tests generally more robust. Add some missing NoError checks. Only access [0] after asserting it's not empty. Make the assert/require messages provide better info about what value is being checked. Include messages in all of the NoError checks. * [1006]: Restrict lines to 130 charcters (except for the one function name that's super huge. * [1006]: Update some test failure messages to provide more context (specially in loops). * [1006]: Remove all the lines that just do ctx.WithBlockTime(time.Now()) since that doesn't actually do anything. * [1006]: Fix the TestFullBIP44Path test to actually test what we're trying to test. * [995]: Fix the TestFullBIP44Path test to actually test what we're trying to test. * [995]: Use the FundModuleAccount function instead of minting and sending. * [995]: Bump Cosmos-SDK to v0.46.1-pio-1-rc1 (from v0.46.0-pio-1-rc5). It's exactly the same, just named more accurately. * [995]: lazy-load the govVoteMsgUrls messages, just to be safe. * [995]: Update the isGovernanceMessage func to account for the new version of governance messages. * [995]: Fix the gov msg count. * [995]: Lint fixes. I really hate that the Go convention for acronyms in variable names is to capitalize all or none of them. * [995]: Switch msgURL setting logic to just look for an empty slice instead of slice of specific length. Makes it a little more future-safe. * [995]: Switch those checks to a nil check and add a comment about it. * [1006]: Remove loop over map. * [1006]: Enhance an error message. * [1006]: Correct the amount being used by the message fee invoker. * [1006]: Switch to our feegrant keeper interface and add a log message. * [1006]: Add some debug logging. * [1006]: Fix a comment. * [1006]: Make the simulation gas limits a little more accurate by not bypassing some msg fee lookups. * [1006]: Make an IsInitGenesis function to use in place of the ctx.BlockHeight() > 0 check. Probably going to need this elsewhere too. * [1006]: Only check their accounts for the msg-based fees (don't include the base fee in the check). And only deduct the base fee if we're not in InitGenesis. * [1006]: Send the address to DeductFees since the full account isn't needed. Also, add the account address to the error in there. * [1006]: Fix a couple tests that broke because I changed which amounts are being checked in accounts and also changed the error message returned in one case. * [1006]: Update the comment on checkTxFeeWithValidatorMinGasPrices too. * [1006]: Add the min_fee_charged event/attribute back in and change baseFee back to what it was before (fee - additional fees). * [1006]: Add a test that makes sure events are emitted for failed Txs wwhere the min fee is charged. * [1006]: Expand the TestFailedTx to check both with and without msg-based fees. * [1006]: Add a changelog entry. * [1006]: Expand the event documentation a bit more. * [1006]: Grammar fix. * [1006]: Remove the upgrade test script that got accidentally added a bit ago. Co-authored-by: Matt Witkowski <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Replaces the deprecated SetFullFundraiserPath with SetCoinType and SetPurpose.
closes: #1011
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/) or specification (x/<module>/spec/)godoccomments.Unreleasedsection inCHANGELOG.mdFiles changedin the Github PR explorerCodecov Reportin the comment section below once CI passes