You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by jb...@apache.org on 2017/08/21 23:19:16 UTC

[geode-native] branch develop updated: GEODE-3497: Fix failing test on Solaris SPARC

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

jbarrett 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 cc6dafe  GEODE-3497: Fix failing test on Solaris SPARC
cc6dafe is described below

commit cc6dafee2cd82fabbbb770686fa9b9d87ba4eeb3
Author: David Kimura <dk...@pivotal.io>
AuthorDate: Mon Aug 21 13:05:49 2017 -0700

    GEODE-3497: Fix failing test on Solaris SPARC
---
 cppcache/test/ClientProxyMembershipIDFactoryTest.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/cppcache/test/ClientProxyMembershipIDFactoryTest.cpp b/cppcache/test/ClientProxyMembershipIDFactoryTest.cpp
index b9fec88..d593812 100644
--- a/cppcache/test/ClientProxyMembershipIDFactoryTest.cpp
+++ b/cppcache/test/ClientProxyMembershipIDFactoryTest.cpp
@@ -26,17 +26,18 @@ using namespace apache::geode::client;
 TEST(ClientProxyMembershipIDFactoryTest, testCreate) {
   ClientProxyMembershipIDFactory factory("myDs");
 
-  auto id = factory.create("myHost", 1, 2, "myClientID", 3);
+  auto hostAddr = htonl(1);
+  auto id = factory.create("myHost", hostAddr, 2, "myClientID", 3);
   ASSERT_NE(nullptr, id);
 
   EXPECT_EQ("myDs", id->getDSName());
-  EXPECT_EQ(1, static_cast<uint32_t>(*id->getHostAddr()));
+  EXPECT_EQ(hostAddr, *reinterpret_cast<uint32_t*>(id->getHostAddr()));
   EXPECT_EQ(4, id->getHostAddrLen());
   EXPECT_EQ(2, id->getHostPort());
 
   auto uniqueTag = id->getUniqueTag();
   ASSERT_NE("", uniqueTag);
-  EXPECT_EQ(std::string(":1:0:0:0:2:myDs:").append(uniqueTag),
+  EXPECT_EQ(std::string(":0:0:0:1:2:myDs:").append(uniqueTag),
             id->getHashKey());
   EXPECT_TRUE(std::regex_search(
       id->getDSMemberIdForThinClientUse(),

-- 
To stop receiving notification emails like this one, please contact
['"dev@geode.apache.org" <de...@geode.apache.org>'].