Updated test

linkedin · Feb 15, 2024 · eac5326 · eac5326
1 parent e63d984
commit eac5326
Showing 1 changed file with 21 additions and 7 deletions.
diff --git a/.../tests/tests-allavro/src/test/java/com/linkedin/avroutil1/builder/SpecificRecordTest.java b/.../tests/tests-allavro/src/test/java/com/linkedin/avroutil1/builder/SpecificRecordTest.java
@@ -255,7 +255,9 @@ public void testSpecificRecordBuilder14(String stringField, String package$, Flo
     Assert.assertSame(builderTester.get(0), stringField);
     Assert.assertSame(builderTester.get(1), package$);
     Assert.assertSame(builderTester.get(6), min);
-    Assert.assertSame(builderTester.get(7), arrayOfRecord);
+
+    // Equal, not same as the Record has string fields
+    Assert.assertEquals(builderTester.get(7), arrayOfRecord);
     Assert.assertSame(builderTester.get(10), simpleUnion);
     Assert.assertSame(builderTester.get(11), fixedType);
     Assert.assertEquals(builderTester.get(2), exception);
@@ -391,7 +393,9 @@ public void testSpecificRecordBuilder15(String stringField, String package$, Flo
     Assert.assertSame(builderTester.get(0), stringField);
     Assert.assertSame(builderTester.get(1), package$);
     Assert.assertSame(builderTester.get(6), min);
-    Assert.assertSame(builderTester.get(7), arrayOfRecord);
+
+    // Equal, not same as the Record has string fields
+    Assert.assertEquals(builderTester.get(7), arrayOfRecord);
     Assert.assertSame(builderTester.get(10), simpleUnion);
     Assert.assertSame(builderTester.get(11), fixedType);
     Assert.assertEquals(builderTester.get(2), exception);
@@ -528,7 +532,9 @@ public void testSpecificRecordBuilder16(String stringField, String package$, Flo
     Assert.assertSame(builderTester.get(0), stringField);
     Assert.assertSame(builderTester.get(1), package$);
     Assert.assertSame(builderTester.get(6), min);
-    Assert.assertSame(builderTester.get(7), arrayOfRecord);
+
+    // Equal, not same as the Record has string fields
+    Assert.assertEquals(builderTester.get(7), arrayOfRecord);
     Assert.assertSame(builderTester.get(10), simpleUnion);
     Assert.assertSame(builderTester.get(11), fixedType);
     Assert.assertEquals(builderTester.get(2), exception);
@@ -664,7 +670,9 @@ public void testSpecificRecordBuilder17(String stringField, String package$, Flo
     Assert.assertSame(builderTester.get(0), stringField);
     Assert.assertSame(builderTester.get(1), package$);
     Assert.assertSame(builderTester.get(6), min);
-    Assert.assertSame(builderTester.get(7), arrayOfRecord);
+
+    // Equal, not same as the Record has string fields
+    Assert.assertEquals(builderTester.get(7), arrayOfRecord);
     Assert.assertSame(builderTester.get(10), simpleUnion);
     Assert.assertSame(builderTester.get(11), fixedType);
     Assert.assertEquals(builderTester.get(2), exception);
@@ -804,7 +812,9 @@ public void testSpecificRecordBuilder18(String stringField, String package$, Flo
     Assert.assertSame(builderTester.get(0), stringField);
     Assert.assertSame(builderTester.get(1), package$);
     Assert.assertSame(builderTester.get(6), min);
-    Assert.assertSame(builderTester.get(7), arrayOfRecord);
+
+    // Equal, not same as the Record has string fields
+    Assert.assertEquals(builderTester.get(7), arrayOfRecord);
     Assert.assertSame(builderTester.get(10), simpleUnion);
     Assert.assertSame(builderTester.get(11), fixedType);
     Assert.assertEquals(builderTester.get(2), exception);
@@ -946,7 +956,9 @@ public void testSpecificRecordBuilder19(String stringField, String package$, Flo
     Assert.assertSame(builderTester.get(0), stringField);
     Assert.assertSame(builderTester.get(1), package$);
     Assert.assertSame(builderTester.get(6), min);
-    Assert.assertSame(builderTester.get(7), arrayOfRecord);
+
+    // Equal, not same as the Record has string fields
+    Assert.assertEquals(builderTester.get(7), arrayOfRecord);
     Assert.assertSame(builderTester.get(10), simpleUnion);
     Assert.assertSame(builderTester.get(11), fixedType);
     Assert.assertEquals(builderTester.get(2), exception);
@@ -1224,7 +1236,9 @@ public void testSpecificRecordBuilder111(String stringField, String package$, Fl
     Assert.assertSame(builderTester.get(0), stringField);
     Assert.assertSame(builderTester.get(1), package$);
     Assert.assertSame(builderTester.get(6), min);
-    Assert.assertSame(builderTester.get(7), arrayOfRecord);
+
+    // Equal, not same as the Record has string fields
+    Assert.assertEquals(builderTester.get(7), arrayOfRecord);
     Assert.assertSame(builderTester.get(10), simpleUnion);
     Assert.assertSame(builderTester.get(11), fixedType);
     Assert.assertEquals(builderTester.get(2), exception);