Skip to content

Commit 533224b

Browse files
authored
Cleanup nv22 MIGRATION-ONLY errors (#287)
Cleanup nv22 MIGRATION-ONLY errors
1 parent 9c769c7 commit 533224b

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

builtin/v13/check.go

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -267,11 +267,8 @@ func CheckDealStatesAgainstSectors(acc *builtin.MessageAccumulator, minerSummari
267267
continue
268268
}
269269

270-
// TODO: If you are reading this error after nv22 (v13 actors), delete this invariant.
271-
// It exists to test the v13 migration ONLY.
272270
sectorDeal, found := minerSummary.Deals[dealID]
273271
if !found {
274-
acc.Require(deal.SlashEpoch >= 0, "MIGRATION-ONLY: un-slashed deal %d not referenced in active sectors of miner %v", dealID, deal.Provider)
275272
continue
276273
}
277274

@@ -314,9 +311,9 @@ func CheckDealStatesAgainstSectors(acc *builtin.MessageAccumulator, minerSummari
314311

315312
for _, dealID := range dealIDs {
316313
_, found := minerSummary.Deals[dealID]
317-
// TODO: If you are reading this error after nv22 (v13 actors), delete this invariant.
318-
// It exists to test the v13 migration ONLY.
319-
acc.Require(found, "MIGRATION-ONLY: deal %d not found in miner %v for sector %v", dealID, maddr, sectorID)
314+
if !found {
315+
continue
316+
}
320317

321318
_, found = marketDealToSector[dealID]
322319
acc.Require(!found, "deal %d found in multiple sectors", dealID)

0 commit comments

Comments
 (0)