diff --git a/.github/workflows/qa-rpc-integration-tests.yml b/.github/workflows/qa-rpc-integration-tests.yml index 74cf9e0aca1..57a7a0c8fdb 100644 --- a/.github/workflows/qa-rpc-integration-tests.yml +++ b/.github/workflows/qa-rpc-integration-tests.yml @@ -32,7 +32,7 @@ jobs: - name: Checkout RPC Tests Repository & Install Requirements run: | rm -rf ${{ runner.workspace }}/rpc-tests - git -c advice.detachedHead=false clone --depth 1 --branch v1.0.0 https://github.com/erigontech/rpc-tests ${{runner.workspace}}/rpc-tests + git -c advice.detachedHead=false clone --depth 1 --branch v1.7.0 https://github.com/erigontech/rpc-tests ${{runner.workspace}}/rpc-tests cd ${{ runner.workspace }}/rpc-tests pip3 install -r requirements.txt diff --git a/.github/workflows/scripts/run_rpc_tests.sh b/.github/workflows/scripts/run_rpc_tests.sh old mode 100644 new mode 100755 index 0f94f68c039..b8e7f7c0dff --- a/.github/workflows/scripts/run_rpc_tests.sh +++ b/.github/workflows/scripts/run_rpc_tests.sh @@ -6,13 +6,23 @@ set +e # Disable exit on error disabled_tests=( # Erigon2 and Erigon3 never supported this api methods trace_rawTransaction - # false positives: Erigon return expected response. but rpc-test somehow doesn't see 1 field. - erigon_getHeaderByHash,erigon_getHeaderByNumber,eth_feeHistory + # remove these lines after https://github.com/erigontech/rpc-tests/pull/284 and https://github.com/erigontech/erigon/pull/12655 + eth_feeHistory/test_12.json + eth_feeHistory/test_13.json + eth_feeHistory/test_15.json + eth_feeHistory/test_16.json + eth_feeHistory/test_17.json + eth_feeHistory/test_18.json + eth_feeHistory/test_19.json + eth_feeHistory/test_20.json # total difficulty field was removed, then added back - eth_getBlockByHash,eth_getBlockByNumber + # remove this line after https://github.com/erigontech/rpc-tests/pull/282 + eth_getBlockByHash/test_10.json + eth_getBlockByNumber/test_12.json # Erigon bugs debug_accountRange,debug_storageRangeAt # need update rpc-test - because Erigon is correct (@AskAlexSharov will do after https://github.com/erigontech/erigon/pull/12634) + # remove this line after https://github.com/erigontech/rpc-tests/pull/273 debug_getModifiedAccountsByHash,debug_getModifiedAccountsByNumber # Erigon bug https://github.com/erigontech/erigon/issues/12603 erigon_getLatestLogs,erigon_getLogsByHash/test_04.json @@ -25,8 +35,8 @@ disabled_tests=( debug_traceBlockByNumber/test_12.tar # remove this line after https://github.com/erigontech/rpc-tests/pull/281 parity_getBlockReceipts + parity_listStorageKeys/test_12.json # to investigate - debug_traceBlockByHash debug_traceCallMany/test_02.tar debug_traceCallMany/test_04.tar debug_traceCallMany/test_05.tar @@ -38,11 +48,13 @@ disabled_tests=( engine_exchangeTransitionConfigurationV1/test_01.json engine_getClientVersionV1/test_1.json erigon_getBalanceChangesInBlock - eth_createAccessList/test_16.json + trace_replayBlockTransactions/test_29.tar + # do these perhaps require Erigon up? admin_nodeInfo/test_01.json admin_peers/test_01.json erigon_nodeInfo/test_1.json eth_coinbase/test_01.json + eth_createAccessList/test_16.json eth_getTransactionByHash/test_02.json eth_getWork/test_01.json eth_mining/test_01.json @@ -51,48 +63,12 @@ disabled_tests=( eth_submitWork/test_1.json net_peerCount/test_1.json net_version/test_1.json - txpool_content/test_01.json txpool_status/test_1.json - web3_clientVersion/test_1.json - eth_estimateGas/test_14.json - trace_replayBlockTransactions/test_29.tar - # recently started to fail - debug_traceTransaction/test_20.json - debug_traceTransaction/test_21.json - debug_traceTransaction/test_22.json - debug_traceTransaction/test_25.json - debug_traceTransaction/test_30.tar - debug_traceTransaction/test_33.json - debug_traceTransaction/test_35.tar - debug_traceTransaction/test_36.json - debug_traceTransaction/test_37.tar - debug_traceTransaction/test_38.tar - debug_traceTransaction/test_43.json - debug_traceTransaction/test_44.json - debug_traceTransaction/test_62.json - debug_traceTransaction/test_64.json - debug_traceTransaction/test_74.tar - debug_traceTransaction/test_75.tar - debug_traceTransaction/test_77.json - debug_traceTransaction/test_78.tar - debug_traceTransaction/test_79.tar - debug_traceTransaction/test_80.tar - debug_traceTransaction/test_81.tar - debug_traceTransaction/test_82.tar - debug_traceTransaction/test_83.tar - debug_traceTransaction/test_84.tar - debug_traceTransaction/test_85.tar - debug_traceTransaction/test_87.json - debug_traceTransaction/test_90.tar - debug_traceTransaction/test_91.tar - debug_traceTransaction/test_92.tar - debug_traceTransaction/test_93.json - debug_traceTransaction/test_96.json - trace_filter/test_16.json) + web3_clientVersion/test_1.json) # Transform the array into a comma-separated string disabled_test_list=$(IFS=,; echo "${disabled_tests[*]}") python3 ./run_tests.py -p 8545 --continue -f --json-diff -x "$disabled_test_list" -exit $? \ No newline at end of file +exit $?