• Mark Tyneway's avatar
    contracts-bedrock: cleanup tests · dc37fa23
    Mark Tyneway authored
    Use the `vm.expectEmit(address)` over the variant that uses all of the
    booleans when they are all true. This reduces code and makes it easier
    to read for reviewers. Each boolean means if the log topic/data should
    be compared. Generally, we want them to all be true which is why the
    short hand of just expecting it to come from an address exists, that
    assumes all of the values are true and asserts it emits from the
    address. We should always follow this style of `vm.expectEmit` unless
    there is a good reason not to, ie its not known that a particular topic
    will be or it is too complex to compute it. If this is the case then
    perhaps the smart contract is too complex and the architecture needs to
    be thought about more.
    dc37fa23
Name
Last commit
Last update
..
DelayedVetoable.t.sol Loading commit data...
L1CrossDomainMessenger.t.sol Loading commit data...
L1ERC721Bridge.t.sol Loading commit data...
L1StandardBridge.t.sol Loading commit data...
L2OutputOracle.t.sol Loading commit data...
OptimismPortal.t.sol Loading commit data...
ProtocolVersions.t.sol Loading commit data...
ResourceMetering.t.sol Loading commit data...
SuperchainConfig.t.sol Loading commit data...
SystemConfig.t.sol Loading commit data...