From 533224b92afae152b443357b417c91233d5f9554 Mon Sep 17 00:00:00 2001 From: Phi-rjan Date: Wed, 10 Jul 2024 07:53:18 +0200 Subject: [PATCH] Cleanup nv22 MIGRATION-ONLY errors (#287) Cleanup nv22 MIGRATION-ONLY errors --- builtin/v13/check.go | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/builtin/v13/check.go b/builtin/v13/check.go index ee20b0e0..7de8fe05 100644 --- a/builtin/v13/check.go +++ b/builtin/v13/check.go @@ -267,11 +267,8 @@ func CheckDealStatesAgainstSectors(acc *builtin.MessageAccumulator, minerSummari continue } - // TODO: If you are reading this error after nv22 (v13 actors), delete this invariant. - // It exists to test the v13 migration ONLY. sectorDeal, found := minerSummary.Deals[dealID] if !found { - acc.Require(deal.SlashEpoch >= 0, "MIGRATION-ONLY: un-slashed deal %d not referenced in active sectors of miner %v", dealID, deal.Provider) continue } @@ -314,9 +311,9 @@ func CheckDealStatesAgainstSectors(acc *builtin.MessageAccumulator, minerSummari for _, dealID := range dealIDs { _, found := minerSummary.Deals[dealID] - // TODO: If you are reading this error after nv22 (v13 actors), delete this invariant. - // It exists to test the v13 migration ONLY. - acc.Require(found, "MIGRATION-ONLY: deal %d not found in miner %v for sector %v", dealID, maddr, sectorID) + if !found { + continue + } _, found = marketDealToSector[dealID] acc.Require(!found, "deal %d found in multiple sectors", dealID)