Reorder express lane tx validation

Put round check first. This fixes a test and it makes sense to let the
caller know they were sending submissions for the wrong round, before
telling them there was no controller for the current round.

execution/gethexec/express_lane_service.go

@@ -292,13 +292,13 @@ func (es *expressLaneService) validateExpressLaneTx(msg *timeboost.ExpressLaneSu
 	if msg.AuctionContractAddress != es.auctionContractAddr {
 		return errors.Wrapf(timeboost.ErrWrongAuctionContract, "msg auction contract address %s does not match sequencer auction contract address %s", msg.AuctionContractAddress, es.auctionContractAddr)
 	}
-	if !es.currentRoundHasController() {
-		return timeboost.ErrNoOnchainController
-	}
 	currentRound := timeboost.CurrentRound(es.initialTimestamp, es.roundDuration)
 	if msg.Round != currentRound {
 		return errors.Wrapf(timeboost.ErrBadRoundNumber, "express lane tx round %d does not match current round %d", msg.Round, currentRound)
 	}
+	if !es.currentRoundHasController() {
+		return timeboost.ErrNoOnchainController
+	}
 	// Reconstruct the message being signed over and recover the sender address.
 	signingMessage, err := msg.ToMessageBytes()
 	if err != nil {