commit 90c35457c956074f2fdf2c2b9ad482ccfdafc8a2
author Marc Kupietz <kupietz@ids-mannheim.de> Wed Sep 24 14:34:52 2025 +0200
committer Marc Kupietz <kupietz@ids-mannheim.de> Wed Sep 24 14:34:52 2025 +0200
tree 4953fb92e70f04c348f141461a0ea38979501ee3
parent 8b4667341a9cc6f94af40634092ee8b425a12676

Fix assert argument order in UserdataTest

Change-Id: I4d1fe89902dcabc60e89a757d5ac5468d01e0418

src/test/java/de/ids_mannheim/korap/user/UserdataTest.java

diff --git a/src/test/java/de/ids_mannheim/korap/user/UserdataTest.java b/src/test/java/de/ids_mannheim/korap/user/UserdataTest.java
index 499208f..3e3d2de 100644
--- a/src/test/java/de/ids_mannheim/korap/user/UserdataTest.java
+++ b/src/test/java/de/ids_mannheim/korap/user/UserdataTest.java
@@ -32,7 +32,7 @@
         Userdata userData = new UserSettingProcessor();
         userData.read(new HashMap<>(), false);
         String jsonSettings = userData.serialize();
-        assertEquals(jsonSettings, "{}");
+        assertEquals("{}", jsonSettings);
     }
 
     @Test
@@ -40,7 +40,7 @@
         Userdata userData = new UserSettingProcessor();
         userData.read(null, false);
         String jsonSettings = userData.serialize();
-        assertEquals(jsonSettings, "{}");
+        assertEquals("{}", jsonSettings);
     }
 
     // EM: based on MH code, supposedly to validate entries like email
@@ -114,12 +114,12 @@
         settings.read(m, true);
         assertNotEquals(m.size(), settings.size());
         assertEquals(settings.defaultFields().length, settings.size());
-        assertEquals(settings.get(Attributes.DEFAULT_FOUNDRY_RELATION),
-                "rel_1");
-        assertEquals(settings.get(Attributes.DEFAULT_FOUNDRY_POS), "pos_1");
-        assertEquals(settings.get(Attributes.DEFAULT_FOUNDRY_LEMMA), "lemma_1");
-        assertEquals(settings.get(Attributes.DEFAULT_FOUNDRY_CONSTITUENT),
-                "const_1");
+        assertEquals("rel_1",
+            settings.get(Attributes.DEFAULT_FOUNDRY_RELATION));
+        assertEquals("pos_1", settings.get(Attributes.DEFAULT_FOUNDRY_POS));
+        assertEquals("lemma_1", settings.get(Attributes.DEFAULT_FOUNDRY_LEMMA));
+        assertEquals("const_1",
+            settings.get(Attributes.DEFAULT_FOUNDRY_CONSTITUENT));
         assertEquals(10, settings.get(Attributes.PAGE_LENGTH));
     }
 
@@ -134,7 +134,7 @@
             details.setData(JsonUtils.toJSON(m));
             String[] missing = details.findMissingFields();
             assertEquals(1, missing.length);
-            assertEquals(missing[0], "email");
+            assertEquals("email", missing[0]);
             details.checkRequired();
         });
     }
@@ -153,7 +153,7 @@
         array.add("message");
         details.setField("errors", array);
         assertEquals(100, details.get("/errors/0"));
-        assertEquals(details.get("/errors/1"), "message");
+        assertEquals("message", details.get("/errors/1"));
     }
 
     @Test
@@ -166,7 +166,7 @@
         UserDetails details2 = new UserDetails(-1);
         details2.setField(Attributes.COUNTRY, "Germany");
         details.update(details2);
-        assertEquals(details.get(Attributes.FIRSTNAME), "first");
-        assertEquals(details.get(Attributes.COUNTRY), "Germany");
+        assertEquals("first", details.get(Attributes.FIRSTNAME));
+        assertEquals("Germany", details.get(Attributes.COUNTRY));
     }
 }