Skip to content

Commit 1a85d5b

Browse files
committed
Fix test, address reviewer comments.
1 parent 104b723 commit 1a85d5b

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

client/rest-high-level/src/test/java/org/elasticsearch/client/RequestConvertersTests.java

+5-1
Original file line numberDiff line numberDiff line change
@@ -167,6 +167,7 @@
167167
import static org.hamcrest.CoreMatchers.equalTo;
168168
import static org.hamcrest.Matchers.hasEntry;
169169
import static org.hamcrest.Matchers.hasKey;
170+
import static org.hamcrest.Matchers.is;
170171
import static org.hamcrest.Matchers.notNullValue;
171172
import static org.hamcrest.Matchers.nullValue;
172173

@@ -1958,12 +1959,15 @@ public void testSnapshotsStatus() {
19581959
SnapshotsStatusRequest snapshotsStatusRequest = new SnapshotsStatusRequest(repository, snapshots);
19591960
setRandomMasterTimeout(snapshotsStatusRequest, expectedParams);
19601961
snapshotsStatusRequest.ignoreUnavailable(ignoreUnavailable);
1961-
expectedParams.put("ignore_unavailable", Boolean.toString(ignoreUnavailable));
1962+
if (ignoreUnavailable) {
1963+
expectedParams.put("ignore_unavailable", Boolean.toString(true));
1964+
}
19621965

19631966
Request request = RequestConverters.snapshotsStatus(snapshotsStatusRequest);
19641967
assertThat(request.getEndpoint(), equalTo(endpoint));
19651968
assertThat(request.getMethod(), equalTo(HttpGet.METHOD_NAME));
19661969
assertThat(request.getParameters(), equalTo(expectedParams));
1970+
assertThat(request.getEntity(), is(nullValue()));
19671971
}
19681972

19691973
public void testDeleteSnapshot() {

0 commit comments

Comments
 (0)