You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by up...@apache.org on 2019/02/25 23:23:27 UTC
[geode] branch develop updated: GEODE-6174: Expecting 400 instead
of 500 for invalid input
This is an automated email from the ASF dual-hosted git repository.
upthewaterspout pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/develop by this push:
new 0e5d4a6 GEODE-6174: Expecting 400 instead of 500 for invalid input
0e5d4a6 is described below
commit 0e5d4a66d69c4c189da549800c3ac29ab91f1d0e
Author: Dan Smith <up...@apache.org>
AuthorDate: Mon Feb 25 15:20:09 2019 -0800
GEODE-6174: Expecting 400 instead of 500 for invalid input
In RegionManagementDunitTest, we were expecting a http 500. With the
changes in c10fe0700f44ce772ade016419ac582fc8671977, we are now
returning a 400 (BAD REQUEST) instead of 500 (INTERNAL SERVER ERROR) for
the invalid input this test was providing.
---
.../geode/management/internal/rest/RegionManagementDunitTest.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/geode-assembly/src/distributedTest/java/org/apache/geode/management/internal/rest/RegionManagementDunitTest.java b/geode-assembly/src/distributedTest/java/org/apache/geode/management/internal/rest/RegionManagementDunitTest.java
index 9b03fb5..04c8bc0 100644
--- a/geode-assembly/src/distributedTest/java/org/apache/geode/management/internal/rest/RegionManagementDunitTest.java
+++ b/geode-assembly/src/distributedTest/java/org/apache/geode/management/internal/rest/RegionManagementDunitTest.java
@@ -149,7 +149,7 @@ public class RegionManagementDunitTest {
String json = "{\"type\": \"REPLICATE\"}";
ClusterManagementResult result = restClient.doPostAndAssert("/regions", json)
- .hasStatusCode(500)
+ .hasStatusCode(400)
.getClusterManagementResult();
assertThat(result.isRealizedOnAllOrNone()).isTrue();