Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

spec: claims module #255

Merged
merged 14 commits into from
Feb 9, 2022
Merged

spec: claims module #255

merged 14 commits into from
Feb 9, 2022

Conversation

fedekunze
Copy link
Contributor

Description

Closes: ENG-408


All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

PR review checkboxes:

I have...

  • added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • included the correct type prefix in the PR title
  • targeted the correct branch (see PR Targeting)
  • provided a link in the PR description to the relevant issue or specification
  • reviewed "Files changed" and left comments if necessary
  • confirmed all required CI checks have passed

Code maintenance:

I have...

  • written unit and integration tests
  • added relevant godoc and code comments.
  • updated relevant documentation (docs/) or specification (x/<module>/spec/)

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

  • confirmed the correct type prefix in the PR title
  • confirmed all author checklist items have been addressed
  • confirmed that this PR does not change production code

@linear
Copy link

linear bot commented Feb 1, 2022

@github-actions github-actions bot added the docs label Feb 1, 2022
@github-actions github-actions bot added the proto label Feb 8, 2022
@@ -29,7 +29,7 @@ service Query {
// ClaimsRecord returns the claims record for a given address
rpc ClaimsRecord(QueryClaimsRecordRequest)
returns (QueryClaimsRecordResponse) {
option (google.api.http).get = "/evmos/claims/v1/claims_record/{address}";
option (google.api.http).get = "/evmos/claims/v1/claims_records/{address}";
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hanchon I'm changing this because it's a standard API practice to use plurals

@github-actions github-actions bot added the CLI label Feb 8, 2022
3. Transfer the claimable amount from the escrow account to the user balance
4. Update the claim record or delete it if all the actions have been claimed.

## IBC Middleware - IBC Transfer Action
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the remaining part is on #199

@fedekunze fedekunze marked this pull request as ready for review February 8, 2022 14:35
@codecov
Copy link

codecov bot commented Feb 8, 2022

Codecov Report

Merging #255 (f142f43) into main (1f374de) will not change coverage.
The diff coverage is 100.00%.

❗ Current head f142f43 differs from pull request most recent head 8914b6e. Consider uploading reports for the commit 8914b6e to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #255   +/-   ##
=======================================
  Coverage   75.09%   75.09%           
=======================================
  Files          76       76           
  Lines        4084     4084           
=======================================
  Hits         3067     3067           
  Misses        884      884           
  Partials      133      133           
Impacted Files Coverage Δ
x/claims/types/params.go 36.92% <100.00%> (ø)

Copy link
Contributor

@danburck danburck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I added some remarks when I thought it wasn't clear enough as a first-time reader

x/claims/spec/01_concepts.md Outdated Show resolved Hide resolved
x/claims/spec/01_concepts.md Show resolved Hide resolved
x/claims/spec/01_concepts.md Outdated Show resolved Hide resolved
x/claims/spec/01_concepts.md Outdated Show resolved Hide resolved
x/claims/spec/01_concepts.md Outdated Show resolved Hide resolved
x/claims/spec/03_state_transitions.md Outdated Show resolved Hide resolved
x/claims/spec/03_state_transitions.md Outdated Show resolved Hide resolved
x/claims/spec/04_hooks.md Outdated Show resolved Hide resolved
x/claims/spec/04_hooks.md Outdated Show resolved Hide resolved
x/claims/spec/04_hooks.md Outdated Show resolved Hide resolved
@fedekunze
Copy link
Contributor Author

@danburck I incorporated your suggestions 👍

@fedekunze fedekunze enabled auto-merge (squash) February 9, 2022 13:51
@fedekunze fedekunze merged commit 2f9b12a into main Feb 9, 2022
@fedekunze fedekunze deleted the ENG-408-claims-spec branch February 9, 2022 13:51
khoslaventures pushed a commit that referenced this pull request Feb 15, 2022
* fix: spelling (#254)

* bug (erc20): Compile built-in contracts in the build process (#246)

* wip

* use the raw value for json (#253)

* use the raw value for json

* use the raw value for json

* check if solc and jq is installed

* feat(erc20): wip iterate over contracts

* feat(erc20): iterate over contracts

* feat(erc20): add ERC20Burnable contract

* feat(erc20): lint Makefile

* feat(erc20): move contracts dir

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>

* build(deps): bump github.com/cosmos/ibc-go/v3 (#257)

* build(deps): bump github.com/cosmos/cosmos-sdk from 0.45.0 to 0.45.1 (#258)

Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.45.0 to 0.45.1.
- [Release notes](https://github.com/cosmos/cosmos-sdk/releases)
- [Changelog](https://github.com/cosmos/cosmos-sdk/blob/v0.45.1/CHANGELOG.md)
- [Commits](cosmos/cosmos-sdk@v0.45.0...v0.45.1)

---
updated-dependencies:
- dependency-name: github.com/cosmos/cosmos-sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat(inflation): update `bondingIncentive` automatically (#252)

* feat(inflation): update bondedRatio automically and bTarget and maxVariance as params

* feat(inflation): update comments

* feat(inflation): rename b_target to bonding_target

* feat(inflation): set maxVariance to zero at genesis

* feat(inflation): fix test

* feat(inflation): don"t store bondedRatio in store

* feat(inflation): fix unit tests

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* `x/incentives`: integration tests (#256)

* wip

* tests(incentives): add test setup

* tests(inflation): wip

* tests(inflation): wip

* wip

* integration(incentives): setup integration test suite

* integration(incentives): wip debugging distribution

* integration(incentives): fix integration tests distribution

* integration(incentives): remove old integration test folder

* integration(incentives): add check for incentiveProposal if contract exists at address

* remove ethermint replace mod

* ENG 477 Integration Tests - `x/inflation` (#259)

* wip

* tests(incentives): add test setup

* tests(inflation): wip

* tests(inflation): wip

* wip

* wip

* feat(inflation): add integration tests

* feat(inflation): lint

* feat(inflation): remove local ethermint dependency

* test if github ci pipeline fails with failing integration tests

* integration(inflation): Add community pool test

* integration(inflation): fix import

* integration(inflation): fix import

* build(deps): bump github.com/onsi/gomega from 1.17.0 to 1.18.1 (#261)

Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.17.0 to 1.18.1.
- [Release notes](https://github.com/onsi/gomega/releases)
- [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md)
- [Commits](onsi/gomega@v1.17.0...v1.18.1)

---
updated-dependencies:
- dependency-name: github.com/onsi/gomega
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ENG 470 inflation module spec (#263)

* spec(inflation): fixes from writing the spec

* spec(inflation): import md

* spec(inflation): refactor

* spec(inflation): fix lint

* spec(evmos): fix module list (#264)

* spec: claims module (#255)

* claims: spec

* hooks and concepts

* spec client

* swagger

* events

* address comments from review

* decay period update

* fix

* epoch: unit tests (#265)

* tests(epochs): add types tests

* tests(epochs): refactor epoch hook interface

* tests(epochs): tidy

* tests(epochs): remove unused handler

* tests(epochs): remove unused cli test

* tests(epochs): delete simulation file

* tests(epochs): add keeper grpc tests

* tests(epochs): remove unused comment

* docs: grpc gateway docs (#266)

* feat: ibc transfer claims middleware  (#199)

* claims module

* update

* fixes

* actions

* hook

* app setup

* params

* rm spec

* claim fixes

* gRPC fixes

* fix lint

* proto-lint

* tests

* feat: ibc transfer hook middleware

* update version logic

* fixes

* cleanup ibc module

* grpc and genesis

* cli'

* lint

* middleware

* app setup

* grpc

* minor change

* test setup

* swagger update

* claim queries

* claim records tests

* genesis check

* Apply suggestions from code review

* remove dup calls

* Update x/claim/keeper/ibc_callbacks.go

Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>

* fixes

* fix object order on app.go file (#210)

* claims: tests (#205)

* WIP: finish up claims unit tests

* WIP: if action is completed, no rewards

* fix more tests

* update claim type

* WIP: finish up claims unit tests

* WIP: if action is completed, no rewards

* fix more tests

* bookmark

* rename default denom

* fix moar tests

* change the denom

* bookmark

* comment out the clawback

* claim type permission change

* PR feedback address

* Update x/claim/types/claim_record_test.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* ibc callbacks tests (#214)

* wip ibc callbacks tests

* test fixes

* wip test, send and receive ibc

* OnReceive callback tests

* onAck tests

* fixes to ibc testing

* rm claim

* deps: use IBC-go v3 newest commit

* fixes (wip)

* hook

* merge claim records logic

* ibc transfer claim

* fix out of range

* gitignore

* comments

* ibc fix

* claims module ibctesting (#262)

* change ibc-go version, fix tests errors

* fix claim tests

* update tests

* wip ack not working

* port ibctesting

* fix mergeclaim and test

* port testutil

* replace mint with inflation

* delete unnecessary ibctesting files

* delete more files

* fix claim on ibc-ack

* cleanup

* lint

* claims test

* revert changes to ibctesting app

* claims hooks tests

* ibc callback unittests

* validate params test

* types params tests

* lint

Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>

* tests

Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>

* build(deps): bump github.com/onsi/ginkgo/v2 from 2.1.1 to 2.1.2 (#269)

* bug(erc20): wip fix gas estimation (#267)

* bug(erc20): wip fix gas estimation

* bug(erc20): fix gas estimation

* bug(erc20): rename payload to data

* bug(sigverify): fix multisig (#270)

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* deps(ethermint): bump to v0.10.0-beta1 (#272)

* add changelog

Co-authored-by: Rajiv Patel-O'Connor <rajiv.patel.oconnor@gmail.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>
Co-authored-by: Prajjwol Gautam <not@acyb.org>
danburck added a commit that referenced this pull request Feb 15, 2022
* release: v1-alpha1 changelog (#251)

* release: v1.0.0-beta1 changelog (#273)

* fix: spelling (#254)

* bug (erc20): Compile built-in contracts in the build process (#246)

* wip

* use the raw value for json (#253)

* use the raw value for json

* use the raw value for json

* check if solc and jq is installed

* feat(erc20): wip iterate over contracts

* feat(erc20): iterate over contracts

* feat(erc20): add ERC20Burnable contract

* feat(erc20): lint Makefile

* feat(erc20): move contracts dir

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>

* build(deps): bump github.com/cosmos/ibc-go/v3 (#257)

* build(deps): bump github.com/cosmos/cosmos-sdk from 0.45.0 to 0.45.1 (#258)

Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.45.0 to 0.45.1.
- [Release notes](https://github.com/cosmos/cosmos-sdk/releases)
- [Changelog](https://github.com/cosmos/cosmos-sdk/blob/v0.45.1/CHANGELOG.md)
- [Commits](cosmos/cosmos-sdk@v0.45.0...v0.45.1)

---
updated-dependencies:
- dependency-name: github.com/cosmos/cosmos-sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat(inflation): update `bondingIncentive` automatically (#252)

* feat(inflation): update bondedRatio automically and bTarget and maxVariance as params

* feat(inflation): update comments

* feat(inflation): rename b_target to bonding_target

* feat(inflation): set maxVariance to zero at genesis

* feat(inflation): fix test

* feat(inflation): don"t store bondedRatio in store

* feat(inflation): fix unit tests

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* `x/incentives`: integration tests (#256)

* wip

* tests(incentives): add test setup

* tests(inflation): wip

* tests(inflation): wip

* wip

* integration(incentives): setup integration test suite

* integration(incentives): wip debugging distribution

* integration(incentives): fix integration tests distribution

* integration(incentives): remove old integration test folder

* integration(incentives): add check for incentiveProposal if contract exists at address

* remove ethermint replace mod

* ENG 477 Integration Tests - `x/inflation` (#259)

* wip

* tests(incentives): add test setup

* tests(inflation): wip

* tests(inflation): wip

* wip

* wip

* feat(inflation): add integration tests

* feat(inflation): lint

* feat(inflation): remove local ethermint dependency

* test if github ci pipeline fails with failing integration tests

* integration(inflation): Add community pool test

* integration(inflation): fix import

* integration(inflation): fix import

* build(deps): bump github.com/onsi/gomega from 1.17.0 to 1.18.1 (#261)

Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.17.0 to 1.18.1.
- [Release notes](https://github.com/onsi/gomega/releases)
- [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md)
- [Commits](onsi/gomega@v1.17.0...v1.18.1)

---
updated-dependencies:
- dependency-name: github.com/onsi/gomega
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ENG 470 inflation module spec (#263)

* spec(inflation): fixes from writing the spec

* spec(inflation): import md

* spec(inflation): refactor

* spec(inflation): fix lint

* spec(evmos): fix module list (#264)

* spec: claims module (#255)

* claims: spec

* hooks and concepts

* spec client

* swagger

* events

* address comments from review

* decay period update

* fix

* epoch: unit tests (#265)

* tests(epochs): add types tests

* tests(epochs): refactor epoch hook interface

* tests(epochs): tidy

* tests(epochs): remove unused handler

* tests(epochs): remove unused cli test

* tests(epochs): delete simulation file

* tests(epochs): add keeper grpc tests

* tests(epochs): remove unused comment

* docs: grpc gateway docs (#266)

* feat: ibc transfer claims middleware  (#199)

* claims module

* update

* fixes

* actions

* hook

* app setup

* params

* rm spec

* claim fixes

* gRPC fixes

* fix lint

* proto-lint

* tests

* feat: ibc transfer hook middleware

* update version logic

* fixes

* cleanup ibc module

* grpc and genesis

* cli'

* lint

* middleware

* app setup

* grpc

* minor change

* test setup

* swagger update

* claim queries

* claim records tests

* genesis check

* Apply suggestions from code review

* remove dup calls

* Update x/claim/keeper/ibc_callbacks.go

Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>

* fixes

* fix object order on app.go file (#210)

* claims: tests (#205)

* WIP: finish up claims unit tests

* WIP: if action is completed, no rewards

* fix more tests

* update claim type

* WIP: finish up claims unit tests

* WIP: if action is completed, no rewards

* fix more tests

* bookmark

* rename default denom

* fix moar tests

* change the denom

* bookmark

* comment out the clawback

* claim type permission change

* PR feedback address

* Update x/claim/types/claim_record_test.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* ibc callbacks tests (#214)

* wip ibc callbacks tests

* test fixes

* wip test, send and receive ibc

* OnReceive callback tests

* onAck tests

* fixes to ibc testing

* rm claim

* deps: use IBC-go v3 newest commit

* fixes (wip)

* hook

* merge claim records logic

* ibc transfer claim

* fix out of range

* gitignore

* comments

* ibc fix

* claims module ibctesting (#262)

* change ibc-go version, fix tests errors

* fix claim tests

* update tests

* wip ack not working

* port ibctesting

* fix mergeclaim and test

* port testutil

* replace mint with inflation

* delete unnecessary ibctesting files

* delete more files

* fix claim on ibc-ack

* cleanup

* lint

* claims test

* revert changes to ibctesting app

* claims hooks tests

* ibc callback unittests

* validate params test

* types params tests

* lint

Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>

* tests

Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>

* build(deps): bump github.com/onsi/ginkgo/v2 from 2.1.1 to 2.1.2 (#269)

* bug(erc20): wip fix gas estimation (#267)

* bug(erc20): wip fix gas estimation

* bug(erc20): fix gas estimation

* bug(erc20): rename payload to data

* bug(sigverify): fix multisig (#270)

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* deps(ethermint): bump to v0.10.0-beta1 (#272)

* add changelog

Co-authored-by: Rajiv Patel-O'Connor <rajiv.patel.oconnor@gmail.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>
Co-authored-by: Prajjwol Gautam <not@acyb.org>

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Rajiv Patel-O'Connor <rajiv.patel.oconnor@gmail.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>
Co-authored-by: Prajjwol Gautam <not@acyb.org>
@fedekunze fedekunze mentioned this pull request Feb 28, 2022
12 tasks
fedekunze added a commit that referenced this pull request Feb 28, 2022
* fix: spelling (#254)

* bug (erc20): Compile built-in contracts in the build process (#246)

* wip

* use the raw value for json (#253)

* use the raw value for json

* use the raw value for json

* check if solc and jq is installed

* feat(erc20): wip iterate over contracts

* feat(erc20): iterate over contracts

* feat(erc20): add ERC20Burnable contract

* feat(erc20): lint Makefile

* feat(erc20): move contracts dir

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>

* build(deps): bump github.com/cosmos/ibc-go/v3 (#257)

* build(deps): bump github.com/cosmos/cosmos-sdk from 0.45.0 to 0.45.1 (#258)

Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.45.0 to 0.45.1.
- [Release notes](https://github.com/cosmos/cosmos-sdk/releases)
- [Changelog](https://github.com/cosmos/cosmos-sdk/blob/v0.45.1/CHANGELOG.md)
- [Commits](cosmos/cosmos-sdk@v0.45.0...v0.45.1)

---
updated-dependencies:
- dependency-name: github.com/cosmos/cosmos-sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat(inflation): update `bondingIncentive` automatically (#252)

* feat(inflation): update bondedRatio automically and bTarget and maxVariance as params

* feat(inflation): update comments

* feat(inflation): rename b_target to bonding_target

* feat(inflation): set maxVariance to zero at genesis

* feat(inflation): fix test

* feat(inflation): don"t store bondedRatio in store

* feat(inflation): fix unit tests

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* `x/incentives`: integration tests (#256)

* wip

* tests(incentives): add test setup

* tests(inflation): wip

* tests(inflation): wip

* wip

* integration(incentives): setup integration test suite

* integration(incentives): wip debugging distribution

* integration(incentives): fix integration tests distribution

* integration(incentives): remove old integration test folder

* integration(incentives): add check for incentiveProposal if contract exists at address

* remove ethermint replace mod

* ENG 477 Integration Tests - `x/inflation` (#259)

* wip

* tests(incentives): add test setup

* tests(inflation): wip

* tests(inflation): wip

* wip

* wip

* feat(inflation): add integration tests

* feat(inflation): lint

* feat(inflation): remove local ethermint dependency

* test if github ci pipeline fails with failing integration tests

* integration(inflation): Add community pool test

* integration(inflation): fix import

* integration(inflation): fix import

* build(deps): bump github.com/onsi/gomega from 1.17.0 to 1.18.1 (#261)

Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.17.0 to 1.18.1.
- [Release notes](https://github.com/onsi/gomega/releases)
- [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md)
- [Commits](onsi/gomega@v1.17.0...v1.18.1)

---
updated-dependencies:
- dependency-name: github.com/onsi/gomega
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ENG 470 inflation module spec (#263)

* spec(inflation): fixes from writing the spec

* spec(inflation): import md

* spec(inflation): refactor

* spec(inflation): fix lint

* spec(evmos): fix module list (#264)

* spec: claims module (#255)

* claims: spec

* hooks and concepts

* spec client

* swagger

* events

* address comments from review

* decay period update

* fix

* epoch: unit tests (#265)

* tests(epochs): add types tests

* tests(epochs): refactor epoch hook interface

* tests(epochs): tidy

* tests(epochs): remove unused handler

* tests(epochs): remove unused cli test

* tests(epochs): delete simulation file

* tests(epochs): add keeper grpc tests

* tests(epochs): remove unused comment

* docs: grpc gateway docs (#266)

* feat: ibc transfer claims middleware  (#199)

* claims module

* update

* fixes

* actions

* hook

* app setup

* params

* rm spec

* claim fixes

* gRPC fixes

* fix lint

* proto-lint

* tests

* feat: ibc transfer hook middleware

* update version logic

* fixes

* cleanup ibc module

* grpc and genesis

* cli'

* lint

* middleware

* app setup

* grpc

* minor change

* test setup

* swagger update

* claim queries

* claim records tests

* genesis check

* Apply suggestions from code review

* remove dup calls

* Update x/claim/keeper/ibc_callbacks.go

Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>

* fixes

* fix object order on app.go file (#210)

* claims: tests (#205)

* WIP: finish up claims unit tests

* WIP: if action is completed, no rewards

* fix more tests

* update claim type

* WIP: finish up claims unit tests

* WIP: if action is completed, no rewards

* fix more tests

* bookmark

* rename default denom

* fix moar tests

* change the denom

* bookmark

* comment out the clawback

* claim type permission change

* PR feedback address

* Update x/claim/types/claim_record_test.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* ibc callbacks tests (#214)

* wip ibc callbacks tests

* test fixes

* wip test, send and receive ibc

* OnReceive callback tests

* onAck tests

* fixes to ibc testing

* rm claim

* deps: use IBC-go v3 newest commit

* fixes (wip)

* hook

* merge claim records logic

* ibc transfer claim

* fix out of range

* gitignore

* comments

* ibc fix

* claims module ibctesting (#262)

* change ibc-go version, fix tests errors

* fix claim tests

* update tests

* wip ack not working

* port ibctesting

* fix mergeclaim and test

* port testutil

* replace mint with inflation

* delete unnecessary ibctesting files

* delete more files

* fix claim on ibc-ack

* cleanup

* lint

* claims test

* revert changes to ibctesting app

* claims hooks tests

* ibc callback unittests

* validate params test

* types params tests

* lint

Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>

* tests

Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>

* build(deps): bump github.com/onsi/ginkgo/v2 from 2.1.1 to 2.1.2 (#269)

* bug(erc20): wip fix gas estimation (#267)

* bug(erc20): wip fix gas estimation

* bug(erc20): fix gas estimation

* bug(erc20): rename payload to data

* bug(sigverify): fix multisig (#270)

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* deps(ethermint): bump to v0.10.0-beta1 (#272)

* danburck/release changelog (#274)

* release: v1-alpha1 changelog (#251)

* release: v1.0.0-beta1 changelog (#273)

* fix: spelling (#254)

* bug (erc20): Compile built-in contracts in the build process (#246)

* wip

* use the raw value for json (#253)

* use the raw value for json

* use the raw value for json

* check if solc and jq is installed

* feat(erc20): wip iterate over contracts

* feat(erc20): iterate over contracts

* feat(erc20): add ERC20Burnable contract

* feat(erc20): lint Makefile

* feat(erc20): move contracts dir

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>

* build(deps): bump github.com/cosmos/ibc-go/v3 (#257)

* build(deps): bump github.com/cosmos/cosmos-sdk from 0.45.0 to 0.45.1 (#258)

Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.45.0 to 0.45.1.
- [Release notes](https://github.com/cosmos/cosmos-sdk/releases)
- [Changelog](https://github.com/cosmos/cosmos-sdk/blob/v0.45.1/CHANGELOG.md)
- [Commits](cosmos/cosmos-sdk@v0.45.0...v0.45.1)

---
updated-dependencies:
- dependency-name: github.com/cosmos/cosmos-sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat(inflation): update `bondingIncentive` automatically (#252)

* feat(inflation): update bondedRatio automically and bTarget and maxVariance as params

* feat(inflation): update comments

* feat(inflation): rename b_target to bonding_target

* feat(inflation): set maxVariance to zero at genesis

* feat(inflation): fix test

* feat(inflation): don"t store bondedRatio in store

* feat(inflation): fix unit tests

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* `x/incentives`: integration tests (#256)

* wip

* tests(incentives): add test setup

* tests(inflation): wip

* tests(inflation): wip

* wip

* integration(incentives): setup integration test suite

* integration(incentives): wip debugging distribution

* integration(incentives): fix integration tests distribution

* integration(incentives): remove old integration test folder

* integration(incentives): add check for incentiveProposal if contract exists at address

* remove ethermint replace mod

* ENG 477 Integration Tests - `x/inflation` (#259)

* wip

* tests(incentives): add test setup

* tests(inflation): wip

* tests(inflation): wip

* wip

* wip

* feat(inflation): add integration tests

* feat(inflation): lint

* feat(inflation): remove local ethermint dependency

* test if github ci pipeline fails with failing integration tests

* integration(inflation): Add community pool test

* integration(inflation): fix import

* integration(inflation): fix import

* build(deps): bump github.com/onsi/gomega from 1.17.0 to 1.18.1 (#261)

Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.17.0 to 1.18.1.
- [Release notes](https://github.com/onsi/gomega/releases)
- [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md)
- [Commits](onsi/gomega@v1.17.0...v1.18.1)

---
updated-dependencies:
- dependency-name: github.com/onsi/gomega
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ENG 470 inflation module spec (#263)

* spec(inflation): fixes from writing the spec

* spec(inflation): import md

* spec(inflation): refactor

* spec(inflation): fix lint

* spec(evmos): fix module list (#264)

* spec: claims module (#255)

* claims: spec

* hooks and concepts

* spec client

* swagger

* events

* address comments from review

* decay period update

* fix

* epoch: unit tests (#265)

* tests(epochs): add types tests

* tests(epochs): refactor epoch hook interface

* tests(epochs): tidy

* tests(epochs): remove unused handler

* tests(epochs): remove unused cli test

* tests(epochs): delete simulation file

* tests(epochs): add keeper grpc tests

* tests(epochs): remove unused comment

* docs: grpc gateway docs (#266)

* feat: ibc transfer claims middleware  (#199)

* claims module

* update

* fixes

* actions

* hook

* app setup

* params

* rm spec

* claim fixes

* gRPC fixes

* fix lint

* proto-lint

* tests

* feat: ibc transfer hook middleware

* update version logic

* fixes

* cleanup ibc module

* grpc and genesis

* cli'

* lint

* middleware

* app setup

* grpc

* minor change

* test setup

* swagger update

* claim queries

* claim records tests

* genesis check

* Apply suggestions from code review

* remove dup calls

* Update x/claim/keeper/ibc_callbacks.go

Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>

* fixes

* fix object order on app.go file (#210)

* claims: tests (#205)

* WIP: finish up claims unit tests

* WIP: if action is completed, no rewards

* fix more tests

* update claim type

* WIP: finish up claims unit tests

* WIP: if action is completed, no rewards

* fix more tests

* bookmark

* rename default denom

* fix moar tests

* change the denom

* bookmark

* comment out the clawback

* claim type permission change

* PR feedback address

* Update x/claim/types/claim_record_test.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* ibc callbacks tests (#214)

* wip ibc callbacks tests

* test fixes

* wip test, send and receive ibc

* OnReceive callback tests

* onAck tests

* fixes to ibc testing

* rm claim

* deps: use IBC-go v3 newest commit

* fixes (wip)

* hook

* merge claim records logic

* ibc transfer claim

* fix out of range

* gitignore

* comments

* ibc fix

* claims module ibctesting (#262)

* change ibc-go version, fix tests errors

* fix claim tests

* update tests

* wip ack not working

* port ibctesting

* fix mergeclaim and test

* port testutil

* replace mint with inflation

* delete unnecessary ibctesting files

* delete more files

* fix claim on ibc-ack

* cleanup

* lint

* claims test

* revert changes to ibctesting app

* claims hooks tests

* ibc callback unittests

* validate params test

* types params tests

* lint

Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>

* tests

Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>

* build(deps): bump github.com/onsi/ginkgo/v2 from 2.1.1 to 2.1.2 (#269)

* bug(erc20): wip fix gas estimation (#267)

* bug(erc20): wip fix gas estimation

* bug(erc20): fix gas estimation

* bug(erc20): rename payload to data

* bug(sigverify): fix multisig (#270)

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* deps(ethermint): bump to v0.10.0-beta1 (#272)

* add changelog

Co-authored-by: Rajiv Patel-O'Connor <rajiv.patel.oconnor@gmail.com>
Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>
Co-authored-by: Prajjwol Gautam <not@acyb.org>

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Rajiv Patel-O'Connor <rajiv.patel.oconnor@gmail.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>
Co-authored-by: Prajjwol Gautam <not@acyb.org>

* build(deps): bump github.com/onsi/ginkgo/v2 from 2.1.2 to 2.1.3 (#275)

Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.1.2 to 2.1.3.
- [Release notes](https://github.com/onsi/ginkgo/releases)
- [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md)
- [Commits](onsi/ginkgo@v2.1.2...v2.1.3)

---
updated-dependencies:
- dependency-name: github.com/onsi/ginkgo/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* tests: add sigverify tests (#277)

* test: sigverify nits (#278)

* deps: bump IBC v3 rc0 (#279)

* feat(vesting): Add BDD tests and `ClawbackVestingAccount` by Agoric (#268)

* integrationTests(Vesting): add vesting tests under icentives module
\

* integrationTests(Vesting): add vesting tests for transfer

* integrationTests(Vesting): correct spelling

* integrationTests(Vesting): update comment

* integrationTests(Vesting): get locked and spendable coins from store

* integrationTests(Vesting): move testing files to new module folder (still with incentives deps) and add Ethereum tx tests

* integrationTests(Vesting): refactor test structure

* integrationTests(Vesting): refactor wording

* integrationTests(Vesting): split cliff and lock cases

* integrationTests(Vesting): split cliff and lock cases

* merge main and add todos

* feat(vesting): copy files and make proto

* feat(vesting): implement custom staking logic and move keeper functions from types to keeper

* feat(vesting): replace keeper types import

* feat(vesting): remove duplicate sdk proto and types

* feat(vesting): fix lint and codec

* feat(vesting): refactor vesting tests to use clawback vesting accounts

* feat(vesting): add lockup tests

* feat(vesting): add governance tests

* feat(vesting): comment out tests

* feat(vesting): delete vesting calculator

* fix IBC test

* rm exported

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>

* ibctesting: use `EthAccounts` for IBC tests (#280)

* ibctesting: use EthAccounts for IBC tests

* fix lint

* impr(vesting): refactor `x/vesting` module (#281)

* feat(vesting): refactor types/msg_test

* feat(vesting): refactor period tests

* feat(vesting): refactor clawback vesting account tests

* feat(vesting): remove init account helper

* feat(vesting): refactore computeClawback tests to TDD

* feat(vesting): wip msg_server_test.go

* feat(vesting): add msg server tests with one open TODO

* feat(vesting): rename clawback address -> accountAddress

* feat(vesting): replace amino codec with cdc in module.go

* feat(vesting): fix EthAccount conversion to Baseaccount

* feat(vesting): add code analysis of post_reward function

* feat(vesting): remove post reward logic

* feat(vesting): remove unsused interface methods

* feat(vesting): fix nil dereference lint

* feat(vesting): fix unchecked assertion lint

* feat(vesting): remove AllowWithdrawAddr hook

* feat(vesting): address PR comments

* feat(vesting): remove unvested slashing logic from addGrant

* feat(vesting): add CreateVesitngAccountCmd to cli

* feat(vesting): add types/utils test

* fix gosec

Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com>

* fix(vesting): call correct staking hooks (#285)

* fix(vesting): call correct staking hooks

* fix(vesting): refactor stakinghooks interface

* docs: testnet fixes (#288)

* Update faucet.md

* Update join.md

* docs: update testnet docs (#289)

* feat(vesting): ENG-385 add ante decorators (#286)

* add eth ante

* merge main

* wip

* simplify test

* refactor integration ante test

* fix lint

* remove comments

* add EthVestingTransactionDecorator description`

* add CHANGELOG

* vesting: delegation and gov ante handler (#287)

* feat(vesting): refactor types/msg_test

* feat(vesting): refactor period tests

* feat(vesting): refactor clawback vesting account tests

* feat(vesting): remove init account helper

* feat(vesting): refactore computeClawback tests to TDD

* feat(vesting): wip msg_server_test.go

* feat(vesting): add msg server tests with one open TODO

* feat(vesting): rename clawback address -> accountAddress

* feat(vesting): replace amino codec with cdc in module.go

* vesting delegation first approach

* integration tests

* fix delegation vesting

* governance ante decorator

* test gov vote and cleanup

* delete re-added lines

Co-authored-by: Daniel Burckhardt <daniel.m.burckhardt@gmail.com>

* integration(vesting):refactor perform ethTx into separate function

* cleanup ante folder

* fix lint

* errors and comments

* address some comments

* remove governance ante decorator

* replace ethante with new Handler options on evmos

* refactor ante vesting errors

Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>

* Update config.go

* build(deps): bump golangci/golangci-lint-action from 2.5.2 to 3 (#290)

Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 2.5.2 to 3.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](golangci/golangci-lint-action@v2.5.2...v3)

---
updated-dependencies:
- dependency-name: golangci/golangci-lint-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* deps: ethermint upgrade to v0.10.0 (#293)

* Update disk_optimization.md (#294)

* feat(vesting): grpc queries locked, unvested, vested (#291)

* wip vesting queries

* wip add queries

* add grpc query tests

* fix lint

* feat(vesting): add LockedOnly method

* feat(vesting): refactor queries

* fix indentation

* feat(vesting): adress comments

* bug(inflation): multiply daily epoch mint provision calculation by Power reduction (#295)

* bug(inflation): multiply daily epoch mint provision calculation by Power reduction

* bug(inflation): fix tests

* bug(inflation): fix comment

* bug(inflation): use sdk.DefaultPowerReduction

* bug(inflation): fix genesis tests

* vesting: refactor vesting query (#297)

* wip

* bug(vesting): Refactor UnvestedOnly logic

* bug(vesting): clean PR from unrelated changes

* bug(vesting): delete unused genesis proto

* bug(vesting): refactor comments

* Update x/vesting/types/clawback_vesting_account.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

* vesting: register gRPC web (#299)

* claims: ignore vesting accounts on clawback (#298)

* swagger

* swagger: add IBC and ethermint (#301)

* swagger: add IBC and ethermint

* add missing modules

* fix inflation

* lint

* fix tests

* ante: add validator min commission decorator (#302)

* ante: add validator min commission decorator

* changelog

* rm decorator

* rm decorator eip712

* cmd: add genaccount flags for vesting with clawback (#303)

* bug(vesting): add clawback vesting account as add-genaccount evmosd cmd

* bug(vesting): add to init.sh

* bug(vesting): refactor into switch statement

* bug(vesting): remove debugging from init.sh

* bug(vesting): update flag description

* bug(vesting): fix init.sh total supply

* bug(vesting): clean init.sh

* bug(vesting): refactor `HasLockedCoins`  (#306)

* bug(vesting): fail delegation if account is locked

* bug(vesting): fix error message

* bug(vesting): make locked && vested coins delegatable again

* docs: add cd step to join doc (#307)

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

Co-authored-by: Rajiv Patel-O'Connor <rajiv.patel.oconnor@gmail.com>
Co-authored-by: Daniel Burckhardt <daniel.m.burckhardt@gmail.com>
Co-authored-by: Prajjwol Gautam <prajjwol@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Guillermo Paoletti <guillermo.paoletti@gmail.com>
Co-authored-by: Ramiro Carlucho <ramirocarlucho@gmail.com>
Co-authored-by: Prajjwol Gautam <not@acyb.org>
Co-authored-by: litvinsky <60661362+litvinsky@users.noreply.github.com>
Co-authored-by: Joe Schmoe <64335177+jolube@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants