test(oogx): CALL different scenarios and gas accumulator

[lorenzogentile404]

Addressing this too here #1663.

[OlivierBBB]

WRT

linea-tracer/arithmetization/src/test/java/net/consensys/linea/zktracer/exceptions/OutOfGasExceptionTest.java

Line 162 in 41a0c0b

} else {

The method can definitely be simplified (e.g. there is no need for nested if .. else ...) Also it is missing memory expansion costs. Maybe we could use the static Hub.GAS_PROJECTOR's methods ?

[lorenzogentile404]

@OlivierBBB using GasProjector requires passing the frame, so executing the program. In this case, we need to know the cost before execution. That is the reason why I used that approach. For memory expansion, during execution we end up using a method of FrontierGasCalculator.class:

  public long memoryExpansionGasCost(MessageFrame frame, long offset, long length) {
    long pre = memoryCost((long)frame.memoryWordSize());
    long post = memoryCost(frame.calculateMemoryExpansion(offset, length));
    return post == Long.MAX_VALUE ? Long.MAX_VALUE : post - pre;
  }

that also requires the frame. Let's have a chat about it.

[cla-assistant]

All committers have signed the CLA.

[cla-assistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[amkCha]

seen with @lorenzogentile404 Extract and merge tests for calls and leave the rest for a separate PR for gas cost in case of memory expansion

[sonarqubecloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud