Skip to content

Commit

Permalink
Ok/6984 update e2e network tests (#7276)
Browse files Browse the repository at this point in the history
* Update get_proof.test.ts

* test

* fix get storage

* Update e2e_network_tests.yml

* fix GA file

* done

* revert
  • Loading branch information
avkos authored Sep 24, 2024
1 parent aa471e7 commit c602fc6
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 15 deletions.
20 changes: 12 additions & 8 deletions .github/workflows/e2e_network_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,12 @@ on:
- release/**
tags:
- v4.*

# For test purpose only:
# pull_request:
# branches:
# - '4.x'
# types: [ opened, reopened, synchronize ]
jobs:
build:
name: Build Packages
Expand All @@ -16,11 +22,10 @@ jobs:
node-version: 18
cache: yarn
- run: yarn
- run: tar -czf /tmp/web3-16.js.tar.gz --exclude="./.git" ./
- uses: actions/upload-artifact@v3
- uses: actions/cache/save@v4
with:
name: web3-16.js.tar.gz
path: /tmp/web3-16.js.tar.gz
path: ./
key: web3-e2e-18-${{github.sha}}
e2e:
name: E2E Network Tests
needs: build
Expand All @@ -41,11 +46,10 @@ jobs:
- uses: actions/setup-node@v3
with:
node-version: 18
- uses: actions/download-artifact@v3
- uses: actions/cache/restore@v4
with:
name: web3-16.js.tar.gz
path: /tmp
- run: tar -xf /tmp/web3-16.js.tar.gz -C ./
path: ./
key: web3-e2e-18-${{github.sha}}
- run: |
yarn test:e2e:${{ matrix.network }}:${{ matrix.mode }}
shell: bash
4 changes: 2 additions & 2 deletions packages/web3/test/e2e/get_proof.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -55,9 +55,9 @@ describe(`${getSystemTestBackend()} tests - getProof`, () => {
| 'blockNumber';
}>({
block: [
'earliest',
// 'earliest', block is earlier than 128 blocks ago "Returned error: missing trie node"
'latest',
'pending',
// 'pending', block is not available "Returned error: missing trie node"
'safe',
'finalized',
'blockHash',
Expand Down
9 changes: 4 additions & 5 deletions packages/web3/test/e2e/mainnet/get_storage_at.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,9 @@ describe(`${getSystemTestBackend()} tests - getStorageAt`, () => {
it.each(
toAllVariants<{
storageSlot: Numbers;
block:
| 'earliest'
| 'latest'
| 'pending'
block: // | 'earliest'
| 'latest'
// | 'pending'
| 'finalized'
| 'safe'
| 'blockHash'
Expand Down Expand Up @@ -80,7 +79,7 @@ describe(`${getSystemTestBackend()} tests - getStorageAt`, () => {
} else if (block === 'blockHash' || block === 'blockNumber') {
// eslint-disable-next-line jest/no-conditional-expect
expect(result).toBe(
'0x00000000000000000000000000000000000000000000000000b8b61e3be91403',
'0x00000000000000000000000000000000000000000000000000c354b137cba7ba',
);
} else {
// eslint-disable-next-line jest/no-conditional-expect
Expand Down

1 comment on commit c602fc6

@github-actions
Copy link

Choose a reason for hiding this comment

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

Benchmark

Benchmark suite Current: c602fc6 Previous: aa471e7 Ratio
processingTx 22432 ops/sec (±6.29%) 21158 ops/sec (±9.13%) 0.94
processingContractDeploy 39716 ops/sec (±8.35%) 39080 ops/sec (±6.04%) 0.98
processingContractMethodSend 15739 ops/sec (±9.03%) 15073 ops/sec (±10.51%) 0.96
processingContractMethodCall 28613 ops/sec (±6.55%) 27542 ops/sec (±6.80%) 0.96
abiEncode 45466 ops/sec (±7.38%) 41986 ops/sec (±6.88%) 0.92
abiDecode 30872 ops/sec (±7.68%) 29298 ops/sec (±7.95%) 0.95
sign 1492 ops/sec (±3.27%) 1506 ops/sec (±0.77%) 1.01
verify 369 ops/sec (±0.47%) 357 ops/sec (±3.31%) 0.97

This comment was automatically generated by workflow using github-action-benchmark.

Please sign in to comment.