* (refactor): Added new MsgUpdateParams tx and generated new proto files
* (refactor): Refactor for migration of x/params
* (fix): Refactor to use single Params store key for easier more readable code
* (fix): removed unused
* (fix): add validation
* (fix): fix linter
* remove line
* Added changes from code review
* Apply changes from code review
* (fix): Made ParamKey back to a string
* Added CHANGELOG entry
* Apply suggestions from code review
Co-authored-by: Tomas Guerra <54514587+GAtom22@users.noreply.github.com>
* (fix): remove HTTP endpoint exposure
* Apply suggestions from code review
Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com>
* fix: Apply changes from code review and run linter
* Update x/feemarket/keeper/params.go
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
* Update x/feemarket/types/msg.go
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
* tests: added tests for msg_server and msg
* tests: add failing test for migration
* Update x/feemarket/keeper/params.go
Co-authored-by: Tomas Guerra <54514587+GAtom22@users.noreply.github.com>
Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
* Add min_gas_price to feemarket params
* Add MinGasPriceDecorators
* feemarket integration tests for MinGasPrice
* Restructure integration tests
* Simplify integration tests context
We use DeliverTx context to set up the app, otherwise not all settings are initialized.
We test CheckTx with `s.app.BaseApp.CheckTx(req)`, which uses the `CheckTx` mode and context.
* Update MinGasPrice spec in feemarket module
* reorder ethermint module order for initializing genesis
* feemarket migrations for adding MinGasPrice param
* update changelog
* Additional unit tests for MinGasPrice = 0, tx gas price > 0 (PR review)
https://github.com/tharsis/ethermint/pull/1104#discussion_r884991661
* Use 0 MinGasPrice for transaction simulations
* Fix duplicate registration of feemarket GenesisState and Params (PR review)
https://github.com/tharsis/ethermint/pull/1104#issuecomment-1141893712
* evm: fixed commented out simulations, pubsub, and handler tests
fixes#640
simulations are very basic: they can be built and executed,
but they don't generate any EVM-related transactions yet.
(It should be a matter of adding simulation-related code to the
modules + potentially extra helpers to the simulation.)
handler tests miss some extra assertions due to changes
in the return values snapshotting logic (ADR-001 and ADR-002).
Besides the test suites identified in the audit,
there's also "importer_test.go" which wasn't yet fixed.
(it'd require major rewriting + extra test resources)
* gofumpt
