diff options
author | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2020-01-07 16:37:40 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-01-07 16:37:40 +0100 |
commit | 425aad7766cd96a397eb48e49c0f8483eefb3e1d (patch) | |
tree | 71472f7b3a5a443d744de595dabd1111bb328df4 /vespa-http-client/src | |
parent | f5885708bb0a9b96b0feca718cabbd93a54f6ed5 (diff) | |
parent | aa055ffe9699168ea1c14ba405248352d4a07452 (diff) |
Merge pull request #11681 from vespa-engine/bjorncs/apache-commons-libraries-cleanup
Bjorncs/apache commons libraries cleanup
Diffstat (limited to 'vespa-http-client/src')
-rw-r--r-- | vespa-http-client/src/test/java/com/yahoo/vespa/http/client/core/XmlFeedReaderTest.java | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/vespa-http-client/src/test/java/com/yahoo/vespa/http/client/core/XmlFeedReaderTest.java b/vespa-http-client/src/test/java/com/yahoo/vespa/http/client/core/XmlFeedReaderTest.java index 9abbf916cff..925f3105878 100644 --- a/vespa-http-client/src/test/java/com/yahoo/vespa/http/client/core/XmlFeedReaderTest.java +++ b/vespa-http-client/src/test/java/com/yahoo/vespa/http/client/core/XmlFeedReaderTest.java @@ -1,8 +1,9 @@ // Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.http.client.core; +import com.fasterxml.jackson.databind.JsonNode; +import com.fasterxml.jackson.dataformat.xml.XmlMapper; import com.yahoo.vespa.http.client.FeedClient; -import org.apache.commons.lang3.StringEscapeUtils; import org.junit.Test; import org.mockito.invocation.InvocationOnMock; import org.mockito.stubbing.Answer; @@ -17,7 +18,6 @@ import java.util.List; import java.util.concurrent.atomic.AtomicBoolean; import java.util.concurrent.atomic.AtomicInteger; -import static org.hamcrest.CoreMatchers.containsString; import static org.hamcrest.MatcherAssert.assertThat; import static org.hamcrest.core.Is.is; import static org.mockito.ArgumentMatchers.any; @@ -32,6 +32,8 @@ public class XmlFeedReaderTest { private final static String feedResource3 = "/xml-challenge2.xml"; private final static String feedResource4 = "/xml-challenge3.xml"; + private static final XmlMapper xmlMapper = new XmlMapper(); + private final String updateDocUpdate = "<?xml version=\"1.0\"?>\n" + "<vespafeed>\n" + @@ -232,13 +234,9 @@ public class XmlFeedReaderTest { InputStream inputStream2 = XmlFeedReaderTest.class.getResourceAsStream(filename); String rawXML = new java.util.Scanner(inputStream2, "UTF-8").useDelimiter("\\A").next(); - String rawDoc = rawXML.toString().split("<document")[1].split("</document>")[0]; - assertThat(rawDoc.length() > 30, is(true)); - - String decodedRawXml = StringEscapeUtils.unescapeXml(rawDoc); - String decodedDoc = StringEscapeUtils.unescapeXml(document); - - assertThat(decodedDoc, containsString(decodedRawXml)); + JsonNode decodedDocument = xmlMapper.readTree(document); + JsonNode rawDocuments = xmlMapper.readTree(rawXML); + assertThat(decodedDocument, is(rawDocuments.get("document"))); } @Test public void testCData() throws Exception { |