bgd-labs / aave-proposals-v3

MIT License
16 stars 18 forks source link

Activate new steward - Review #348

Closed marczeller closed 3 weeks ago

github-actions[bot] commented 3 weeks ago

Foundry report

forge 0.2.0 (fb86e5d 2024-06-05T00:17:41.128117927Z)
Build log ```shell Compiling 288 files with Solc 0.8.19 Solc 0.8.19 finished in 145.08s Compiler run successful with warnings: Warning (3628): This contract has a payable fallback function, but no receive ether function. Consider adding a receive ether function. --> src/20240502_AaveV1Ethereum_AaveV1Deprecation/AaveV1Ethereum_AaveV1Deprecation_20240502.t.sol:94:1: | 94 | contract AaveV1Ethereum_AaveV1Deprecation_20240502_Test is ProtocolV3TestBase { | ^ (Relevant source part starts here and spans across multiple lines). Note: The payable fallback function is defined here. --> src/20240502_AaveV1Ethereum_AaveV1Deprecation/AaveV1Ethereum_AaveV1Deprecation_20240502.t.sol:297:3: | 297 | fallback() external payable {} | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | Contract | Size (B) | Margin (B) | |---------------------------------------------------------------------------|----------|------------| | AaveGovernanceV2 | 45 | 24,531 | | AaveSafetyModule | 45 | 24,531 | | AaveSwapper | 5,572 | 19,004 | | AaveV1 | 45 | 24,531 | | AaveV1Ethereum_AaveV1Deprecation_20240502 | 2,593 | 21,983 | | AaveV2Avalanche | 45 | 24,531 | | AaveV2AvalancheAssets | 45 | 24,531 | | AaveV2Avalanche_LTLTVReductionsOnAaveV2Stablecoins_20240510 | 210 | 24,366 | | AaveV2Avalanche_MigrateRobotsToChainlinkAutomationV2_20240422 | 518 | 24,058 | | AaveV2Avalanche_ReserveFactorUpgrades_20240506 | 1,022 | 23,554 | | AaveV2Avalanche_ReserveFactorUpgrades_20240524 | 1,022 | 23,554 | | AaveV2Avalanche_StablecoinIRUpdates_20240424 | 1,858 | 22,718 | | AaveV2Avalanche_TemporaryFreezeOfLongTailV2Assets_20240418 | 186 | 24,390 | | AaveV2Ethereum | 45 | 24,531 | | AaveV2EthereumAMM | 45 | 24,531 | | AaveV2EthereumAMMAssets | 45 | 24,531 | | AaveV2EthereumAssets | 45 | 24,531 | | AaveV2Ethereum_AAMPLSecondDistribution_20240429 | 2,370 | 22,206 | | AaveV2Ethereum_AprilFinanceUpdate_20240421 | 7,897 | 16,679 | | AaveV2Ethereum_ChaosLabsEthereumV2LTReductions_20240509 | 358 | 24,218 | | AaveV2Ethereum_DeprecationOfSmallCapStablecoinsOnV2Ethereum_20240502 | 3,472 | 21,104 | | AaveV2Ethereum_LTLTVReductionsOnAaveV2Stablecoins_20240510 | 210 | 24,366 | | AaveV2Ethereum_ReserveFactorUpgrades_20240506 | 1,778 | 22,798 | | AaveV2Ethereum_ReserveFactorUpgrades_20240524 | 1,778 | 22,798 | | AaveV2Ethereum_StablecoinIRUpdates_20240424 | 1,858 | 22,718 | | AaveV2Ethereum_TemporaryFreezeOfLongTailV2Assets_20240418 | 793 | 23,783 | | AaveV2Polygon | 45 | 24,531 | | AaveV2PolygonAssets | 45 | 24,531 | | AaveV2Polygon_AprilFinanceUpdate_20240421 | 6,129 | 18,447 | | AaveV2Polygon_AprilFinanceUpdate_20240421_PartB | 451 | 24,125 | | AaveV2Polygon_LTLTVReductionsOnAaveV2Stablecoins_20240510 | 210 | 24,366 | | AaveV2Polygon_MigrateRobotsToChainlinkAutomationV2_20240422 | 518 | 24,058 | | AaveV2Polygon_TemporaryFreezeOfLongTailV2Assets_20240418 | 186 | 24,390 | | AaveV3Arbitrum | 45 | 24,531 | | AaveV3ArbitrumAssets | 45 | 24,531 | | AaveV3ArbitrumEModes | 45 | 24,531 | | AaveV3Arbitrum_GeneralizedLTLTVReductionsOnAaveV3Step2_20240425 | 3,669 | 20,907 | | AaveV3Arbitrum_MigrateRobotsToChainlinkAutomationV2_20240422 | 2,954 | 21,622 | | AaveV3Arbitrum_StablecoinIRUpdates_20240424 | 5,381 | 19,195 | | AaveV3Avalanche | 45 | 24,531 | | AaveV3AvalancheAssets | 45 | 24,531 | | AaveV3AvalancheEModes | 45 | 24,531 | | AaveV3Avalanche_GeneralizedLTLTVReductionsOnAaveV3Step2_20240425 | 3,377 | 21,199 | | AaveV3Avalanche_MigrateRobotsToChainlinkAutomationV2_20240422 | 3,577 | 20,999 | | AaveV3Avalanche_StablecoinIRUpdates_20240424 | 4,628 | 19,948 | | AaveV3BNB | 45 | 24,531 | | AaveV3BNBAssets | 45 | 24,531 | | AaveV3BNBEModes | 45 | 24,531 | | AaveV3BNB_GeneralizedLTLTVReductionsOnAaveV3Step2_20240425 | 3,528 | 21,048 | | AaveV3BNB_StablecoinIRUpdates_20240424 | 4,132 | 20,444 | | AaveV3Base | 45 | 24,531 | | AaveV3BaseAssets | 45 | 24,531 | | AaveV3BaseEModes | 45 | 24,531 | | AaveV3Base_GeneralizedLTLTVReductionsOnAaveV3Step2_20240425 | 3,517 | 21,059 | | AaveV3Base_StablecoinIRUpdates_20240424 | 3,868 | 20,708 | | AaveV3Base_WeETHAaveV3BaseOnboarding_20240527 | 4,997 | 19,579 | | AaveV3Ethereum | 45 | 24,531 | | AaveV3EthereumAssets | 45 | 24,531 | | AaveV3EthereumEModes | 45 | 24,531 | | AaveV3Ethereum_ACIAdAstra_20240506 | 835 | 23,741 | | AaveV3Ethereum_AaveV31CantinaCompetition_20240503 | 344 | 24,232 | | AaveV3Ethereum_AprilFinanceUpdatePartB_20240515 | 4,488 | 20,088 | | AaveV3Ethereum_GeneralizedLTLTVReductionsOnAaveV3Step2_20240425 | 3,669 | 20,907 | | AaveV3Ethereum_GhoStewardUpdate_20240602 | 2,571 | 22,005 | | AaveV3Ethereum_LlamariskRiskProvider_20240421 | 641 | 23,935 | | AaveV3Ethereum_MigrateRobotsToChainlinkAutomationV2_20240422 | 5,800 | 18,776 | | AaveV3Ethereum_OnboardUSDeAaveV3Ethereum_20240528 | 5,012 | 19,564 | | AaveV3Ethereum_OrbitProgramRenewal_20240513 | 1,670 | 22,906 | | AaveV3Ethereum_StablecoinIRUpdates_20240424 | 5,130 | 19,446 | | AaveV3Ethereum_StkGHOIncentives_20240424 | 1,476 | 23,100 | | AaveV3Ethereum_UpdatingWeETHRiskParameters_20240426 | 4,111 | 20,465 | | AaveV3Gnosis | 45 | 24,531 | | AaveV3GnosisAssets | 45 | 24,531 | | AaveV3GnosisEModes | 45 | 24,531 | | AaveV3Gnosis_AprilFinanceUpdate_20240421 | 380 | 24,196 | | AaveV3Gnosis_GeneralizedLTLTVReductionsOnAaveV3Step2_20240425 | 3,519 | 21,057 | | AaveV3Gnosis_StablecoinIRUpdates_20240424 | 4,123 | 20,453 | | AaveV3Optimism | 45 | 24,531 | | AaveV3OptimismAssets | 45 | 24,531 | | AaveV3OptimismEModes | 45 | 24,531 | | AaveV3Optimism_ChaosLabsParameterRecommendationsSUSDOnV3Optimism_20240528 | 3,973 | 20,603 | | AaveV3Optimism_GeneralizedLTLTVReductionsOnAaveV3Step2_20240425 | 3,669 | 20,907 | | AaveV3Optimism_MigrateRobotsToChainlinkAutomationV2_20240422 | 2,844 | 21,732 | | AaveV3Optimism_SUSDRiskParametersUpdate_20240517 | 3,372 | 21,204 | | AaveV3Optimism_StablecoinIRUpdates_20240424 | 5,130 | 19,446 | | AaveV3Polygon | 45 | 24,531 | | AaveV3PolygonAssets | 45 | 24,531 | | AaveV3PolygonEModes | 45 | 24,531 | | AaveV3Polygon_GeneralizedLTLTVReductionsOnAaveV3Step2_20240425 | 3,668 | 20,908 | | AaveV3Polygon_MigrateRobotsToChainlinkAutomationV2_20240422 | 2,773 | 21,803 | | AaveV3Polygon_StablecoinIRUpdates_20240424 | 5,380 | 19,196 | | AaveV3Scroll | 45 | 24,531 | | AaveV3ScrollAssets | 45 | 24,531 | | AaveV3ScrollEModes | 45 | 24,531 | | AaveV3Scroll_GeneralizedLTLTVReductionsOnAaveV3Step2_20240425 | 3,374 | 21,202 | | AaveV3Scroll_StablecoinIRUpdates_20240424 | 3,620 | 20,956 | | Address | 45 | 24,531 | | ChainHelpers | 45 | 24,531 | | ChainIds | 45 | 24,531 | | ConfiguratorInputTypes | 45 | 24,531 | | Create2Utils | 123 | 24,453 | | DataTypes | 45 | 24,531 | | EngineFlags | 45 | 24,531 | | Errors | 4,673 | 19,903 | | GovV3Helpers | 2,581 | 21,995 | | GovV3StorageHelpers | 45 | 24,531 | | GovernanceV3Arbitrum | 45 | 24,531 | | GovernanceV3Avalanche | 45 | 24,531 | | GovernanceV3BNB | 45 | 24,531 | | GovernanceV3Base | 45 | 24,531 | | GovernanceV3Ethereum | 45 | 24,531 | | GovernanceV3Gnosis | 45 | 24,531 | | GovernanceV3Metis | 45 | 24,531 | | GovernanceV3Optimism | 45 | 24,531 | | GovernanceV3Polygon | 45 | 24,531 | | GovernanceV3PolygonZkEvm | 45 | 24,531 | | GovernanceV3Scroll | 45 | 24,531 | | IpfsUtils | 45 | 24,531 | | MiscEthereum | 45 | 24,531 | | MiscPolygon | 45 | 24,531 | | OrbitProgramData | 422 | 24,154 | | PayloadsControllerUtils | 45 | 24,531 | | ProxyHelpers | 45 | 24,531 | | ReserveConfiguration | 130 | 24,446 | | SafeCast | 45 | 24,531 | | SafeERC20 | 45 | 24,531 | | StorageHelpers | 45 | 24,531 | | WadRayMath | 45 | 24,531 | ```
Test error :finnadie::x: ```shell No files changed, compilation skipped Ran 11 tests for src/20240602_AaveV3Ethereum_GhoStewardUpdate/AaveV3Ethereum_GhoStewardUpdate_20240602.t.sol:AaveV3Ethereum_GhoStewardUpdate_20240602_Test [PASS] testLowerGhoBorrowCap() (gas: 640548) [PASS] testLowerGhoBorrowRate() (gas: 919840) [PASS] testLowerGsmBuySellFeesBuyFee() (gas: 970219) [PASS] testLowerGsmExposureCap() (gas: 637496) [PASS] testUpdateFacilitatorBucketCapacity() (gas: 722268) [PASS] testUpdateGhoBorrowCap() (gas: 640517) [PASS] testUpdateGhoBorrowRate() (gas: 919785) [PASS] testUpdateGsmBuySellFeesBuyFee() (gas: 970202) [PASS] testUpdateGsmExposureCap() (gas: 637543) [PASS] test_adminPermissions() (gas: 592527) [FAIL. Reason: EvmError: Revert] test_defaultProposalExecution() (gas: 8937393460516734916) Suite result: FAILED. 10 passed; 1 failed; 0 skipped; finished in 350.72s (850.69s CPU time) Ran 1 test suite in 350.73s (350.72s CPU time): 10 tests passed, 1 failed, 0 skipped (11 total tests) Failing tests: Encountered 1 failing test in src/20240602_AaveV3Ethereum_GhoStewardUpdate/AaveV3Ethereum_GhoStewardUpdate_20240602.t.sol:AaveV3Ethereum_GhoStewardUpdate_20240602_Test [FAIL. Reason: EvmError: Revert] test_defaultProposalExecution() (gas: 8937393460516734916) Encountered a total of 1 failing tests, 10 tests succeeded ```