diff options
author | Jon Marius Venstad <venstad@gmail.com> | 2021-06-28 21:32:21 +0200 |
---|---|---|
committer | Jon Marius Venstad <venstad@gmail.com> | 2021-06-28 21:32:21 +0200 |
commit | 91301d6a6672c812e2ea0bb78fc1517d2d18dfb0 (patch) | |
tree | a7ea8439dd0fb0575ccec3ecf810797bd2bc2d4b /vespa-feed-client/src/test/java | |
parent | 0e888703ca92c8fe7a4a55e607e8e5a883c6bf4e (diff) |
Parse tracese properly
Diffstat (limited to 'vespa-feed-client/src/test/java')
-rw-r--r-- | vespa-feed-client/src/test/java/ai/vespa/feed/client/HttpFeedClientTest.java | 42 |
1 files changed, 35 insertions, 7 deletions
diff --git a/vespa-feed-client/src/test/java/ai/vespa/feed/client/HttpFeedClientTest.java b/vespa-feed-client/src/test/java/ai/vespa/feed/client/HttpFeedClientTest.java index dd0ead799ff..fe079270f9b 100644 --- a/vespa-feed-client/src/test/java/ai/vespa/feed/client/HttpFeedClientTest.java +++ b/vespa-feed-client/src/test/java/ai/vespa/feed/client/HttpFeedClientTest.java @@ -79,8 +79,22 @@ class HttpFeedClientTest { ("{\n" + " \"pathId\": \"/document/v1/ns/type/docid/0\",\n" + " \"id\": \"id:ns:type::0\",\n" + - " \"message\": \"Relax, take it easy,\",\n" + - " \"trace\": \"for there is nothing that we can do.\"\n" + + " \"message\": \"Relax, take it easy.\",\n" + + " \"trace\": [\n" + + " {\n" + + " \"message\": \"For there is nothing that we can do.\"\n" + + " },\n" + + " {\n" + + " \"fork\": [\n" + + " {\n" + + " \"message\": \"Relax, take is easy.\"\n" + + " },\n" + + " {\n" + + " \"message\": \"Blame it on me or blame it on you.\"\n" + + " }\n" + + " ]\n" + + " }\n" + + " ]\n" + "}").getBytes(UTF_8)); return CompletableFuture.completedFuture(response); } @@ -95,8 +109,22 @@ class HttpFeedClientTest { .get(); assertEquals(Result.Type.conditionNotMet, result.type()); assertEquals(id, result.documentId()); - assertEquals(Optional.of("Relax, take it easy,"), result.resultMessage()); - assertEquals(Optional.of("for there is nothing that we can do."), result.traceMessage()); + assertEquals(Optional.of("Relax, take it easy."), result.resultMessage()); + assertEquals(Optional.of("[\n" + + " {\n" + + " \"message\": \"For there is nothing that we can do.\"\n" + + " },\n" + + " {\n" + + " \"fork\": [\n" + + " {\n" + + " \"message\": \"Relax, take is easy.\"\n" + + " },\n" + + " {\n" + + " \"message\": \"Blame it on me or blame it on you.\"\n" + + " }\n" + + " ]\n" + + " }\n" + + " ]"), result.traceMessage()); // Put is a POST, and a Vespa error is a ResultException. dispatch.set((documentId, request) -> { @@ -111,7 +139,7 @@ class HttpFeedClientTest { " \"pathId\": \"/document/v1/ns/type/docid/0\",\n" + " \"id\": \"id:ns:type::0\",\n" + " \"message\": \"Ooops! ... I did it again.\",\n" + - " \"trace\": \"I played with your heart. Got lost in the game.\"\n" + + " \"trace\": [ { \"message\": \"I played with your heart. Got lost in the game.\" } ]\n" + "}").getBytes(UTF_8)); return CompletableFuture.completedFuture(response); } @@ -132,7 +160,7 @@ class HttpFeedClientTest { .get()); assertTrue(expected.getCause() instanceof ResultException); assertEquals("Ooops! ... I did it again.", expected.getCause().getMessage()); - assertEquals("I played with your heart. Got lost in the game.", ((ResultException) expected.getCause()).getTrace().get()); + assertEquals("[ { \"message\": \"I played with your heart. Got lost in the game.\" } ]", ((ResultException) expected.getCause()).getTrace().get()); // Handler error is a FeedException. @@ -148,7 +176,7 @@ class HttpFeedClientTest { " \"pathId\": \"/document/v1/ns/type/docid/0\",\n" + " \"id\": \"id:ns:type::0\",\n" + " \"message\": \"Alla ska i jorden.\",\n" + - " \"trace\": \"Din tid den kom, och senn så for den. \"\n" + + " \"trace\": [ { \"message\": \"Din tid den kom, och senn så for den.\" } ]\n" + "}").getBytes(UTF_8)); return CompletableFuture.completedFuture(response); } |