1269 read only execute sc

[Thykof]

Delivery Quality Checklist

• Breaking Changes in API:
  Does this PR introduce breaking changes in the API?

  • If yes, have you considered making it backward compatible?
  • If backward compatibility is not considered, set the "breaking-change" label.

• Changelog:

  • For bugfix PR, set the "bugfix" label
  • If this change should not appear in changelog, use "ignore-for-changelog" label

• Version Update Handling:
  Have you ensured that the version update by user is handled correctly?

• PR Dependency:
  Does this PR depend on another PR?

  • If yes, is it necessary for the dependency to be released prior to merging this one?

• Documentation:

  • Are any necessary changes made to user-facing documentation?
  • Confirm that API documentation is updated with any relevant changes.
  • Check that README and other documentation files are accurate and current.

[codecov]

Codecov Report

Attention: 65 lines in your changes are missing coverage. Please review.

Comparison is base (c39ddb9) 17.59% compared to head (8f948c7) 23.14%.

Files	Patch %	Lines
pkg/node/sendoperation/readonlycall.go	0.00%	64 Missing ⚠️
pkg/node/sendoperation/interfaces.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1273      +/-   ##
==========================================
+ Coverage   17.59%   23.14%   +5.55%     
==========================================
  Files          24       28       +4     
  Lines        1466     1702     +236     
==========================================
+ Hits          258      394     +136     
- Misses       1196     1295      +99     
- Partials       12       13       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[peterjah]

from the swagger file:

              maxGas:
                $ref: "#/definitions/Amount"
                description: Maximum number of gas unit that a operation will be able consume.
                default: 700000000 # DefaultGasLimit

  Amount:
    description: Amount in nanoMassa.
    type: string
    x-nullable: false

I think there is better to do here.
I suggest to allow amount to be nullable here, and if null, do the estimation.
If we don't want to change the behaviour everywhere, you can create a new swagger type 'nullableAmount' by instance...