changing test name and adding OP issue link

Author: anupsv

e2e/server_test.go

@@ -23,27 +23,27 @@ func isPanic(err string) bool {
 		strings.Contains(err, "nil pointer dereference")
 }
 
-func TestOptimismClientWithKeccak256CommitmentIntegration(t *testing.T) {
+func TestOpClientKeccak256MalformedInputs(t *testing.T) {
 	if !runIntegrationTests || runTestnetIntegrationTests {
 		t.Skip("Skipping test as TESTNET env set or INTEGRATION var not set")
 	}
 
-	// nil commitment. Should return an error but currently is not. This needs to be fixed by OP
-	// t.Run("nil commitment case", func(t *testing.T) {
-	//	var commit op_plasma.CommitmentData
-	//	_, err := daClient.GetInput(ts.Ctx, commit)
-	//	require.Error(t, err)
-	//	assert.True(t, !isPanic(err.Error()))
-	// })
-
 	t.Parallel()
-
 	testCfg := e2e.TestConfig(useMemory())
 	testCfg.UseKeccak256ModeS3 = true
-
 	ts, kill := e2e.CreateTestSuite(t, testCfg)
 	defer kill()
 
+	// nil commitment. Should return an error but currently is not. This needs to be fixed by OP
+	// Ref: https://github.com/ethereum-optimism/optimism/issues/11987
+	//daClient := op_plasma.NewDAClient(ts.Address(), false, true)
+	//t.Run("nil commitment case", func(t *testing.T) {
+	//	var commit op_plasma.CommitmentData
+	//	_, err := daClient.GetInput(ts.Ctx, commit)
+	//	require.Error(t, err)
+	//	assert.True(t, !isPanic(err.Error()))
+	//})
+
 	daClientPcFalse := op_plasma.NewDAClient(ts.Address(), false, false)
 
 	t.Run("input bad data to SetInput & GetInput", func(t *testing.T) {