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();