Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MT Cannon: add mips movf/movt tests #12392

Merged
merged 2 commits into from
Oct 11, 2024

Conversation

GrapeBaBa
Copy link
Contributor

Description

This PR add mips movf/movt tests

Tests

Additional context

Metadata

@GrapeBaBa GrapeBaBa requested review from a team as code owners October 9, 2024 14:59
@GrapeBaBa GrapeBaBa requested a review from Inphi October 9, 2024 14:59
Copy link

codecov bot commented Oct 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.39%. Comparing base (fb62380) to head (0261205).
Report is 14 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12392      +/-   ##
===========================================
+ Coverage    64.32%   64.39%   +0.06%     
===========================================
  Files           52       52              
  Lines         4348     4356       +8     
===========================================
+ Hits          2797     2805       +8     
- Misses        1376     1377       +1     
+ Partials       175      174       -1     
Flag Coverage Δ
cannon-go-tests 64.39% <100.00%> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
cannon/mipsevm/testutil/state.go 73.91% <100.00%> (+1.95%) ⬆️

... and 3 files with indirect coverage changes

@GrapeBaBa
Copy link
Contributor Author

@Inphi @mbaxter How can I rerun the build? I don't know why my test code affect another op-e2e-test.

Copy link
Contributor

@mbaxter mbaxter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - just a few nits (optional suggestions).

cannon/mipsevm/tests/evm_common_test.go Outdated Show resolved Hide resolved
cannon/mipsevm/tests/evm_common_test.go Outdated Show resolved Hide resolved
cannon/mipsevm/tests/evm_common_test.go Outdated Show resolved Hide resolved
cannon/mipsevm/tests/evm_common_test.go Show resolved Hide resolved
@GrapeBaBa GrapeBaBa requested a review from a team as a code owner October 11, 2024 12:06
Signed-off-by: Chen Kai <[email protected]>
@mbaxter mbaxter added this pull request to the merge queue Oct 11, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 11, 2024
@mbaxter mbaxter added this pull request to the merge queue Oct 11, 2024
Merged via the queue into ethereum-optimism:develop with commit a5d0e30 Oct 11, 2024
58 checks passed
samlaf pushed a commit to samlaf/optimism that referenced this pull request Nov 10, 2024
* feat:add movf/movt tests

Signed-off-by: Chen Kai <[email protected]>

* fix:fix code review nit

Signed-off-by: Chen Kai <[email protected]>

---------

Signed-off-by: Chen Kai <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MTCannon: Create movf/movt opcodes tests
2 participants