* Introduce RawContractMessage type
* Add json signbytes test for proposals
* No assumptions on MsgIBCSend.data content
* Smart query uses RawContractMessage
* Revert method signature change to be consistent
* Review comment
* Update after discussions
* added golangci lint check on pull requests
* changelog update
* updated to use circleci instead of github actions
* testing a golang change
* added workflow
* updated how we find the golangci command in the make file
* using orb for golangci
* Added golangci install make command
* use docker image for golangci
* stop using make in circleci
* reverted golang change to trigger ci
* gofmt
* make fmt
* fixed a few things
* updated version on golintci
* fixed all the lint errors
* check version
* skipped wrongly failing lint
* Revert generated file changes
* fix import grouping, return errors on failures, initialization of arrays revert
* fixed a few lint errors
* addressed more code review comments
* updated with error check
* increased timeout for golangci-lint
* dont format autogenerated files
Co-authored-by: Carlton Hanna <channa@figure.com>
If we use a wallet address which does not exist, it gives the following error:
```
Error: empty address string is not allowed
```
which is coming from the SDK and confusing to the user.
The reason is because one error is not handled properly, in the new proposal, the error is handled which returns the following error if the address does not exist:
```
Error: The specified item could not be found in the keyring
```
* Run benchmarks on circle CI
* Add benchmark for secp256k1 verification
* Add compilation benchmark
* Move parallelism back to 1 for benchmarks
* Review comments
Co-authored-by: Alex Peters <alpe@users.noreply.github.com>
* More keeper tests
x/wasm/keeper tests are extended to test various input validation.
Keeper input is validated before passing to the keeper method when used
within wasmd application. We cannot ensure such validation when this
keeper is used outside of wasmd application. To keep it safe, fully
validate keeper methods input.
hackatom.wasm is loaded into memory during initialization to avoid
reading file in each test separately. Once migrated to go 1.16, embed
package should be used instead.
Run goimport on certain files.
Some comments fixed or removed.
* ensure that creator address is not nil
