axonweb3 / axon

Axon is a Layer 2 framework of CKB with native cross-chain and interoperability.
https://axonweb3.io
MIT License
65 stars 39 forks source link

fix: rpc method `eth_call` shouldn't set `estimate = true` #1609

Closed yangby-cryptape closed 9 months ago

yangby-cryptape commented 9 months ago

What this PR does / why we need it?

This PR resolves a bug introduced since #1603: AxonExcutor::call(..) is not only used in eth_estimateGas, but also in eth_call.

What is the impact of this PR?

No Breaking Change

CI Settings
### **CI Usage** **Tip**: Check the CI you want to run below, and then comment `/run-ci`. **CI Switch** - [x] Web3 Compatible Tests - [x] OCT 1-5 And 12-15 - [x] OCT 6-10 - [x] OCT 11 - [x] OCT 16-19 - [x] v3 Core Tests ### **CI Description** | CI Name | Description | | ----------------------------------------- | ------------------------------------------------------------------------- | | *Web3 Compatible Test* | Test the Web3 compatibility of Axon | | *v3 Core Test* | Run the compatibility tests provided by Uniswap V3 | | *OCT 1-5 \| 6-10 \| 11 \| 12-15 \| 16-19* | Run the compatibility tests provided by OpenZeppelin |