fix: remove checkExtraDataItemsCountPerTransaction from second phase

lidofinance · Dec 2, 2024 · 5de258b · 5de258b
1 parent e7b546e
commit 5de258b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/contracts/0.8.9/oracle/AccountingOracle.sol b/contracts/0.8.9/oracle/AccountingOracle.sol
@@ -552,10 +552,6 @@ contract AccountingOracle is BaseOracle {
             }
         }
 
-        IOracleReportSanityChecker(LOCATOR.oracleReportSanityChecker()).checkExtraDataItemsCountPerTransaction(
-            data.extraDataItemsCount
-        );
-
         LEGACY_ORACLE.handleConsensusLayerReport(data.refSlot, data.clBalanceGwei * 1e9, data.numValidators);
 
         uint256 slotsElapsed = data.refSlot - prevRefSlot;

diff --git a/test/0.8.9/oracle/accountingOracle.submitReport.test.ts b/test/0.8.9/oracle/accountingOracle.submitReport.test.ts
@@ -373,8 +373,7 @@ describe("AccountingOracle.sol:submitReport", () => {
     });
 
     context("enforces data safety boundaries", () => {
-      // TODO: restore test, but it is suspected to be inrelevant, must revert as it actually does
-      it.skip("passes fine when extra data do not feet in a single third phase transaction", async () => {
+      it("passes fine when extra data do not feet in a single third phase transaction", async () => {
         const MAX_ITEMS_PER_EXTRA_DATA_TRANSACTION = 1;
 
         expect(reportFields.extraDataItemsCount).to.be.greaterThan(MAX_ITEMS_PER_EXTRA_DATA_TRANSACTION);