Fix SamlServiceProviderDocumentTests (#54718) (#54724)

Don't assume byte for byte equality because internal structures do not guarantee order
elastic · Apr 3, 2020 · 3205d2b · 3205d2b
1 parent 8579eda
commit 3205d2b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...r/src/test/java/org/elasticsearch/xpack/idp/saml/sp/SamlServiceProviderDocumentTests.java b/...r/src/test/java/org/elasticsearch/xpack/idp/saml/sp/SamlServiceProviderDocumentTests.java
@@ -31,7 +31,7 @@
 import java.util.Map;
 import java.util.stream.Collectors;
 
-import static org.hamcrest.MatcherAssert.assertThat;
+import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertToXContentEquivalent;
 import static org.hamcrest.Matchers.emptyIterable;
 import static org.hamcrest.Matchers.equalTo;
 import static org.hamcrest.Matchers.not;
@@ -146,7 +146,7 @@ private SamlServiceProviderDocument assertXContentRoundTrip(SamlServiceProviderD
             assertThat(obj2, equalTo(obj1));
 
             final BytesReference bytes2 = XContentHelper.toXContent(obj2, xContentType, humanReadable);
-            assertThat(bytes2, equalTo(bytes1));
+            assertToXContentEquivalent(bytes1, bytes2, xContentType);
 
             return obj2;
         }