[TEST] Enhance failure message when bulk updates have failures

dakrone · dakrone · commit 1e4751ec470b · 2018-08-03T15:27:10.000-06:00
diff --git a/server/src/test/java/org/elasticsearch/action/bulk/BulkWithUpdatesIT.java b/server/src/test/java/org/elasticsearch/action/bulk/BulkWithUpdatesIT.java
@@ -402,7 +402,7 @@ public void testBulkUpdateLargerVolume() throws Exception {
                     .setScript(new Script(ScriptType.INLINE, CustomScriptPlugin.NAME, "ctx.op = \"delete\"", Collections.emptyMap())));
         }
         response = builder.execute().actionGet();
-        assertThat(response.hasFailures(), equalTo(false));
+        assertThat("expected no failures but got: " + response.buildFailureMessage(), response.hasFailures(), equalTo(false));
         assertThat(response.getItems().length, equalTo(numDocs));
         for (int i = 0; i < numDocs; i++) {
             assertThat(response.getItems()[i].getItemId(), equalTo(i));

Original file line number	Diff line number	Diff line change
`@@ -402,7 +402,7 @@ public void testBulkUpdateLargerVolume() throws Exception {`
`402`	`402`	`.setScript(new Script(ScriptType.INLINE, CustomScriptPlugin.NAME, "ctx.op = \"delete\"", Collections.emptyMap())));`
`403`	`403`	`}`
`404`	`404`	`response = builder.execute().actionGet();`
`405`		`- assertThat(response.hasFailures(), equalTo(false));`
	`405`	`+ assertThat("expected no failures but got: " + response.buildFailureMessage(), response.hasFailures(), equalTo(false));`
`406`	`406`	`assertThat(response.getItems().length, equalTo(numDocs));`
`407`	`407`	`for (int i = 0; i < numDocs; i++) {`
`408`	`408`	`assertThat(response.getItems()[i].getItemId(), equalTo(i));`