Enabling payload validation for Sequencer by default (#467)

* Enabling payload validation for Sequencer by default to send newPayload and ensure EL node doesn't return syncing

* Simplifying setValidatorConfig logic

Author: Filter94

jvm-libs/test-utils/src/main/kotlin/maru/test/cluster/MaruConfigHelper.kt

@@ -119,18 +119,14 @@ internal fun setP2pConfig(
 
 internal fun setValidatorConfig(
   config: MaruConfig,
+  payloadValidationEnabled: Boolean,
   elNode: ElNode?,
 ): MaruConfig {
   if (elNode == null) return config
 
   val updatedValidatorConfig =
-    config.validatorElNode?.copy(
-      engineApiEndpoint =
-        config.validatorElNode!!.engineApiEndpoint.copy(
-          endpoint = URI.create(elNode.engineApiUrl()).toURL(),
-        ),
-    ) ?: ValidatorElNode(
-      payloadValidationEnabled = false,
+    ValidatorElNode(
+      payloadValidationEnabled = payloadValidationEnabled,
       engineApiEndpoint = ApiEndpointConfig(endpoint = URI.create(elNode.engineApiUrl()).toURL()),
     )
   val updatedForkTransition =

jvm-libs/test-utils/src/main/kotlin/maru/test/cluster/TestMaruAppFactory.kt

@@ -25,10 +25,19 @@ fun createMaru(
 ): MaruApp {
   initPersistence(config.persistence, nodeKeyData)
   var effectiveConfig = config
-  effectiveConfig = setValidatorConfig(effectiveConfig, elNode)
   effectiveConfig =
-    setQbftConfigIfSequencer(effectiveConfig, isSequencer = nodeRole == NodeRole.Sequencer, nodeKeyData)
-  effectiveConfig = setP2pConfig(effectiveConfig, bootnodes, staticpeers)
+    setValidatorConfig(
+      config = effectiveConfig,
+      payloadValidationEnabled = nodeRole == NodeRole.Sequencer,
+      elNode = elNode,
+    )
+  effectiveConfig =
+    setQbftConfigIfSequencer(
+      config = effectiveConfig,
+      isSequencer = nodeRole == NodeRole.Sequencer,
+      nodeKeyData = nodeKeyData,
+    )
+  effectiveConfig = setP2pConfig(config = effectiveConfig, bootnodes = bootnodes, staticpeers = staticpeers)
 
   return MaruAppFactory().create(
     config = effectiveConfig,