update rpc-tests to version 1.7 and exception list (#12656)

Co-authored-by: Michelangelo Riccobene <[email protected]>

.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
 

.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 $?
+exit $?