Update PropertyOrderTest to be more meaningful

eclipse-ee4j · Oct 14, 2019 · aeff4be · aeff4be
1 parent c12865a
commit aeff4be
Showing 1 changed file with 18 additions and 16 deletions.
diff --git a/src/test/java/org/eclipse/yasson/customization/PropertyOrderTest.java b/src/test/java/org/eclipse/yasson/customization/PropertyOrderTest.java
@@ -99,30 +99,32 @@ public void testLexicographicalPropertyOrderRenamedProperties() {
 
     @Test
     public void testJsonbPropertyOrderOnRenamedProperties() {
-        assertEquals("{\"from\":10,\"count\":11}", defaultJsonb.toJson(new Range(10, 11)));
+        assertEquals("{\"c\":11,\"d\":10,\"aExtra\":\"extra\"}", defaultJsonb.toJson(new Range(10, 11)));
     }
-
-    @JsonbPropertyOrder(
-            {
-                    "fromIndex",
-                    "numberOfItems"
-            }
-    )
+
+    // By default, this object would use A-X ordering with property order:
+    // anExtraProp, propA, propB
+    // But with JsonbPropertyOrder we will put props 'propB' and 'propA' first, and leftovers will go at the end, resulting in:
+    // propB, propA, anExtraProp
+    @JsonbPropertyOrder({"propB","propA"})
     public class Range {
 
-        @JsonbProperty("from")
-        public final int fromIndex;
+        @JsonbProperty("d")
+        public final int propA;
 
-        @JsonbProperty("count")
-        public final int numberOfItems;
+        @JsonbProperty("c")
+        public final int propB;
+
+        @JsonbProperty("aExtra")
+        public final String anExtraProp = "extra";
 
         @JsonbCreator
         public Range(
-                @JsonbProperty("from") int fromIndex,
-                @JsonbProperty("count") int numberOfItems
+                @JsonbProperty("d") int propA,
+                @JsonbProperty("c") int propB
         ) {
-            this.fromIndex = fromIndex;
-            this. numberOfItems = numberOfItems;
+            this.propA = propA;
+            this. propB = propB;
         }
     }
 }