@inphi review

packages/contracts-bedrock/test/dispute/OutputBisectionGame.t.sol

@@ -1157,7 +1157,9 @@ contract OutputBisection_1v1_Actors_Test is OutputBisectionGame_Init {
         while (true) {
             // Allow the dishonest actor to make their moves, and then the honest actor.
             (uint256 numMovesA,) = dishonest.move();
-            (uint256 numMovesB,) = honest.move();
+            (uint256 numMovesB, bool success) = honest.move();
+
+            require(success, "Honest actor's moves should always be successful");
 
             // If both actors have run out of moves, we're done.
             if (numMovesA == 0 && numMovesB == 0) break;

packages/contracts-bedrock/test/mocks/AlphabetVM2.sol

@@ -31,7 +31,9 @@ contract AlphabetVM2 is IBigStepper {
         uint256 claim;
         if ((keccak256(_stateData) << 8) == (Claim.unwrap(ABSOLUTE_PRESTATE) << 8)) {
             // If the state data is empty, then the absolute prestate is the claim.
-            (bytes32 dat,) = oracle.readPreimage(PreimageKeyLib.localizeIdent(4, _localContext), 0);
+            (bytes32 dat,) = oracle.readPreimage(
+                PreimageKeyLib.localizeIdent(LocalPreimageKey.STARTING_L2_BLOCK_NUMBER, _localContext), 0
+            );
             uint256 startingL2BlockNumber = (uint256(dat) >> 128) & 0xFFFFFFFF;
             traceIndex = startingL2BlockNumber << 4;
             (uint256 absolutePrestateClaim) = abi.decode(_stateData, (uint256));