[avro] branch master updated: AVRO-3690: [Java] Fix flaky test ‘testMultipleFieldAliases’

[nielsbasjes]

This is an automated email from the ASF dual-hosted git repository.

nielsbasjes pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/avro.git


The following commit(s) were added to refs/heads/master by this push:
     new 30c4cbfde AVRO-3690: [Java] Fix flaky test ‘testMultipleFieldAliases’
30c4cbfde is described below

commit 30c4cbfde573100d60f1e27dac26bd4803ca6d31
Author: Zhengxu Jin <33077399+jinzhen...@users.noreply.github.com>
AuthorDate: Sat Feb 18 09:44:41 2023 -0600

    AVRO-3690: [Java] Fix flaky test ‘testMultipleFieldAliases’
    
    Co-authored-by: Martin Grigorov <marti...@users.noreply.github.com>
---
 .../test/java/org/apache/avro/reflect/TestReflect.java   | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git 
a/lang/java/avro/src/test/java/org/apache/avro/reflect/TestReflect.java 
b/lang/java/avro/src/test/java/org/apache/avro/reflect/TestReflect.java
index 379a3db18..d7149ff53 100644
--- a/lang/java/avro/src/test/java/org/apache/avro/reflect/TestReflect.java
+++ b/lang/java/avro/src/test/java/org/apache/avro/reflect/TestReflect.java
@@ -1281,13 +1281,15 @@ public class TestReflect {
   }
 
   @Test
-  void multipleFieldAliases() {
-
-    Schema expectedSchema = 
SchemaBuilder.record(ClassWithMultipleAliasesOnField.class.getSimpleName())
-        
.namespace("org.apache.avro.reflect.TestReflect").fields().name("primitiveField").aliases("alias1",
 "alias2")
-        
.type(Schema.create(org.apache.avro.Schema.Type.INT)).noDefault().endRecord();
-
-    check(ClassWithMultipleAliasesOnField.class, expectedSchema.toString());
+  public void testMultipleFieldAliases() {
+    Field field = new Field("primitiveField", Schema.create(Schema.Type.INT));
+    field.addAlias("alias1");
+    field.addAlias("alias2");
+    Schema avroMultiMeta = 
Schema.createRecord("ClassWithMultipleAliasesOnField", null, 
"org.apache.avro.reflect.TestReflect",
+        false, Arrays.asList(field));
+
+    Schema schema = ReflectData.get().getSchema(ClassWithAliasOnField.class);
+    assertEquals(avroMultiMeta, schema);
   }
 
   private static class OptionalTest {