You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by bb...@apache.org on 2019/08/02 22:14:30 UTC

[geode-native] branch develop updated: GEODE-7040: Fix negative test case for values which can't be deserialized on server (#506)

This is an automated email from the ASF dual-hosted git repository.

bbender pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode-native.git


The following commit(s) were added to refs/heads/develop by this push:
     new 01bd930  GEODE-7040: Fix negative test case for values which can't be deserialized on server (#506)
01bd930 is described below

commit 01bd9307d3eaad20be11265807a5e945e6de6b96
Author: Blake Bender <bb...@pivotal.io>
AuthorDate: Fri Aug 2 15:14:25 2019 -0700

    GEODE-7040: Fix negative test case for values which can't be deserialized on server (#506)
    
    - test wasn't connecting to locator properly, so it was getting the wrong exception
    
    Co-authored-by: Mike Martell <mm...@pivotal.io>
---
 cppcache/integration/test/FunctionExecutionTest.cpp | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/cppcache/integration/test/FunctionExecutionTest.cpp b/cppcache/integration/test/FunctionExecutionTest.cpp
index d731c90..1b90e41 100644
--- a/cppcache/integration/test/FunctionExecutionTest.cpp
+++ b/cppcache/integration/test/FunctionExecutionTest.cpp
@@ -134,7 +134,7 @@ TEST(FunctionExecutionTest, UnknownFunctionAsyncOnRegion) {
 }
 
 TEST(FunctionExecutionTest,
-     Disabled_FunctionReturnsObjectWhichCantBeDeserializedOnServer) {
+     FunctionReturnsObjectWhichCantBeDeserializedOnServer) {
   Cluster cluster{LocatorCount{1}, ServerCount{1}};
   cluster.getGfsh()
       .create()
@@ -142,23 +142,21 @@ TEST(FunctionExecutionTest,
       .withName("region")
       .withType("REPLICATE")
       .execute();
-
   cluster.getGfsh()
       .deploy()
       .jar(getFrameworkString(FrameworkVariable::JavaObjectJarPath))
       .execute();
-
   auto cache = cluster.createCache();
   auto region = cache.createRegionFactory(RegionShortcut::PROXY)
                     .setPoolName("default")
                     .create("region");
-
   const char *GetScopeSnapshotsFunction =
       "executeFunction_SendObjectWhichCantBeDeserialized";
+
   auto functionService = FunctionService::onRegion(region);
-  ASSERT_THROW(functionService.execute(GetScopeSnapshotsFunction),
-               apache::geode::client::IllegalStateException);
 
+  ASSERT_THROW(functionService.execute(GetScopeSnapshotsFunction),
+               apache::geode::client::MessageException);
   cache.close();
 }