fix: Fixing Flaky Tests (#725)

priyanka-28 · web-flow · commit facab196a737 · 2022-12-13T09:20:38.000+05:30
diff --git a/src/main/java/com/twilio/jwt/taskrouter/Policy.java b/src/main/java/com/twilio/jwt/taskrouter/Policy.java
@@ -4,6 +4,7 @@
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonProperty;
+import com.fasterxml.jackson.annotation.JsonPropertyOrder;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.twilio.http.HttpMethod;
 
@@ -17,6 +18,7 @@
 @JsonAutoDetect(fieldVisibility = JsonAutoDetect.Visibility.ANY)
 @JsonInclude(JsonInclude.Include.NON_NULL)
 @JsonIgnoreProperties(ignoreUnknown = true)
+@JsonPropertyOrder(alphabetic = true)
 @ToString
 public class Policy {
 
diff --git a/src/test/java/com/twilio/jwt/taskrouter/PolicyTest.java b/src/test/java/com/twilio/jwt/taskrouter/PolicyTest.java
@@ -27,7 +27,7 @@ public void testToJson() throws IOException {
             .build();
 
         Assert.assertEquals(
-                "{\"url\":\"http://localhost\",\"method\":\"GET\",\"query_filter\":{\"foo\":{\"required\":true}},\"post_filter\":{},\"allow\":true}",
+                "{\"allow\":true,\"method\":\"GET\",\"post_filter\":{},\"query_filter\":{\"foo\":{\"required\":true}},\"url\":\"http://localhost\"}",
                 p.toJson());
     }
 

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public void testToJson() throws IOException {`
`27`	`27`	`.build();`
`28`	`28`
`29`	`29`	`Assert.assertEquals(`
`30`		`- "{\"url\":\"http://localhost\",\"method\":\"GET\",\"query_filter\":{\"foo\":{\"required\":true}},\"post_filter\":{},\"allow\":true}",`
	`30`	`+ "{\"allow\":true,\"method\":\"GET\",\"post_filter\":{},\"query_filter\":{\"foo\":{\"required\":true}},\"url\":\"http://localhost\"}",`
`31`	`31`	`p.toJson());`
`32`	`32`	`}`
`33`	`33`