Transpose expected and actual, and remove duplicate info from message.
Previously:
```
> Throwable java.lang.AssertionError: Expected all shards successful but got successful [8] total [9]
> Expected: <8>
>      but: was <9>
```
Now:
```
> Throwable java.lang.AssertionError: Expected all shards successful
> Expected: <9>
>      but: was <8>
```
@@ -346,15 +346,15 @@ public class ElasticsearchAssertions {
public static void assertAllSuccessful(BroadcastResponse response) {
assertNoFailures(response);
- assertThat("Expected all shards successful but got successful [" + response.getSuccessfulShards() + "] total [" + response.getTotalShards() + "]",
- response.getTotalShards(), equalTo(response.getSuccessfulShards()));
+ assertThat("Expected all shards successful",
+ response.getSuccessfulShards(), equalTo(response.getTotalShards()));
assertVersionSerializable(response);
}
public static void assertAllSuccessful(SearchResponse response) {
assertNoFailures(response);
- assertThat("Expected all shards successful but got successful [" + response.getSuccessfulShards() + "] total [" + response.getTotalShards() + "]",
- response.getTotalShards(), equalTo(response.getSuccessfulShards()));
+ assertThat("Expected all shards successful",
+ response.getSuccessfulShards(), equalTo(response.getTotalShards()));
assertVersionSerializable(response);
}
