Skip to content

Commit

Permalink
Test fixes + wrappers
Browse files Browse the repository at this point in the history
  • Loading branch information
@Fletch153
Fletch153 committed Sep 5, 2023
1 parent 64b2553 commit 6af03dc
Show file tree
Hide file tree
Showing 7 changed files with 49 additions and 43 deletions.
2 changes: 1 addition & 1 deletion contracts/foundry-lib/forge-std
Submodule forge-std updated 27 files
+6 −48 .github/workflows/ci.yml
+0 −29 .github/workflows/sync.yml
+1 −1 foundry.toml
+1 −1 lib/ds-test
+1 −1 package.json
+3 −5 src/Base.sol
+3 −4 src/Script.sol
+8 −8 src/StdAssertions.sol
+21 −24 src/StdChains.sol
+17 −205 src/StdCheats.sol
+1 −16 src/StdInvariant.sol
+14 −14 src/StdJson.sol
+3 −54 src/StdStorage.sol
+2 −2 src/StdStyle.sol
+3 −12 src/StdUtils.sol
+3 −4 src/Test.sol
+24 −182 src/Vm.sol
+382 −394 src/console2.sol
+0 −13,248 src/safeconsole.sol
+84 −145 test/StdAssertions.t.sol
+52 −112 test/StdChains.t.sol
+43 −242 test/StdCheats.t.sol
+10 −10 test/StdError.t.sol
+12 −27 test/StdMath.t.sol
+34 −66 test/StdStorage.t.sol
+4 −4 test/StdStyle.t.sol
+35 −80 test/StdUtils.t.sol
44 changes: 22 additions & 22 deletions contracts/gas-snapshots/llo-feeds.gas-snapshot
View file
Original file line number Diff line number Diff line change
Expand Up @@ -160,9 +160,9 @@ RewardManagerUpdateRewardRecipientsTest:test_updateRecipientsToDifferentSet() (g
RewardManagerUpdateRewardRecipientsTest:test_updateRecipientsToDifferentSetWithInvalidWeights() (gas: 254576)
RewardManagerUpdateRewardRecipientsTest:test_updateRecipientsUpdateAndRemoveExistingForLargerSet() (gas: 246971)
RewardManagerUpdateRewardRecipientsTest:test_updateRecipientsUpdateAndRemoveExistingForSmallerSet() (gas: 245257)
VerificationdeactivateConfigWhenThereAreMultipleDigestsTest:test_correctlyRemovesAMiddleDigest() (gas: 24155)
VerificationdeactivateConfigWhenThereAreMultipleDigestsTest:test_correctlyRemovesTheFirstDigest() (gas: 24122)
VerificationdeactivateConfigWhenThereAreMultipleDigestsTest:test_correctlyUnsetsDigestsInSequence() (gas: 44043)
VerificationdeactivateConfigWhenThereAreMultipleDigestsTest:test_correctlyRemovesAMiddleDigest() (gas: 24177)
VerificationdeactivateConfigWhenThereAreMultipleDigestsTest:test_correctlyRemovesTheFirstDigest() (gas: 24144)
VerificationdeactivateConfigWhenThereAreMultipleDigestsTest:test_correctlyUnsetsDigestsInSequence() (gas: 44109)
VerificationdeactivateConfigWhenThereAreMultipleDigestsTest:test_revertsIfCalledByNonOwner() (gas: 11278)
VerificationdeactivateConfigWhenThereAreMultipleDigestsTest:test_revertsIfRemovingAnEmptyDigest() (gas: 10907)
VerificationdeactivateConfigWhenThereAreMultipleDigestsTest:test_revertsIfRemovingAnNonExistentDigest() (gas: 13381)
Expand All @@ -183,8 +183,8 @@ VerifierBulkVerifyBillingReport:test_verifyWithBulkLink() (gas: 572112)
VerifierBulkVerifyBillingReport:test_verifyWithBulkNative() (gas: 573880)
VerifierBulkVerifyBillingReport:test_verifyWithBulkNativeUnwrapped() (gas: 583847)
VerifierBulkVerifyBillingReport:test_verifyWithBulkNativeUnwrappedReturnsChange() (gas: 590874)
VerifierConstructorTest:test_revertsIfInitializedWithEmptyVerifierProxy() (gas: 59939)
VerifierConstructorTest:test_setsTheCorrectProperties() (gas: 1788491)
VerifierConstructorTest:test_revertsIfInitializedWithEmptyVerifierProxy() (gas: 59967)
VerifierConstructorTest:test_setsTheCorrectProperties() (gas: 1815769)
VerifierDeactivateFeedWithVerifyTest:test_currentReportAllowsVerification() (gas: 188300)
VerifierDeactivateFeedWithVerifyTest:test_currentReportFailsVerification() (gas: 109631)
VerifierDeactivateFeedWithVerifyTest:test_previousReportAllowsVerification() (gas: 95877)
Expand All @@ -194,13 +194,13 @@ VerifierProxyAccessControlledVerificationTest:test_revertsIfNoAccess() (gas: 105
VerifierProxyConstructorTest:test_correctlySetsTheCorrectAccessControllerInterface() (gas: 1357277)
VerifierProxyConstructorTest:test_correctlySetsTheOwner() (gas: 1337313)
VerifierProxyConstructorTest:test_correctlySetsVersion() (gas: 6939)
VerifierProxyInitializeVerifierTest:test_revertsIfDigestAlreadySet() (gas: 50333)
VerifierProxyInitializeVerifierTest:test_revertsIfDigestAlreadySet() (gas: 50355)
VerifierProxyInitializeVerifierTest:test_revertsIfNotCorrectVerifier() (gas: 13572)
VerifierProxyInitializeVerifierTest:test_revertsIfNotOwner() (gas: 13441)
VerifierProxyInitializeVerifierTest:test_revertsIfVerifierAlreadyInitialized() (gas: 42047)
VerifierProxyInitializeVerifierTest:test_revertsIfZeroAddress() (gas: 10970)
VerifierProxyInitializeVerifierTest:test_setFeeManagerZeroAddress() (gas: 10913)
VerifierProxyInitializeVerifierTest:test_updatesVerifierIfVerifier() (gas: 49583)
VerifierProxyInitializeVerifierTest:test_updatesVerifierIfVerifier() (gas: 49605)
VerifierProxySetAccessControllerTest:test_emitsTheCorrectEvent() (gas: 35384)
VerifierProxySetAccessControllerTest:test_revertsIfCalledByNonOwner() (gas: 11367)
VerifierProxySetAccessControllerTest:test_successfullySetsNewAccessController() (gas: 32120)
Expand All @@ -211,20 +211,20 @@ VerifierProxyUnsetVerifierWithPreviouslySetVerifierTest:test_correctlyUnsetsVeri
VerifierProxyUnsetVerifierWithPreviouslySetVerifierTest:test_emitsAnEventAfterUnsettingVerifier() (gas: 17943)
VerifierProxyVerifyTest:test_proxiesToTheCorrectVerifier() (gas: 198408)
VerifierProxyVerifyTest:test_revertsIfNoVerifierConfigured() (gas: 116352)
VerifierSetConfigFromSourceMultipleDigestsTest:test_correctlySetsConfigWhenDigestsAreRemoved() (gas: 538560)
VerifierSetConfigFromSourceMultipleDigestsTest:test_correctlyUpdatesDigestsOnMultipleVerifiersInTheProxy() (gas: 964093)
VerifierSetConfigFromSourceMultipleDigestsTest:test_correctlyUpdatesTheDigestInTheProxy() (gas: 520118)
VerifierSetConfigFromSourceTest:test_revertsIfCalledByNonOwner() (gas: 179353)
VerifierSetConfigTest:test_correctlyUpdatesTheConfig() (gas: 1057669)
VerifierSetConfigTest:test_revertsIfCalledByNonOwner() (gas: 179222)
VerifierSetConfigTest:test_revertsIfDuplicateSigners() (gas: 251572)
VerifierSetConfigTest:test_revertsIfFaultToleranceIsZero() (gas: 176521)
VerifierSetConfigTest:test_revertsIfNotEnoughSigners() (gas: 15813)
VerifierSetConfigTest:test_revertsIfSetWithTooManySigners() (gas: 22169)
VerifierSetConfigTest:test_revertsIfSignerContainsZeroAddress() (gas: 228002)
VerifierSetConfigWhenThereAreMultipleDigestsTest:test_correctlySetsConfigWhenDigestsAreRemoved() (gas: 538347)
VerifierSetConfigWhenThereAreMultipleDigestsTest:test_correctlyUpdatesDigestsOnMultipleVerifiersInTheProxy() (gas: 963662)
VerifierSetConfigWhenThereAreMultipleDigestsTest:test_correctlyUpdatesTheDigestInTheProxy() (gas: 519899)
VerifierSetConfigFromSourceMultipleDigestsTest:test_correctlySetsConfigWhenDigestsAreRemoved() (gas: 538851)
VerifierSetConfigFromSourceMultipleDigestsTest:test_correctlyUpdatesDigestsOnMultipleVerifiersInTheProxy() (gas: 964679)
VerifierSetConfigFromSourceMultipleDigestsTest:test_correctlyUpdatesTheDigestInTheProxy() (gas: 520412)
VerifierSetConfigFromSourceTest:test_revertsIfCalledByNonOwner() (gas: 179453)
VerifierSetConfigTest:test_correctlyUpdatesTheConfig() (gas: 1057922)
VerifierSetConfigTest:test_revertsIfCalledByNonOwner() (gas: 179244)
VerifierSetConfigTest:test_revertsIfDuplicateSigners() (gas: 251602)
VerifierSetConfigTest:test_revertsIfFaultToleranceIsZero() (gas: 176543)
VerifierSetConfigTest:test_revertsIfNotEnoughSigners() (gas: 15835)
VerifierSetConfigTest:test_revertsIfSetWithTooManySigners() (gas: 22191)
VerifierSetConfigTest:test_revertsIfSignerContainsZeroAddress() (gas: 228032)
VerifierSetConfigWhenThereAreMultipleDigestsTest:test_correctlySetsConfigWhenDigestsAreRemoved() (gas: 538600)
VerifierSetConfigWhenThereAreMultipleDigestsTest:test_correctlyUpdatesDigestsOnMultipleVerifiersInTheProxy() (gas: 964168)
VerifierSetConfigWhenThereAreMultipleDigestsTest:test_correctlyUpdatesTheDigestInTheProxy() (gas: 520152)
VerifierSupportsInterfaceTest:test_falseIfIsNotCorrectInterface() (gas: 5590)
VerifierSupportsInterfaceTest:test_trueIfIsCorrectInterface() (gas: 5611)
VerifierTestBillingReport:test_verifyWithLink() (gas: 273128)
Expand All @@ -248,7 +248,7 @@ VerifierVerifySingleConfigDigestTest:test_setsTheCorrectEpoch() (gas: 190530)
Verifier_accessControlledVerify:testVerifyWithAccessControl_gas() (gas: 205085)
Verifier_bulkVerifyWithFee:testBulkVerifyProxyWithLinkFeeSuccess_gas() (gas: 542038)
Verifier_bulkVerifyWithFee:testBulkVerifyProxyWithNativeFeeSuccess_gas() (gas: 565410)
Verifier_setConfig:testSetConfigSuccess_gas() (gas: 922406)
Verifier_setConfig:testSetConfigSuccess_gas() (gas: 922637)
Verifier_verify:testVerifyProxySuccess_gas() (gas: 195530)
Verifier_verify:testVerifySuccess_gas() (gas: 182991)
Verifier_verifyWithFee:testVerifyProxyWithLinkFeeSuccess_gas() (gas: 244551)
Expand Down
1 change: 1 addition & 0 deletions contracts/src/v0.8/llo-feeds/test/mocks/ErroredVerifier.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ contract ErroredVerifier is IVerifier {
bytes32,
uint256,
address,
uint32,
address[] memory,
bytes32[] memory,
uint8,
Expand Down
5 changes: 5 additions & 0 deletions contracts/src/v0.8/llo-feeds/test/verifier/VerifierSetConfigFromSourceTest.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ contract VerifierSetConfigFromSourceTest is BaseTest {
FEED_ID,
12345,
address(12345),
0,
_getSignerAddresses(signers),
s_offchaintransmitters,
FAULT_TOLERANCE,
Expand All @@ -39,6 +40,7 @@ contract VerifierSetConfigFromSourceMultipleDigestsTest is BaseTestWithMultipleC
FEED_ID,
12345,
address(12345),
0,
_getSignerAddresses(newSigners),
s_offchaintransmitters,
4,
Expand All @@ -60,6 +62,7 @@ contract VerifierSetConfigFromSourceMultipleDigestsTest is BaseTestWithMultipleC
FEED_ID_2,
12345,
address(12345),
0,
_getSignerAddresses(newSigners),
s_offchaintransmitters,
4,
Expand All @@ -77,6 +80,7 @@ contract VerifierSetConfigFromSourceMultipleDigestsTest is BaseTestWithMultipleC
FEED_ID_3,
12345,
address(12345),
0,
_getSignerAddresses(newSigners),
s_offchaintransmitters,
4,
Expand All @@ -100,6 +104,7 @@ contract VerifierSetConfigFromSourceMultipleDigestsTest is BaseTestWithMultipleC
FEED_ID,
12345,
address(s_verifier),
0,
_getSignerAddresses(newSigners),
s_offchaintransmitters,
4,
Expand Down
Loading

0 comments on commit 6af03dc

Please sign in to comment.