Skip to content
This repository has been archived by the owner on Jan 9, 2025. It is now read-only.

Add test case for issue 683 & 684 #767

Merged
merged 3 commits into from
Oct 11, 2023

Conversation

ClementWalter
Copy link
Member

Time spent on this PR: 0.05

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?

Kakarot doesn't revert when doing 0 - 1 for uint256.

There was already an issue, I just added a test.

What is the new behavior?

Still not passing

@ClementWalter ClementWalter changed the title Add failing test case for issue 683 Add failing test case for issue 683 & 684 Oct 10, 2023
@ClementWalter ClementWalter changed the title Add failing test case for issue 683 & 684 Add test case for issue 683 & 684 Oct 10, 2023
@ClementWalter ClementWalter force-pushed the cw/add-dec-example branch 2 times, most recently from f097b07 to 5deb2a4 Compare October 10, 2023 20:00
@codecov
Copy link

codecov bot commented Oct 10, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (f37e5fa) 69.40% compared to head (390b4d5) 69.33%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #767      +/-   ##
==========================================
- Coverage   69.40%   69.33%   -0.08%     
==========================================
  Files          42       42              
  Lines        6779     6779              
==========================================
- Hits         4705     4700       -5     
- Misses       2074     2079       +5     

see 1 file with indirect coverage changes

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

Copy link
Member

@Eikix Eikix left a comment

Choose a reason for hiding this comment

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

lgtm

@Eikix Eikix merged commit 7c1cd2c into kkrt-labs:main Oct 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants