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.

Sign up for GitHub

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

Jump to bottom

Added integartion tests for XATP module #60

Merged
merged 4 commits into from
Mar 23, 2023
Merged

Added integartion tests for XATP module #60

merged 4 commits into from
Mar 23, 2023

Conversation

psy2848048
Copy link
Collaborator

@psy2848048 psy2848048 commented Dec 26, 2022 •
edited
Loading

Effects

  • Soft update
  • Breaking change
  • Chain fork related

Major Reviewer

Background

  • Need some proposal steps for testing XATP feature
  • Need to test both of simulation and tx propagation

Summary

  • Enabled swap for the XATP test prepare
  • Added proposal steps for the XATP test prepare
  • Added insufficient gas cases both of the native token & XATP token

Checklist

  • Backward compatible?
  • Test enough in your local environment?
  • Add related test cases?

@psy2848048 psy2848048 self-assigned this Dec 26, 2022
@psy2848048 psy2848048 changed the base branch from refactor/ante-fee to feature/xatp December 26, 2022 13:14
@psy2848048 psy2848048 changed the base branch from feature/xatp to refactor/ante-fee December 26, 2022 13:14
@psy2848048 psy2848048 force-pushed the test/xatp branch 2 times, most recently from 7b49318 to 8104bcc Compare December 27, 2022 01:08
@psy2848048 psy2848048 changed the title Added integartion tests for XATP module [DO NOT MERGE] Added integartion tests for XATP module Dec 27, 2022
@psy2848048
Copy link
Collaborator Author

Need to push the change from my local

Base automatically changed from refactor/ante-fee to feature/xatp March 8, 2023 06:58
@JoowonYun JoowonYun changed the base branch from feature/xatp to feature/xatp-economy March 20, 2023 08:52
@psy2848048 psy2848048 changed the title [DO NOT MERGE] Added integartion tests for XATP module Added integartion tests for XATP module Mar 22, 2023
@psy2848048
4-nodes automated integration test infrastructure (#46)
ba8338c 
* feat: added working Dockerfile

* test: multinode docker & docker-compose

* misc: added integration test in github action

* misc: divided unit test & integration test

* fix: added ETH port

* test: added basis lib

* misc: added test trigger

* test: added a test - delegation

* test: added test trigger

* test: added tests: store code

* test: added test - InstantiateContract

* test: added test - contract execution

* docs: added README of integration test

* misc: added go setup before integration test

* test: automated all prepared genesis.json creation process

resolves #48

* test: changed broadcast mode as block and delete sleep
@psy2848048
test: updated common testing part
930a895
@psy2848048
test: added xatp tests
d400f5f
JoowonYun
JoowonYun reviewed Mar 23, 2023
View reviewed changes
integration_test/entrypoint_master.sh Outdated Show resolved Hide resolved
integration_test/init_test.go Outdated Show resolved Hide resolved
integration_test/integration_test.go Outdated Show resolved Hide resolved
@psy2848048 psy2848048 merged commit e58389a into feature/xatp-economy Mar 23, 2023
@JoowonYun JoowonYun deleted the test/xatp branch March 24, 2023 02:15
JoowonYun added a commit that referenced this pull request Mar 30, 2023
@JoowonYun @green-blue0 @psy2848048
XATP module (#73)
e01b7c2 
* add xatp

add upgrade

* fix: correct proto path

* fix: correct module name

* fix: mempool fee decorator test

* refactor: remove unused code

* feat: changed to manage XATP with map

* feat: add to xatp query

* refactor: simple conditions and calculations

* feat: manage XATP as a proposal

* refactor: remove unused SmartQueryGasLimit

* refactor: ante MempoolFeeDecorator

* refactor: ante DeductFeeDecorator

* fix: exception handling when payer is empty in xatp module

* fix: fee precision

* feat: change payer to module account

* feat: apply tax-rate

* feat: xatp economy

* feat: fund xatp pool & query pool

* fix: typo

* fix: validate simulation

* refactor: merge upgrade (xatp & align gas price)

* feat: gas refund when mempool check

* refactor: increased precision in simulate

* fix: change XATP from dec to int

* fix: change the xatp param rate to be greater than 1

* fix: calculate reward pool

* fix: check fee when simulate

* Added integartion tests for XATP module (#60)

* 4-nodes automated integration test infrastructure (#46)

* feat: added working Dockerfile

* test: multinode docker & docker-compose

* misc: added integration test in github action

* misc: divided unit test & integration test

* fix: added ETH port

* test: added basis lib

* misc: added test trigger

* test: added a test - delegation

* test: added test trigger

* test: added tests: store code

* test: added test - InstantiateContract

* test: added test - contract execution

* docs: added README of integration test

* misc: added go setup before integration test

* test: automated all prepared genesis.json creation process

resolves #48

* test: changed broadcast mode as block and delete sleep

* test: updated common testing part

* test: added xatp tests

* test: fixed - get simulation error from sync mode

* refactor: variable names

---------

Co-authored-by: 임진승 <jinseungim@com2us.com>
Co-authored-by: psy2848048 <psy2848048@gmail.com>
JoowonYun added a commit that referenced this pull request Feb 1, 2024
@JoowonYun @green-blue0 @psy2848048
XATP module (#73)
7fbb56f 
* add xatp

add upgrade

* fix: correct proto path

* fix: correct module name

* fix: mempool fee decorator test

* refactor: remove unused code

* feat: changed to manage XATP with map

* feat: add to xatp query

* refactor: simple conditions and calculations

* feat: manage XATP as a proposal

* refactor: remove unused SmartQueryGasLimit

* refactor: ante MempoolFeeDecorator

* refactor: ante DeductFeeDecorator

* fix: exception handling when payer is empty in xatp module

* fix: fee precision

* feat: change payer to module account

* feat: apply tax-rate

* feat: xatp economy

* feat: fund xatp pool & query pool

* fix: typo

* fix: validate simulation

* refactor: merge upgrade (xatp & align gas price)

* feat: gas refund when mempool check

* refactor: increased precision in simulate

* fix: change XATP from dec to int

* fix: change the xatp param rate to be greater than 1

* fix: calculate reward pool

* fix: check fee when simulate

* Added integartion tests for XATP module (#60)

* 4-nodes automated integration test infrastructure (#46)

* feat: added working Dockerfile

* test: multinode docker & docker-compose

* misc: added integration test in github action

* misc: divided unit test & integration test

* fix: added ETH port

* test: added basis lib

* misc: added test trigger

* test: added a test - delegation

* test: added test trigger

* test: added tests: store code

* test: added test - InstantiateContract

* test: added test - contract execution

* docs: added README of integration test

* misc: added go setup before integration test

* test: automated all prepared genesis.json creation process

resolves #48

* test: changed broadcast mode as block and delete sleep

* test: updated common testing part

* test: added xatp tests

* test: fixed - get simulation error from sync mode

* refactor: variable names

---------

Co-authored-by: 임진승 <jinseungim@com2us.com>
Co-authored-by: psy2848048 <psy2848048@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JoowonYun JoowonYun JoowonYun approved these changes

@yoosah yoosah Awaiting requested review from yoosah

Assignees

@psy2848048 psy2848048

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@psy2848048 @JoowonYun