From a30a504c4703f45103f9182ab9307b2b4497fc05 Mon Sep 17 00:00:00 2001
From: Lucio Lelii <lucio.lelii@isti.cnr.it>
Date: Wed, 23 Sep 2026 11:33:27 +0200
Subject: [PATCH] Start human evaluation blocks without a template

---
 .../HumanEvaluationBlockConfiguration.java         |  4 +---
 .../factories/HumanEvaluationBlockFactoryTest.java | 14 ++++++++++++++
 2 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/src/main/java/it/cnr/isti/workflow/manager/blocks/configurations/HumanEvaluationBlockConfiguration.java b/src/main/java/it/cnr/isti/workflow/manager/blocks/configurations/HumanEvaluationBlockConfiguration.java
index 4937883..ffdca10 100644
--- a/src/main/java/it/cnr/isti/workflow/manager/blocks/configurations/HumanEvaluationBlockConfiguration.java
+++ b/src/main/java/it/cnr/isti/workflow/manager/blocks/configurations/HumanEvaluationBlockConfiguration.java
@@ -111,10 +111,8 @@ public class HumanEvaluationBlockConfiguration extends BlockConfiguration<HumanE
     public static HumanEvaluationBlockConfiguration empty() {
         return HumanEvaluationBlockConfiguration.builder()
                 .name(HumanEvaluationBlockType.TYPE)
-                .target("Open ${{targetUrl}} and try the main flow")
                 .taskScript(List.of())
-                .criteria(List.of(new EvaluationCriterion("works", "Does the main flow complete without errors?",
-                        EvaluationScale.PASS_FAIL, true)))
+                .criteria(List.of())
                 .build();
     }
 
diff --git a/src/test/java/it/cnr/isti/workflow/manager/blocks/factories/HumanEvaluationBlockFactoryTest.java b/src/test/java/it/cnr/isti/workflow/manager/blocks/factories/HumanEvaluationBlockFactoryTest.java
index 9a93acd..6e81e07 100644
--- a/src/test/java/it/cnr/isti/workflow/manager/blocks/factories/HumanEvaluationBlockFactoryTest.java
+++ b/src/test/java/it/cnr/isti/workflow/manager/blocks/factories/HumanEvaluationBlockFactoryTest.java
@@ -6,6 +6,7 @@ package it.cnr.isti.workflow.manager.blocks.factories;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.util.List;
@@ -62,6 +63,19 @@ class HumanEvaluationBlockFactoryTest {
                 HumanEvaluationBlockFactory.BLIND_FIELD)));
     }
 
+    @Test
+    void startsWithoutAnEvaluationTemplate() {
+        Block<HumanEvaluationBlockType> block = factory.createEmpty();
+        HumanEvaluationBlockConfiguration configuration =
+                (HumanEvaluationBlockConfiguration) block.getSpecificConfiguration();
+
+        assertNull(configuration.getTarget());
+        assertTrue(configuration.getTaskScript().isEmpty());
+        assertTrue(configuration.getCriteria().isEmpty());
+        assertTrue(block.getInputs().isEmpty());
+        assertFalse(outputNames(block).contains("works"));
+    }
+
     @Test
     void evidenceLeavesAsSeveralFiles() {
         // One screenshot is the common case, but a tester who took four must not lose three.