feat(entropy-sdk): add MockEntropy contract for local testing #3121
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=80&v=4){kind=small}
- Implements IEntropyV2 interface with two-step request/reveal pattern - requestV2() stores request and returns sequence number - mockReveal() allows manual callback triggering with custom random numbers - Enables testing different interleavings of requests and reveals - Includes comprehensive test suite demonstrating usage patterns - Simplifies testing by removing fee requirements and commitment validation Co-Authored-By: Jayant <[email protected]>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
jayantk
reviewed
Oct 10, 2025
| defaultProvider = _defaultProvider; | ||
|
|
||
| providers[_defaultProvider].sequenceNumber = 1; | ||
| providers[_defaultProvider].feeInWei = 0; |
There was a problem hiding this comment.
please initialize this to 1
- Set feeInWei to 1 in MockEntropy constructor and _requestV2 - Update package.json version from 2.0.0 to 2.1.0 - Update test assertion to expect feeInWei = 1 Co-Authored-By: Jayant <[email protected]>
14 tasks
tejasbadadare
approved these changes
Oct 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds
MockEntropycontract implementing theIEntropyV2interface to enable local testing of entropy-dependent contracts. The mock supports a two-phase request/reveal pattern where developers can control the timing and values of random number callbacks.Rationale
Developers need a way to test entropy-based contracts locally without depending on actual entropy providers. The key requirement was enabling testing of different interleavings - developers can request multiple random numbers and then reveal them in any order with custom values.
Two-phase testing pattern:
requestV2()→ returns sequence number, stores requestmockReveal(provider, sequenceNumber, customRandomBytes)→ triggers callback with developer-specified random valueThis allows testing scenarios like request A, request B, reveal B first, then reveal A with different random values.
Key Implementation Details
feeInWeiinitialized to 1 in provider info, butgetFeeV2()methods return 0 for testing simplicity_entropyCallbackRequestedandRevealedevents consistent with real Entropy contractHow has this been tested?
Test scenarios include:
Human Review Checklist
IEntropyV2methods are properly implementedfeeInWei = 1andgetFeeV2() = 0Link to Devin run: https://app.devin.ai/sessions/0d7e089bd7eb4ef5bd60d3baff368b69
Requested by: Jayant ([email protected])
Package version bumped:
@pythnetwork/entropy-sdk-solidityfrom 2.0.0 → 2.1.0