Fix TestSequencerFeePaid rounding

OffchainLabs · Jun 19, 2023 · cec8928 · cec8928
1 parent 257a3e5
commit cec8928
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/system_tests/fees_test.go b/system_tests/fees_test.go
@@ -90,12 +90,14 @@ func TestSequencerFeePaid(t *testing.T) {
 
 		txSize := compressedTxSize(t, tx)
 		l1GasBought := arbmath.BigDiv(l1Charge, l1Estimate).Uint64()
-		l1GasActual := txSize * params.TxDataNonZeroGasEIP2028
+		l1ChargeExpected := arbmath.BigMulByUint(l1Estimate, txSize*params.TxDataNonZeroGasEIP2028)
+		// L1 gas can only be charged in terms of L2 gas, so subtract off any rounding error from the expected value
+		l1ChargeExpected.Sub(l1ChargeExpected, new(big.Int).Mod(l1ChargeExpected, l2info.GasPrice))
 
 		colors.PrintBlue("bytes ", l1GasBought/params.TxDataNonZeroGasEIP2028, txSize)
 
-		if l1GasBought != l1GasActual {
-			Fatal(t, "the sequencer's future revenue does not match its costs", l1GasBought, l1GasActual)
+		if !arbmath.BigEquals(l1Charge, l1ChargeExpected) {
+			Fatal(t, "the sequencer's future revenue does not match its costs", l1Charge, l1ChargeExpected)
 		}
 		return networkRevenue, tipPaidToNet
 	}