chore(ts-tests): add e2e contracts #66

canonbrother · 2022-09-05T09:23:01Z

What this PR does / why we need it:

Develop and test on a simple smart contract functionality on substrate-based blockchain.

Which issue(s) does this PR fixes?:

Fixes #

Additional comments?:

move to constant

…ontracts

Bumps [actions/github-script](https://github.com/actions/github-script) from 6.1.0 to 6.1.1. - [Release notes](https://github.com/actions/github-script/releases) - [Commits](actions/github-script@v6.1.0...d50f485) --- updated-dependencies: - dependency-name: actions/github-script dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* syntax err * support multi platf

…ontracts

* syntax err * support multi platf

ts-tests/package.json

.github/workflows/ci.yml

ts-tests/package.json

ts-tests/containers/MetaDContainer.ts

ts-tests/__tests__/contract.tests.ts

weiyuan95 · 2022-09-08T10:00:40Z

Regarding the failing test on macos, it looks like running more than 3 containers at once isn't possible. Anything more than 3, it seems like the container crashes. Any idea why? @canonbrother

Some logs, if that helps. Container 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7 is the one that can't start properly.

2022-09-08T09:51:50.659Z testcontainers ERROR Failed to exec container 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7 with command "/bin/sh -c nc -vz -w 1 localhost 19933": Error: (HTTP code 409) container stopped/paused - Container 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7 is not running 
2022-09-08T09:51:50.659Z testcontainers ERROR Container failed to be ready: Error: (HTTP code 409) container stopped/paused - Container 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7 is not running 
2022-09-08T09:51:50.667Z testcontainers ERROR Failed to exec container 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7 with command "/bin/bash -c </dev/tcp/localhost/19933": Error: (HTTP code 409) container stopped/paused - Container 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7 is not running 
2022-09-08T09:51:50.667Z testcontainers ERROR Failed to exec container 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7 with command "/bin/sh -c cat /proc/net/tcp* | awk '{print $2}' | grep -i :4ddd": Error: (HTTP code 409) container stopped/paused - Container 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7 is not running 
2022-09-08T09:51:50.677Z testcontainers INFO  Container has already been stopped: 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7

canonbrother · 2022-09-08T10:05:36Z

Regarding the failing test on macos, it looks like running more than 3 containers at once isn't possible. Anything more than 3, it seems like the container crashes. Any idea why? @canonbrother

Some logs, if that helps. Container 1c528e9664a6fcefb9bf4e0e8d697d4a39ef0fd3bb24aec093aab8eb152276c7 is the one that can't start properly.

yep.. @DieHard073055 pinged me this issue.. the container stopped before the test finished while running >3 tests on MacOs... still checking on my end

weiyuan95 · 2022-09-09T06:08:21Z

Needs a rebase @canonbrother. Other than that LGTM other than the tests which seem to be failing only on macOS for some reason.

cwkang1998

LGTM

weiyuan95

LGTM

weiyuan95 · 2022-09-09T06:56:48Z

LGTM

Your approval has no power here! 🤣

ts-tests/package.json

canonbrother · 2022-09-12T05:48:58Z

Needs a rebase @canonbrother. Other than that LGTM other than the tests which seem to be failing only on macOS for some reason.

Exit Code 137: Indicates failures as container receivd SIGKILL (manual intervention or 'oom-killer' [OUT-OF-MEMORY]
assign more memory on Mac docker will do

weiyuan95 · 2022-09-12T06:13:52Z

Needs a rebase @canonbrother. Other than that LGTM other than the tests which seem to be failing only on macOS for some reason.

Exit Code 137: Indicates failures as container receivd SIGKILL (manual intervention or 'oom-killer' [OUT-OF-MEMORY] assign more memory on Mac docker will do

Makes sense, time to request for 256gb ram 😏

…pace design (#86) #### What this PR does / why we need it: Move `ts-tests/*` to utilize pnpm and turbo mono-repo workspace design. With this, we managed to keep configs (eslint, prettier, babel jest) DRY. The PR "workspace-ize" `ts-tests` but ultimately many more PR is required to break this package into multiple smaller mono repo packages. Follow-up PRs required to address these: - [ ] ESLINT will be added back once we move to "testsuite", also part of #79 - [ ] #78 --- **What's new?** As suggested in #66 (comment) I've added hardhat compile turbo workflow. `@birthdayresearch/sticky-turbo-jest` is a turbo-aware `jest-preset`; it automatically run `dependsOn` script before running jest. By taking advantage of content-aware hashing from turborepo. `dependsOn` only runs if the pipeline `inputs` has changed. When you run your test with `@birthdayresearch/sticky-turbo-jest` preset, it uses `displayName` from your jest config to figure out which `pipeline` to use. It will automatically pinpoint the corresponding `dependsOn` and run all the scripts specified within. > TLDR, when you run a dependent test, it automatically recompiles your contract with hardhat when it detects changes in your `contracts/*.sol` file. If it has the cached version, it will simply "output" the result instead of rebuilding it. https://sticky.birthday.dev/packages/sticky-turbo-jest `package.json`: ```json "scripts": { "compile": "hardhat compile", "test": "jest" }, "jest": { "displayName": "test", "preset": "@birthdayresearch/sticky-turbo-jest" }, ``` `turbo.json`: ```json "compile": { "inputs": ["contracts/**"], "outputs": ["artifacts", "cache"] }, "test": { "inputs": ["src/**", "**/*.unit.ts", "tests-e2e/**"], "dependsOn": ["^build", "compile"] }, ``` #### Which issue(s) does this PR fixes?:  Fixes part of #79

#### What this PR does / why we need it: Develop and test on a simple smart contract functionality on substrate-based blockchain. #### Which issue(s) does this PR fixes?:  Fixes # #### Additional comments?: Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…pace design (#86) #### What this PR does / why we need it: Move `ts-tests/*` to utilize pnpm and turbo mono-repo workspace design. With this, we managed to keep configs (eslint, prettier, babel jest) DRY. The PR "workspace-ize" `ts-tests` but ultimately many more PR is required to break this package into multiple smaller mono repo packages. Follow-up PRs required to address these: - [ ] ESLINT will be added back once we move to "testsuite", also part of #79 - [ ] #78 --- **What's new?** As suggested in #66 (comment) I've added hardhat compile turbo workflow. `@birthdayresearch/sticky-turbo-jest` is a turbo-aware `jest-preset`; it automatically run `dependsOn` script before running jest. By taking advantage of content-aware hashing from turborepo. `dependsOn` only runs if the pipeline `inputs` has changed. When you run your test with `@birthdayresearch/sticky-turbo-jest` preset, it uses `displayName` from your jest config to figure out which `pipeline` to use. It will automatically pinpoint the corresponding `dependsOn` and run all the scripts specified within. > TLDR, when you run a dependent test, it automatically recompiles your contract with hardhat when it detects changes in your `contracts/*.sol` file. If it has the cached version, it will simply "output" the result instead of rebuilding it. https://sticky.birthday.dev/packages/sticky-turbo-jest `package.json`: ```json "scripts": { "compile": "hardhat compile", "test": "jest" }, "jest": { "displayName": "test", "preset": "@birthdayresearch/sticky-turbo-jest" }, ``` `turbo.json`: ```json "compile": { "inputs": ["contracts/**"], "outputs": ["artifacts", "cache"] }, "test": { "inputs": ["src/**", "**/*.unit.ts", "tests-e2e/**"], "dependsOn": ["^build", "compile"] }, ``` #### Which issue(s) does this PR fixes?:  Fixes part of #79

canonbrother and others added 30 commits August 22, 2022 14:56

add e2e testcontainer

c923da8

br

df865a2

ts-test package ver 0.0.0

17fe3d8

test defich/metachain img

c6ec290

ci e2e

569ad80

merge conflicts

105b46a

fix ci e2e

23b6be9

rm ts-test/gitignore

8bd47b9

update bal.test.ts

e7b9301

use non-null assertion op

6727c93

move to constant

aee4952

move to constant

typing provider and call -> resp.result

ea8be11

wip

36f83d8

update image tag

9c424c9

syntax err

d3b2335

Merge branch 'main' into canonbrother/e2e-testcontainer

cd25102

proper docker conn

ed1f92c

Merge branch 'canonbrother/e2e-testcontainer' into canonbrother/e2e-c…

c58a2aa

…ontracts

proper docker conn

9f971f5

fix(workflow): docker supports multi platform (#38)

b7f9622

* syntax err * support multi platf

merge base

aa971eb

Merge branch 'main' into canonbrother/e2e-testcontainer

fb3c06a

Merge branch 'canonbrother/e2e-testcontainer' into canonbrother/e2e-c…

33023a6

…ontracts

proper docker conn

5f62c51

fix(workflow): docker supports multi platform (#38)

8506594

* syntax err * support multi platf

rename ethersjs to ethers

2cf6a00

add network differentiate container

cbda49a

extend timeout

2654b37

Merge branch 'main' into canonbrother/e2e-testcontainer

1d27ae2

weiyuan95 reviewed Sep 8, 2022

View reviewed changes

ts-tests/package.json Outdated Show resolved Hide resolved

cwkang1998 reviewed Sep 8, 2022

View reviewed changes

.github/workflows/ci.yml Outdated Show resolved Hide resolved

ts-tests/package.json Outdated Show resolved Hide resolved

ts-tests/containers/MetaDContainer.ts Outdated Show resolved Hide resolved

ts-tests/__tests__/contract.tests.ts Show resolved Hide resolved

canonbrother force-pushed the canonbrother/e2e-contracts branch from 79aef54 to 3b4919c Compare September 8, 2022 10:17

rm web3

2e4c2c9

canonbrother dismissed DieHard073055’s stale review via 2e4c2c9 September 8, 2022 10:36

canonbrother added 2 commits September 8, 2022 19:18

replace truffle by hardhat

658c256

combine hardhat compile with jest in one script

cc4c8bc

canonbrother force-pushed the canonbrother/e2e-contracts branch from caf569d to cc4c8bc Compare September 8, 2022 11:25

defichain-bot removed the area/workflow label Sep 8, 2022

fuxingloh requested review from weiyuan95, cwkang1998 and DieHard073055 and removed request for cwkang1998 September 9, 2022 04:16

Merge branch 'main' into canonbrother/e2e-contracts

2dc64c5

cwkang1998 approved these changes Sep 9, 2022

View reviewed changes

weiyuan95 approved these changes Sep 9, 2022

View reviewed changes

eli-lim approved these changes Sep 9, 2022

View reviewed changes

ts-tests/package.json Show resolved Hide resolved

fuxingloh approved these changes Sep 12, 2022

View reviewed changes

fuxingloh merged commit 31bb3b8 into main Sep 12, 2022

fuxingloh deleted the canonbrother/e2e-contracts branch September 12, 2022 07:03

fuxingloh mentioned this pull request Sep 13, 2022

chore(ts-tests): refactor ts-tests/* to follow pnpm and turbo workspace design #86

Merged

2 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore(ts-tests): add e2e contracts #66

chore(ts-tests): add e2e contracts #66

canonbrother commented Sep 5, 2022 •

edited

Loading

weiyuan95 commented Sep 8, 2022 •

edited

Loading

canonbrother commented Sep 8, 2022

weiyuan95 commented Sep 9, 2022

cwkang1998 left a comment

weiyuan95 left a comment

weiyuan95 commented Sep 9, 2022

canonbrother commented Sep 12, 2022

weiyuan95 commented Sep 12, 2022

chore(ts-tests): add e2e contracts #66

chore(ts-tests): add e2e contracts #66

Conversation

canonbrother commented Sep 5, 2022 • edited Loading

What this PR does / why we need it:

Which issue(s) does this PR fixes?:

Additional comments?:

weiyuan95 commented Sep 8, 2022 • edited Loading

canonbrother commented Sep 8, 2022

weiyuan95 commented Sep 9, 2022

cwkang1998 left a comment

Choose a reason for hiding this comment

weiyuan95 left a comment

Choose a reason for hiding this comment

weiyuan95 commented Sep 9, 2022

canonbrother commented Sep 12, 2022

weiyuan95 commented Sep 12, 2022

canonbrother commented Sep 5, 2022 •

edited

Loading

weiyuan95 commented Sep 8, 2022 •

edited

Loading