From 0ee3190841b3b91e3278e6850edd03a465ea23d4 Mon Sep 17 00:00:00 2001 From: Gordon Brown Date: Tue, 2 Oct 2018 11:24:56 -0600 Subject: [PATCH 1/2] HLRC: Update Start ILM API to use Validatable --- .../client/indexlifecycle/StartILMRequest.java | 10 ++-------- .../client/RequestConvertersTests.java | 4 ++-- .../indexlifecycle/StartILMRequestTests.java | 17 ++++++++--------- 3 files changed, 12 insertions(+), 19 deletions(-) diff --git a/client/rest-high-level/src/main/java/org/elasticsearch/client/indexlifecycle/StartILMRequest.java b/client/rest-high-level/src/main/java/org/elasticsearch/client/indexlifecycle/StartILMRequest.java index f6064c449016e..84cc844a92a98 100644 --- a/client/rest-high-level/src/main/java/org/elasticsearch/client/indexlifecycle/StartILMRequest.java +++ b/client/rest-high-level/src/main/java/org/elasticsearch/client/indexlifecycle/StartILMRequest.java @@ -19,19 +19,13 @@ package org.elasticsearch.client.indexlifecycle; -import org.elasticsearch.action.ActionRequestValidationException; -import org.elasticsearch.action.support.master.AcknowledgedRequest; +import org.elasticsearch.client.TimedRequest; -public class StartILMRequest extends AcknowledgedRequest { +public class StartILMRequest extends TimedRequest { public StartILMRequest() { } - @Override - public ActionRequestValidationException validate() { - return null; - } - @Override public int hashCode() { return 64; diff --git a/client/rest-high-level/src/test/java/org/elasticsearch/client/RequestConvertersTests.java b/client/rest-high-level/src/test/java/org/elasticsearch/client/RequestConvertersTests.java index 28cecaa67bfd7..a35a42be20838 100644 --- a/client/rest-high-level/src/test/java/org/elasticsearch/client/RequestConvertersTests.java +++ b/client/rest-high-level/src/test/java/org/elasticsearch/client/RequestConvertersTests.java @@ -1520,8 +1520,8 @@ public void testSetIndexLifecyclePolicy() throws Exception { public void testStartILM() throws Exception { StartILMRequest req = new StartILMRequest(); Map expectedParams = new HashMap<>(); - setRandomMasterTimeout(req, expectedParams); - setRandomTimeout(req::timeout, AcknowledgedRequest.DEFAULT_ACK_TIMEOUT, expectedParams); + setRandomMasterTimeout(req::setMasterTimeout, StartILMRequest.DEFAULT_MASTER_TIMEOUT, expectedParams); + setRandomTimeoutTimeValue(req::setTimeout, StartILMRequest.DEFAULT_TIMEOUT, expectedParams); Request request = RequestConverters.startILM(req); assertThat(request.getMethod(), equalTo(HttpPost.METHOD_NAME)); diff --git a/client/rest-high-level/src/test/java/org/elasticsearch/client/indexlifecycle/StartILMRequestTests.java b/client/rest-high-level/src/test/java/org/elasticsearch/client/indexlifecycle/StartILMRequestTests.java index f9da5d49e0629..449ef7d1678eb 100644 --- a/client/rest-high-level/src/test/java/org/elasticsearch/client/indexlifecycle/StartILMRequestTests.java +++ b/client/rest-high-level/src/test/java/org/elasticsearch/client/indexlifecycle/StartILMRequestTests.java @@ -19,23 +19,22 @@ package org.elasticsearch.client.indexlifecycle; -import org.elasticsearch.test.AbstractStreamableTestCase; +import org.elasticsearch.test.ESTestCase; +import org.elasticsearch.test.EqualsHashCodeTestUtils; -public class StartILMRequestTests extends AbstractStreamableTestCase { +public class StartILMRequestTests extends ESTestCase { - @Override - protected StartILMRequest createBlankInstance() { - return new StartILMRequest(); - } - - @Override protected StartILMRequest createTestInstance() { return new StartILMRequest(); } public void testValidate() { StartILMRequest request = createTestInstance(); - assertNull(request.validate()); + assertFalse(request.validate().isPresent()); + } + + public void testEqualsAndHashcode() { + EqualsHashCodeTestUtils.checkEqualsAndHashCode(createTestInstance(), (original) -> createTestInstance()); } } From ba8d0a7ddaacd660bd61d7b3afd032418609ec8e Mon Sep 17 00:00:00 2001 From: Gordon Brown Date: Tue, 2 Oct 2018 11:49:18 -0600 Subject: [PATCH 2/2] HLRC: Update Stop ILM API to use Validatable --- .../client/indexlifecycle/StopILMRequest.java | 10 ++-------- .../client/RequestConvertersTests.java | 4 ++-- .../indexlifecycle/StopILMRequestTests.java | 17 ++++++++--------- 3 files changed, 12 insertions(+), 19 deletions(-) diff --git a/client/rest-high-level/src/main/java/org/elasticsearch/client/indexlifecycle/StopILMRequest.java b/client/rest-high-level/src/main/java/org/elasticsearch/client/indexlifecycle/StopILMRequest.java index 4f56110931806..1695fc0dd7aea 100644 --- a/client/rest-high-level/src/main/java/org/elasticsearch/client/indexlifecycle/StopILMRequest.java +++ b/client/rest-high-level/src/main/java/org/elasticsearch/client/indexlifecycle/StopILMRequest.java @@ -19,19 +19,13 @@ package org.elasticsearch.client.indexlifecycle; -import org.elasticsearch.action.ActionRequestValidationException; -import org.elasticsearch.action.support.master.AcknowledgedRequest; +import org.elasticsearch.client.TimedRequest; -public class StopILMRequest extends AcknowledgedRequest { +public class StopILMRequest extends TimedRequest { public StopILMRequest() { } - @Override - public ActionRequestValidationException validate() { - return null; - } - @Override public int hashCode() { return 75; diff --git a/client/rest-high-level/src/test/java/org/elasticsearch/client/RequestConvertersTests.java b/client/rest-high-level/src/test/java/org/elasticsearch/client/RequestConvertersTests.java index a35a42be20838..1bc164a851995 100644 --- a/client/rest-high-level/src/test/java/org/elasticsearch/client/RequestConvertersTests.java +++ b/client/rest-high-level/src/test/java/org/elasticsearch/client/RequestConvertersTests.java @@ -1532,8 +1532,8 @@ public void testStartILM() throws Exception { public void testStopILM() throws Exception { StopILMRequest req = new StopILMRequest(); Map expectedParams = new HashMap<>(); - setRandomMasterTimeout(req, expectedParams); - setRandomTimeout(req::timeout, AcknowledgedRequest.DEFAULT_ACK_TIMEOUT, expectedParams); + setRandomMasterTimeout(req::setMasterTimeout, StopILMRequest.DEFAULT_MASTER_TIMEOUT, expectedParams); + setRandomTimeoutTimeValue(req::setTimeout, StopILMRequest.DEFAULT_TIMEOUT, expectedParams); Request request = RequestConverters.stopILM(req); assertThat(request.getMethod(), equalTo(HttpPost.METHOD_NAME)); diff --git a/client/rest-high-level/src/test/java/org/elasticsearch/client/indexlifecycle/StopILMRequestTests.java b/client/rest-high-level/src/test/java/org/elasticsearch/client/indexlifecycle/StopILMRequestTests.java index 8f1a3756c8ab3..f1618f3f0f0e3 100644 --- a/client/rest-high-level/src/test/java/org/elasticsearch/client/indexlifecycle/StopILMRequestTests.java +++ b/client/rest-high-level/src/test/java/org/elasticsearch/client/indexlifecycle/StopILMRequestTests.java @@ -19,23 +19,22 @@ package org.elasticsearch.client.indexlifecycle; -import org.elasticsearch.test.AbstractStreamableTestCase; +import org.elasticsearch.test.ESTestCase; +import org.elasticsearch.test.EqualsHashCodeTestUtils; -public class StopILMRequestTests extends AbstractStreamableTestCase { +public class StopILMRequestTests extends ESTestCase { - @Override - protected StopILMRequest createBlankInstance() { - return new StopILMRequest(); - } - - @Override protected StopILMRequest createTestInstance() { return new StopILMRequest(); } public void testValidate() { StopILMRequest request = createTestInstance(); - assertNull(request.validate()); + assertFalse(request.validate().isPresent()); + } + + public void testEqualsAndHashcode() { + EqualsHashCodeTestUtils.checkEqualsAndHashCode(createTestInstance(), (original) -> createTestInstance()); } }