Skip to content

chore(cheats): make sign(Wallet) pure#12912

Merged
DaniPopes merged 7 commits intomasterfrom
dani/sign-wallet-pure
Dec 30, 2025
Merged

chore(cheats): make sign(Wallet) pure#12912
DaniPopes merged 7 commits intomasterfrom
dani/sign-wallet-pure

Conversation

@DaniPopes
Copy link
Member

Same as the equivalent fns without Wallet

onbjerg
onbjerg previously approved these changes Dec 22, 2025
mattsse
mattsse previously approved these changes Dec 22, 2025
@DaniPopes DaniPopes dismissed stale reviews from onbjerg and mattsse via f8bf4d4 December 28, 2025 22:43
@DaniPopes DaniPopes requested review from mattsse and onbjerg December 28, 2025 22:43
@DaniPopes DaniPopes added this pull request to the merge queue Dec 29, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 29, 2025
@DaniPopes DaniPopes added this pull request to the merge queue Dec 29, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 29, 2025
@grandizzy grandizzy added this pull request to the merge queue Dec 29, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 29, 2025
@DaniPopes DaniPopes added this pull request to the merge queue Dec 29, 2025
@grandizzy
Copy link
Collaborator

@DaniPopes this should help to solve sepolia drpc flaky #12945

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 29, 2025
@grandizzy grandizzy enabled auto-merge December 29, 2025 12:37
@grandizzy grandizzy added this pull request to the merge queue Dec 29, 2025
github-merge-queue bot pushed a commit that referenced this pull request Dec 29, 2025
* chore(cheats): make sign(Wallet) pure

* ignore

---------

Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Co-authored-by: grandizzy <38490174+grandizzy@users.noreply.github.com>
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 29, 2025
@DaniPopes DaniPopes added this pull request to the merge queue Dec 29, 2025
github-merge-queue bot pushed a commit that referenced this pull request Dec 29, 2025
* chore(cheats): make sign(Wallet) pure

* ignore

---------

Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Co-authored-by: grandizzy <38490174+grandizzy@users.noreply.github.com>
@DaniPopes DaniPopes removed this pull request from the merge queue due to a manual request Dec 29, 2025
@DaniPopes
Copy link
Member Author

theres an actual test failure but due to seed changing (?)

    Encountered 1 failing test in default/repros/Issue2851.t.sol:Issue2851Test
    [FAIL: <empty revert data>]
    	[Sequence] (original: 3, shrunk: 1)
    		sender=0x0000000000000000000000000000000000001DC0 addr=[default/repros/Issue2851.t.sol:Backdoor]0x5615dEB798BB3E4dFa0139dFa1b3D433Cc23b72f calldata=backdoor(uint256) args=[6912213124124532 [6.912e15]]
     invariantNotZero() (runs: 217, calls: 3255, reverts: 35)

what should i do @grandizzy ?

@grandizzy
Copy link
Collaborator

theres an actual test failure but due to seed changing (?)

    Encountered 1 failing test in default/repros/Issue2851.t.sol:Issue2851Test
    [FAIL: <empty revert data>]
    	[Sequence] (original: 3, shrunk: 1)
    		sender=0x0000000000000000000000000000000000001DC0 addr=[default/repros/Issue2851.t.sol:Backdoor]0x5615dEB798BB3E4dFa0139dFa1b3D433Cc23b72f calldata=backdoor(uint256) args=[6912213124124532 [6.912e15]]
     invariantNotZero() (runs: 217, calls: 3255, reverts: 35)

what should i do @grandizzy ?

checking, weird that it happens for win only ...

@grandizzy
Copy link
Collaborator

interesting seems we have this test twice, once here

function invariantNotZero() public {

and once in

/// forge-config: default.fuzz.dictionary.max_fuzz_dictionary_literals = 0
/// forge-config: default.fuzz.seed = '111'
function invariantNotZero() public {
assertEq(back.number(), 1);
}
}

(which if I am not wrong is the one that fails?)

@DaniPopes
Copy link
Member Author

#12953

@DaniPopes DaniPopes enabled auto-merge December 30, 2025 08:09
@DaniPopes DaniPopes added this pull request to the merge queue Dec 30, 2025
Merged via the queue into master with commit 601f5bc Dec 30, 2025
16 checks passed
@DaniPopes DaniPopes deleted the dani/sign-wallet-pure branch December 30, 2025 08:27
@github-project-automation github-project-automation bot moved this to Done in Foundry Dec 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants