op-e2e: batcher action testing review fixes

op-e2e/actions/l2_batcher.go

@@ -127,14 +127,22 @@ func (s *L2Batcher) ActL2BatchBuffer(t Testing) {
 	}
 }
 
+func (s *L2Batcher) ActL2ChannelClose(t Testing) {
+	// Don't run this action if there's no data to submit
+	if s.l2ChannelOut == nil {
+		t.InvalidAction("need to buffer data first, cannot batch submit with empty buffer")
+		return
+	}
+	require.NoError(t, s.l2ChannelOut.Close(), "must close channel before submitting it")
+}
+
 // ActL2BatchSubmit constructs a batch tx from previous buffered L2 blocks, and submits it to L1
 func (s *L2Batcher) ActL2BatchSubmit(t Testing) {
 	// Don't run this action if there's no data to submit
-	if s.l2ChannelOut == nil || s.l2ChannelOut.ReadyBytes() == 0 {
+	if s.l2ChannelOut == nil {
 		t.InvalidAction("need to buffer data first, cannot batch submit with empty buffer")
 		return
 	}
-	require.NoError(t, s.l2ChannelOut.Close(), "must close channel before submitting it")
 	// Collect the output frame
 	data := new(bytes.Buffer)
 	data.WriteByte(derive.DerivationVersion0)

op-e2e/actions/l2_batcher_test.go

@@ -58,6 +58,7 @@ func TestBatcher(gt *testing.T) {
 
 	// batch submit to L1
 	batcher.ActL2BatchBuffer(t)
+	batcher.ActL2ChannelClose(t)
 	batcher.ActL2BatchSubmit(t)
 
 	// confirm batch on L1