Artillery based stress testing is inaccurate due to misconfiguration

[CryptoFewka]

Describe the Bug
The artillery config uses at least one call which is improperly specified:

node/.github/actions/performance-tests/art.yaml

Lines 482 to 486 in 6e9dc53

	params:
	- fromBlock: "0x1"
	toBlock: "latest"
	address: "0x0"
	topics: ["0x0"]

This call results in HTTP status 200, but a body of:

{
  "jsonrpc": "2.0",
  "id": 1,
  "error": {
    "code": -32602,
    "message": "invalid argument 0: invalid address: hex string of odd length"
  }
}

To Reproduce

• Take the parameters in the eth_getLogs specification found here:

  node/.github/actions/performance-tests/art.yaml

  Lines 482 to 486 in 6e9dc53

  	params:
  	- fromBlock: "0x1"
  	toBlock: "latest"
  	address: "0x0"
  	topics: ["0x0"]

• Build a curl command based on them, like this:

curl -s -H "Content-Type: application/json" --data '{"jsonrpc":"2.0","method":"eth_getLogs","params":[{"fromBlock": "0x1","toBlock": "latest","address": "0x0","topics":["0x0"]}],"id":1}' 

• Run it against a zetacored node

Expected Behavior
The intent of the artillery stress test is to judge performance of individual methods. This intent is not met with the current configuration, and it instead measures the ability to recognize invalid input and respond with an error message quickly.

Environment (please complete the following information):

• Ubuntu 22.04 AMD64
• zetacored v18
• athens network

[CryptoFewka]

Perhaps we should add and use the expect plugin to ensure that the response doesn't contain an error property. I do not believe that the assert plugin exists, despite an attempt to define its config to try and catch this exact state.

Ref:

• https://www.artillery.io/docs/reference/extensions/expect
• expect's notHasProperty

Example Response:

{
  "jsonrpc": "2.0",
  "id": 1,
  "error": {
    "code": -32602,
    "message": "invalid argument 0: invalid address: hex string of odd length"
  }
}

Example config (last line contains change) that I would expect to fail on that response:

  - name: eth_getLogs
    flow:
      - post:
          url: "/"
          json:
            id: 1
            jsonrpc: "2.0"
            method: "eth_getLogs"
            params:
              - fromBlock: "0x1"
                toBlock: "latest"
                address: "0x0"
                topics: ["0x0"]
          capture:
            - json: "$"
              as: "response"
          ensure:
            - statusCode: 200
            - statusCode: 201
          expect:
            - notHasProperty: 'data[0].error'