This is an automated email from the ASF dual-hosted git repository.
lgallinat pushed a commit to branch feature/GEODE-3781
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/feature/GEODE-3781 by this
push:
new 70f760f Change getErrorMessage() to getStatus() in CliFunctions
70f760f is described below
commit 70f760fac4c3379659ae1292ba6ce83fc05a3034
Author: Lynn Gallinat <[email protected]>
AuthorDate: Fri Dec 15 16:45:26 2017 -0800
Change getErrorMessage() to getStatus() in CliFunctions
---
.../src/test/java/org/apache/geode/connectors/jdbc/JdbcDUnitTest.java | 4 ++--
.../connectors/jdbc/internal/cli/AlterConnectionFunctionTest.java | 2 +-
.../geode/connectors/jdbc/internal/cli/AlterMappingFunctionTest.java | 2 +-
.../connectors/jdbc/internal/cli/CreateConnectionFunctionTest.java | 2 +-
.../geode/connectors/jdbc/internal/cli/CreateMappingFunctionTest.java | 2 +-
.../connectors/jdbc/internal/cli/DestroyConnectionFunctionTest.java | 2 +-
.../jdbc/internal/cli/DestroyMappingCommandFunctionTest.java | 2 +-
7 files changed, 8 insertions(+), 8 deletions(-)
diff --git
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/JdbcDUnitTest.java
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/JdbcDUnitTest.java
index 293fc51..691548e 100644
---
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/JdbcDUnitTest.java
+++
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/JdbcDUnitTest.java
@@ -244,8 +244,8 @@ public class JdbcDUnitTest implements Serializable {
}
private void createJdbcConnection() {
- final String commandStr = "create jdbc-connection --name=" +
CONNECTION_NAME + " --url="
- + CONNECTION_URL;
+ final String commandStr =
+ "create jdbc-connection --name=" + CONNECTION_NAME + " --url=" +
CONNECTION_URL;
gfsh.executeAndAssertThat(commandStr).statusIsSuccess();
}
diff --git
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionFunctionTest.java
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionFunctionTest.java
index bb17cc4..4f36931 100644
---
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionFunctionTest.java
+++
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionFunctionTest.java
@@ -134,7 +134,7 @@ public class AlterConnectionFunctionTest {
ArgumentCaptor<CliFunctionResult> argument =
ArgumentCaptor.forClass(CliFunctionResult.class);
verify(resultSender, times(1)).lastResult(argument.capture());
-
assertThat(argument.getValue().getErrorMessage()).contains(CONNECTION_NAME);
+ assertThat(argument.getValue().getStatus()).contains(CONNECTION_NAME);
}
@Test
diff --git
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/AlterMappingFunctionTest.java
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/AlterMappingFunctionTest.java
index 97c1c35..d4b8a6c 100644
---
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/AlterMappingFunctionTest.java
+++
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/AlterMappingFunctionTest.java
@@ -139,7 +139,7 @@ public class AlterMappingFunctionTest {
ArgumentCaptor<CliFunctionResult> argument =
ArgumentCaptor.forClass(CliFunctionResult.class);
verify(resultSender, times(1)).lastResult(argument.capture());
- assertThat(argument.getValue().getErrorMessage()).contains(REGION_NAME);
+ assertThat(argument.getValue().getStatus()).contains(REGION_NAME);
}
@Test
diff --git
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionFunctionTest.java
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionFunctionTest.java
index 48a3072..3cb78d1 100644
---
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionFunctionTest.java
+++
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionFunctionTest.java
@@ -129,7 +129,7 @@ public class CreateConnectionFunctionTest {
ArgumentCaptor<CliFunctionResult> argument =
ArgumentCaptor.forClass(CliFunctionResult.class);
verify(resultSender, times(1)).lastResult(argument.capture());
- assertThat(argument.getValue().getErrorMessage())
+ assertThat(argument.getValue().getStatus())
.contains(ConnectionConfigExistsException.class.getName());
}
diff --git
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/CreateMappingFunctionTest.java
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/CreateMappingFunctionTest.java
index 9d7a32f..ffb9b9d 100644
---
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/CreateMappingFunctionTest.java
+++
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/CreateMappingFunctionTest.java
@@ -131,7 +131,7 @@ public class CreateMappingFunctionTest {
ArgumentCaptor<CliFunctionResult> argument =
ArgumentCaptor.forClass(CliFunctionResult.class);
verify(resultSender, times(1)).lastResult(argument.capture());
- assertThat(argument.getValue().getErrorMessage())
+ assertThat(argument.getValue().getStatus())
.contains(RegionMappingExistsException.class.getName());
}
}
diff --git
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionFunctionTest.java
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionFunctionTest.java
index 4f5f14c..17284ea 100644
---
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionFunctionTest.java
+++
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionFunctionTest.java
@@ -118,7 +118,7 @@ public class DestroyConnectionFunctionTest {
ArgumentCaptor<CliFunctionResult> argument =
ArgumentCaptor.forClass(CliFunctionResult.class);
verify(resultSender, times(1)).lastResult(argument.capture());
- assertThat(argument.getValue().getErrorMessage())
+ assertThat(argument.getValue().getStatus())
.contains("Connection named \"" + connectionName + "\" not found");
}
}
diff --git
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyMappingCommandFunctionTest.java
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyMappingCommandFunctionTest.java
index a1224c0..bc1b163 100644
---
a/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyMappingCommandFunctionTest.java
+++
b/geode-connectors/src/test/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyMappingCommandFunctionTest.java
@@ -118,7 +118,7 @@ public class DestroyMappingCommandFunctionTest {
ArgumentCaptor<CliFunctionResult> argument =
ArgumentCaptor.forClass(CliFunctionResult.class);
verify(resultSender, times(1)).lastResult(argument.capture());
- assertThat(argument.getValue().getErrorMessage())
+ assertThat(argument.getValue().getStatus())
.contains("Region mapping for region \"" + regionName + "\" not
found");
}
}
--
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].
