-
Notifications
You must be signed in to change notification settings - Fork 120
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
feat: add e2e test stateful contract #2703
feat: add e2e test stateful contract #2703
Conversation
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Base branches to auto review (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
7c7d22c
to
ca1d772
Compare
ca1d772
to
b9878a6
Compare
8685c21
to
40ff210
Compare
75ccd15
to
83457fc
Compare
c3ef341
to
e4d43d4
Compare
8b95bfb
to
58b865a
Compare
58b865a
to
eaac523
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, just minor comments
e59d239
to
3fa0cab
Compare
efd7f51
to
b62a99e
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feat/stateful-precompiled-contracts #2703 +/- ##
=======================================================================
+ Coverage 70.35% 70.38% +0.02%
=======================================================================
Files 338 342 +4
Lines 18599 18745 +146
=======================================================================
+ Hits 13085 13193 +108
- Misses 4922 4955 +33
- Partials 592 597 +5
|
974fef1
to
ac48e29
Compare
* chore: cleanup Makefile and go.mod * feat: introduce stateful precompiles initial support * chore: remove simapp as a dependency * fix: adapt to testutil to changes * fix: linting * chore: add changelog and remove unused vars * fix: create aggregateAllKeys function * feat: add e2e test stateful contract (#2703) * feat: add test precompiled contract * feat: add test stateful contract * feat: enable contracts granularly * feat: allow passing the app ctx to contracts * revert ctx changes * format ABI * add first e2e test * add RegularCaller contract to e2e * add signer to deployment * fix e2e, use regular abi * minor rewording * use idiomatic errors * remove unused files * rename and generate bindings automatically * create a new e2e section for precompiles * add bindings generator and makefile target * add GetGasStabilityPoolBalance as prototype contract function * include getGasStabilityPoolBalance function * apply code review fixes * include tests for precompiles package * delete regularcaller * ignore bindings from codecov * introduce more unit testing * add unit testing for prototype.go * minor linting fixes * remove ctx, keep fungible keeper private * increase unit tests coverage * add keys for new user_precompile * add init unit test * switch to double quotes * avoid go:embed * fixes derived from code review * refactor some parts of bech32 functions * formatting * new generated mocks * skip precompile tests during upgrade * skip precompile * try to fix generate * debug script * install solc in generate files * install abigen * cleanup action * PR comments pt1 * group prototype unit tests a bit * PR comment missing test * cleanup action a bit * add solc checks to script * cleanup not needed changes * revert ibc change in test * remove new test * reduce simple operations gas required to 500 --------- Co-authored-by: skosito <[email protected]> Co-authored-by: Dmitry S <[email protected]>
Description
Include the first test stateful contract.
How Has This Been Tested?