CI test for forge clone
#13
Closed
Conversation
This file contains 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
* feat: vm.sign for script wallets * more tests * clippy * if let some else * review fixes
* fix(anvil): clean up eth_estimateGas * fix doc * fix doc * fix doc * review fixes
* fix: coverage for internal libraries * optimize * optimize * doc * rm tests * clippy * clippy + fmt * clean up * for loop * review fixes
…-rs#7518) * feat: allow supplying fn name via forge script --sig * fmt * clippy * add test
…dry-rs#7535) fix(bench): avoid panic if execution not success
* feat: print ipc path * moved if check * moved println
* add TestContract * use TestContract * wip * fix * clippy + fmt * smaller diff
* feat(verify): multichain verification for etherscan * fix: bump reqwest * fix: ci cargo deny
…ns (foundry-rs#7550) * fix contracts sources Co-authored-by: lazymio <mio@lazym.io> * fix doc * fmt --------- Co-authored-by: lazymio <mio@lazym.io>
…7557) * fix(fmt): fix indent closing parenthesis enclosed in { } * Fix testdata bad formatting
* feat(forge test): only compile files needed for tests * remove comment * clippy * update fixtures * getCode + getDeployedCode updates * fixes * fix path matching * clippy * add config flag * fix * docs * fmt * patch compilers * fix Cargo.toml * update patch * update patch * doc * rm space * cargo cheats * new output selection fn * log compiler errors on failure * fixes
…dry-rs#7552) * fix dictionary * clippy + fmt * fix
* feat: index cheatcode * some nits to make it work * nit: use as_str() * final changes * chore: reviewed changes
* fix: --match-path is broken * rm docs
* batch size as a option * Update crates/script/src/broadcast.rs Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com> * Update crates/script/src/broadcast.rs Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com> * Update crates/script/src/lib.rs Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com> * Update crates/script/src/lib.rs Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com> * Update crates/script/src/lib.rs Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com> --------- Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
* use alloy-trie for eth_getProof * fmt * collect proofs via single pass * fixes and test * tests * add files * ordered_trie_root * clippy * move to workspace
* fix: smarter getCode validation * clippy + doc fix * fix * multi-version profile and tests * fmt * add more parsing options * clippy + fmt
* add `gasUsed and lastGasUsed methods to Vm * reorder * basic sketching of idea, planning to use Gas struct to deliver as much gas related information rather than a uint256 * add transformation * to prevent recording gas by default, only enable after recordGas is enabled * update struct layout, implementation builds, now connecting to cheatcodes * fix cheatcodes * refactor to use simple u64 as Gas struct doesnt have a default * change from Gas struct to simple u64 as I ran into issues with cache being reset to 0 * it appears cheatcodes are resolved before the actual function calls are therefore it doesnt actually remember the correct value from the previous execution but only of the previous executed cheatcode * still not working * finally works, stupid me didnt realize i had to cross call frames * emit gas record * test convenient single field access * add gas record struct back * pass down isolate bool, only enable gas tracing if enabled * raise error if cheatcode is used outside of isolation mode * mark as view * show gas refund and memory expansion * improve example * add isolation test, currently does not run as expected * fix fmt * avoid formatting changes * avoid commiting formatting changes, editor now configured correctly * lastGasUsed -> lastCallGas * small name fix * remove separate isolation profile, just configure on the runner * fix forge fmt * note on why path should never happen * remove separate isolated param, inherit from config * add support for non-isolation mode * remove isolate standalone, create additional entry in cheats and document subset of cheats that require to be tested in isolation mode as well * improve tests, use asserts and add option to exclude contracts from test filter, not just individual tests or paths * typo, no need to define path exclusion of forks in isolated tests as it is not relevant
…y-rs#7559) * fix(invariant): decode custom error with target contract abis * Changes after review: don't collect
* wip * better naming
…rs#7684) * add `no_metadata`, equivalent to adding `bytecode_hash = "none" and cbor_metadata = false` * add basic smoke test for --no-metadata setting cbor_metadata to false, bytecode_hash to none * Update core.rs Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com> --------- Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
* feat(debugger): update ratatui, use `List` * refactor(debugger): event handlers * fmt
…-rs#7686) * fix(cheatcodes): `expectSafeMemory` w/ new `forge-std` Fixes the `stopExpectSafeMemory` by allowing for the memory allocation of the `stopExpectSafeMemory` selector as well as the potentially out-of-bounds read performed in the `CALL` operation. Currently, forge reports incorrectly that memory safety was violated in a memory safe region of a test, if the free memory pointer was updated to `[exclusiveUpperBound-31, exclusiveUpperBound]`. To fix this, we allow for `MSTORE` operations that store the selector bytes for `stopExpectSafeMemory` outside of the allowed region, as well as `CALL` operations that are to the cheatcode address and contain the `stopExpectSafeMemory` selector in the first 4 bytes of the call arguments. * use `SELECTOR_LEN` const
* fix: hotfix cast logs subscribe * fix features
* feat: optimize compiler runs by reading AST * clippy * fallback to solc * fmt * clippy * update fixtures * fix for windows * wip * bump compilers * clippy + fmt * fix
* fix: better artifacts management * simplify * Update crates/common/src/contracts.rs Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com> * Arc<Config> --------- Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
…oundry-rs#7653) * wip * wip * add docs * clippy * update doc * simplify logic * review fixes * doc * review fixes * fix
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.