From a03ee581ba500e5e324d597a84b1b03e1cd29eaa Mon Sep 17 00:00:00 2001
From: Antonin Delpeuch <antonin@delpeuch.eu>
Date: Fri, 10 May 2024 06:18:32 +0200
Subject: [PATCH] tests: remove dependency of history entry tests to column
 addition op (#6573)

---
 .../history/FileHistoryEntryManagerTests.java |  3 +--
 .../refine/history/HistoryEntryTests.java     | 27 ++++++++++++-------
 2 files changed, 18 insertions(+), 12 deletions(-)

diff --git a/main/tests/server/src/com/google/refine/history/FileHistoryEntryManagerTests.java b/main/tests/server/src/com/google/refine/history/FileHistoryEntryManagerTests.java
index 96d300456237..014890138fd9 100644
--- a/main/tests/server/src/com/google/refine/history/FileHistoryEntryManagerTests.java
+++ b/main/tests/server/src/com/google/refine/history/FileHistoryEntryManagerTests.java
@@ -14,7 +14,6 @@
 import com.google.refine.io.FileHistoryEntryManager;
 import com.google.refine.model.Project;
 import com.google.refine.operations.OperationRegistry;
-import com.google.refine.operations.column.ColumnAdditionOperation;
 import com.google.refine.util.TestUtils;
 
 public class FileHistoryEntryManagerTests extends RefineTest {
@@ -25,7 +24,7 @@ public class FileHistoryEntryManagerTests extends RefineTest {
     @BeforeMethod
     public void setUp() {
         project = mock(Project.class);
-        OperationRegistry.registerOperation(getCoreModule(), "column-addition", ColumnAdditionOperation.class);
+        OperationRegistry.registerOperation(getCoreModule(), "mock-operation", HistoryEntryTests.MockOperation.class);
     }
 
     @Test
diff --git a/main/tests/server/src/com/google/refine/history/HistoryEntryTests.java b/main/tests/server/src/com/google/refine/history/HistoryEntryTests.java
index 79b34102bd27..420c14ea1b8f 100644
--- a/main/tests/server/src/com/google/refine/history/HistoryEntryTests.java
+++ b/main/tests/server/src/com/google/refine/history/HistoryEntryTests.java
@@ -31,14 +31,15 @@
 
 import java.io.IOException;
 
+import com.fasterxml.jackson.annotation.JsonProperty;
 import org.testng.annotations.BeforeMethod;
 import org.testng.annotations.BeforeTest;
 import org.testng.annotations.Test;
 
 import com.google.refine.RefineTest;
+import com.google.refine.model.AbstractOperation;
 import com.google.refine.model.Project;
 import com.google.refine.operations.OperationRegistry;
-import com.google.refine.operations.column.ColumnAdditionOperation;
 import com.google.refine.util.TestUtils;
 
 public class HistoryEntryTests extends RefineTest {
@@ -47,14 +48,9 @@ public class HistoryEntryTests extends RefineTest {
             + "\"id\":1533633623158,"
             + "\"description\":\"Create new column uri based on column country by filling 269 rows with grel:\\\"https://www.wikidata.org/wiki/\\\"+cell.recon.match.id\","
             + "\"time\":\"2018-08-07T09:06:37Z\","
-            + "\"operation\":{\"op\":\"core/column-addition\","
-            + "   \"description\":\"Create column uri at index 2 based on column country using expression grel:\\\"https://www.wikidata.org/wiki/\\\"+cell.recon.match.id\","
-            + "   \"engineConfig\":{\"mode\":\"row-based\",\"facets\":[]},"
-            + "   \"newColumnName\":\"uri\","
-            + "   \"columnInsertIndex\":2,"
-            + "   \"baseColumnName\":\"country\","
-            + "   \"expression\":\"grel:\\\"https://www.wikidata.org/wiki/\\\"+cell.recon.match.id\","
-            + "   \"onError\":\"set-to-blank\"}"
+            + "\"operation\":{\"op\":\"core/mock-operation\","
+            + "   \"description\":\"some description\","
+            + "   \"foo\":\"bar\"}"
             + "}";
 
     public static final String unknownOperationJson = "{"
@@ -69,9 +65,20 @@ public class HistoryEntryTests extends RefineTest {
 
     Project project;
 
+    public static class MockOperation extends AbstractOperation {
+
+        @JsonProperty("foo")
+        public String someParameter = "bar";
+
+        @Override
+        public String getJsonDescription() {
+            return "some description";
+        }
+    }
+
     @BeforeTest
     public void register() {
-        OperationRegistry.registerOperation(getCoreModule(), "column-addition", ColumnAdditionOperation.class);
+        OperationRegistry.registerOperation(getCoreModule(), "mock-operation", MockOperation.class);
     }
 
     @BeforeMethod