Working 42.20 MP

tests/persistence-handlers-42.20-spec.lua

@@ -156,14 +156,24 @@ local function runPortableHandlerSpec()
 
     fireTicks(callbacks.tick, 5)
     expect(metrics.addConstraint == 1, "portable restart must request one native restore")
-    expect(metrics.attachSync == 1, "portable restart must broadcast one restore")
+    expect(metrics.attachSync == 1,
+        "portable restart must immediately synchronize one wrecker-style rigid restore")
+
+    -- The restore is queued before the native call. Once Build 42 exposes the
+    -- reciprocal relation, the pending confirmation broadcasts exactly once.
+    linkPair(towing, towed)
+    now = 1100
+    fireTicks(callbacks.tick, 1)
+    expect(metrics.addConstraint == 1,
+        "portable confirmation must adopt the first native add without re-adding")
+    expect(metrics.attachSync == 1,
+        "portable native acknowledgement must not introduce a second sync phase")
 
     now = 1500
-    fireTicks(callbacks.tick, 5)
+    fireTicks(callbacks.tick, 4)
     expect(metrics.addConstraint == 1, "portable pending restore must not duplicate its constraint")
     expect(metrics.attachSync == 1, "portable pending restore must not duplicate its sync")
 
-    linkPair(towing, towed)
     now = 2000
     fireTicks(callbacks.tick, 5)
     expect(metrics.addConstraint == 1, "portable delayed acknowledgement must be adopted without re-adding")
@@ -187,7 +197,23 @@ local function runPortableHandlerSpec()
     now = 17000
     fireTicks(callbacks.tick, 5)
     expect(metrics.addConstraint == 2, "a reloaded portable peer must re-enter recovery")
-    expect(metrics.attachSync == 2, "a reloaded portable peer must receive one fresh restore sync")
+    expect(metrics.attachSync == 2,
+        "a reloaded portable peer must receive one fresh wrecker-style restore sync")
+
+    linkPair(towing, towed)
+    now = 17100
+    fireTicks(callbacks.tick, 1)
+    expect(metrics.addConstraint == 2,
+        "reloaded portable confirmation must adopt without another native add")
+    expect(metrics.attachSync == 2,
+        "reloaded portable native acknowledgement must not duplicate its sync")
+
+    now = 17500
+    fireTicks(callbacks.tick, 4)
+    expect(metrics.addConstraint == 2,
+        "confirmed reloaded pair must not duplicate its native add")
+    expect(metrics.attachSync == 2,
+        "confirmed reloaded pair must not duplicate its authoritative sync")
     expect(metrics.worldDrops == 0, "portable peer reload must not be mistaken for a break")
     expect(metrics.breakConstraint == 0, "portable peer reload must not break an unrelated constraint")
 end