4-nodes automated integration test infrastructure #46

psy2848048 · 2022-11-23T08:11:56Z

Effects

Soft update
Breaking change
Chain fork related

Major Reviewer

Background

No actual automated integration test
No Dockerfile yet
Actual integration test script - wasm #47 should be reviewed and merge first

Summary

Dockerfile provided
docker-compose provided (only for the multinode integration test purpose) with 4 validator users + 2 genesis users
As this PR contains too many lines, PR will be splitted with the infra setting script & actual tests code

Checklist

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

integration_test/grpcclient_test.go

jhlee-young

LGTM

psy2848048 · 2022-11-28T08:25:30Z

FYI

#48 is an improvement item, but it needs enough time to implement
Leave it as an issue and will update

psy2848048 · 2022-11-29T08:21:21Z

Tested on hypercube, and this test only works on dimension & cube
Proved that #48 should be fixed first for moving forward

Let me close this PR and reopen it

psy2848048 · 2022-12-01T11:38:14Z

The commit resolving #48 is added and reopen this PR

resolves #48

integration_test/validator1/app.toml

integration_test/wallet_test.go

* 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

* 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

* 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>

psy2848048 added 6 commits November 9, 2022 17:50

feat: added working Dockerfile

8c2615a

test: multinode docker & docker-compose

388b0bd

misc: added integration test in github action

bba0d5d

misc: divided unit test & integration test

9e95226

fix: added ETH port

ba74499

test: added basis lib

e95271a

psy2848048 requested review from jhlee-young, jbamlee, yoosah and JoowonYun November 23, 2022 08:11

psy2848048 self-assigned this Nov 23, 2022

psy2848048 force-pushed the test/integration branch from d681b53 to 616043d Compare November 23, 2022 08:18

misc: added test trigger

f0c40fe

psy2848048 force-pushed the test/integration branch from 616043d to f0c40fe Compare November 23, 2022 08:26

jhlee-young reviewed Nov 27, 2022

View reviewed changes

integration_test/grpcclient_test.go Show resolved Hide resolved

jhlee-young approved these changes Nov 28, 2022

View reviewed changes

psy2848048 mentioned this pull request Nov 28, 2022

Pour integration test codes into hypercube #49

Closed

6 tasks

psy2848048 added 6 commits November 28, 2022 18:27

test: added a test - delegation

e74bbb9

test: added test trigger

a727d9e

test: added tests: store code

fe36410

test: added test - InstantiateContract

b77ccbe

test: added test - contract execution

451f189

docs: added README of integration test

5594ecf

psy2848048 force-pushed the test/integration branch from 93a4636 to 396384d Compare November 28, 2022 10:08

misc: added go setup before integration test

fac1c6c

psy2848048 force-pushed the test/integration branch from 396384d to fac1c6c Compare November 28, 2022 10:26

psy2848048 closed this Nov 29, 2022

psy2848048 reopened this Dec 1, 2022

psy2848048 force-pushed the test/integration branch from 1224512 to e204ff2 Compare December 1, 2022 13:00

psy2848048 linked an issue Dec 1, 2022 that may be closed by this pull request

Automate Gentx & genesis.json creation in CI #48

Closed

test: automated all prepared genesis.json creation process

209a7eb

resolves #48

psy2848048 force-pushed the test/integration branch from e204ff2 to 209a7eb Compare December 1, 2022 13:14

JoowonYun reviewed Dec 2, 2022

View reviewed changes

integration_test/validator1/app.toml Show resolved Hide resolved

JoowonYun reviewed Dec 2, 2022

View reviewed changes

integration_test/wallet_test.go Outdated Show resolved Hide resolved

psy2848048 force-pushed the test/integration branch from 7cb4d6c to 4c31279 Compare December 2, 2022 07:42

test: changed broadcast mode as block and delete sleep

a853af5

psy2848048 force-pushed the test/integration branch from 4c31279 to a853af5 Compare December 2, 2022 16:00

JoowonYun approved these changes Dec 5, 2022

View reviewed changes

psy2848048 merged commit ff253e9 into main Dec 5, 2022

psy2848048 deleted the test/integration branch December 20, 2022 07:28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

4-nodes automated integration test infrastructure #46

4-nodes automated integration test infrastructure #46

psy2848048 commented Nov 23, 2022 •

edited

Loading

jhlee-young left a comment

psy2848048 commented Nov 28, 2022

psy2848048 commented Nov 29, 2022

psy2848048 commented Dec 1, 2022

4-nodes automated integration test infrastructure #46

4-nodes automated integration test infrastructure #46

Conversation

psy2848048 commented Nov 23, 2022 • edited Loading

Effects

Major Reviewer

Background

Summary

Checklist

jhlee-young left a comment

Choose a reason for hiding this comment

psy2848048 commented Nov 28, 2022

psy2848048 commented Nov 29, 2022

psy2848048 commented Dec 1, 2022

psy2848048 commented Nov 23, 2022 •

edited

Loading