Use opcode from context and OOG in Arithmetic Operations

[ClementWalter]

Time spent on this PR: 0.3

Pull request type

Please check the type of change your PR introduces:

• Bugfix
• Feature
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation content changes
• Other (please describe):

What is the current behavior?

All the Arithmetic opcodes use the same internal exec_arithmetic_operation function with an extra opcode
parameter.

What is the new behavior?

The opcode number is taken directly from the ctx and StopAndArithmeticOperations.exec_arithmetic_operation
is consequently directly used in evm.cairo.

OOG during these opcodes is now raised.

[codecov]

Codecov Report

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

Comparison is base (cf835ce) 70.83% compared to head (94db972) 70.61%.
Report is 2 commits behind head on main.

❗ Current head 94db972 differs from pull request most recent head c5054c1. Consider uploading reports for the commit c5054c1 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #782      +/-   ##
==========================================
- Coverage   70.83%   70.61%   -0.23%     
==========================================
  Files          45       45              
  Lines        6649     6551      -98     
==========================================
- Hits         4710     4626      -84     
+ Misses       1939     1925      -14     

Files	Coverage Δ
src/kakarot/evm.cairo	100.00% <ø> (ø)
.../instructions/stop_and_arithmetic_operations.cairo	79.61% <89.28%> (+3.34%)	⬆️
src/kakarot/errors.cairo	2.68% <0.00%> (-0.22%)	⬇️

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

[Eikix]

a couple of questions but lgtm
also should we add a todo to fix sign extend

[Eikix]

how come ef-tests are failing?

[Eikix]

lgtm