commit e2e5e22c1183a81fc101e4f2e6c13b7ebd41bc8e
author jfarrell <jfarrell@apache.org> Tue Aug 25 14:59:40 2015 -0400
committer jfarrell <jfarrell@apache.org> Tue Aug 25 14:59:40 2015 -0400
tree e7e54a60b965390a019c48455c381811055e6229
parent 400d02c75fe0eba95068217dec3af1f033c4bce3

THRIFT-3308: Fix broken test cases for 0.9.3 release candidate
client: build
Patch: jfarrell

Fix broken test cases, make check now passes with all files in dist

lib/cpp/test/Makefile.am

diff --git a/lib/cpp/test/Makefile.am b/lib/cpp/test/Makefile.am
index 435f2b5..b19f791 100755
--- a/lib/cpp/test/Makefile.am
+++ b/lib/cpp/test/Makefile.am
@@ -209,8 +209,11 @@
 	TPipeInterruptTest.cpp
 
 TPipedTransportTest_LDADD = \
+	libtestgencpp.la \
 	$(top_builddir)/lib/cpp/libthrift.la \
-	$(BOOST_TEST_LDADD)
+	$(BOOST_TEST_LDADD) \
+	$(BOOST_SYSTEM_LDADD) \
+	$(BOOST_THREAD_LDADD)
 
 #
 # AllProtocolsTest

lib/cpp/test/TPipeInterruptTest.cpp

diff --git a/lib/cpp/test/TPipeInterruptTest.cpp b/lib/cpp/test/TPipeInterruptTest.cpp
index b0e246d..e2f2489 100644
--- a/lib/cpp/test/TPipeInterruptTest.cpp
+++ b/lib/cpp/test/TPipeInterruptTest.cpp
@@ -17,6 +17,8 @@
  * under the License.
  */
 
+#ifdef _WIN32
+
 #include <boost/test/test_tools.hpp>
 #include <boost/test/unit_test_suite.hpp>
 
@@ -74,7 +76,7 @@
       for (;;)
       {
         TPipe client("TPipeInterruptTest");
-        client.setConnectTimeout(1);
+        client.setConnTimeout(1);
         client.open();
       }
     } catch (...) { /*just testing for crashes*/ }
@@ -84,3 +86,4 @@
 }
 
 BOOST_AUTO_TEST_SUITE_END()
+#endif

lib/cpp/test/TPipedTransportTest.cpp

diff --git a/lib/cpp/test/TPipedTransportTest.cpp b/lib/cpp/test/TPipedTransportTest.cpp
index a2ec81e..3221fb9 100644
--- a/lib/cpp/test/TPipedTransportTest.cpp
+++ b/lib/cpp/test/TPipedTransportTest.cpp
@@ -28,7 +28,7 @@
 using apache::thrift::transport::TPipedTransport;
 using apache::thrift::transport::TMemoryBuffer;
 
-BOOST_AUTO_TEST_CASE(test_tpipedtransport_1) {
+BOOST_AUTO_TEST_CASE(test_read_write) {
   boost::shared_ptr<TMemoryBuffer> underlying(new TMemoryBuffer);
   boost::shared_ptr<TMemoryBuffer> pipe(new TMemoryBuffer);
   boost::shared_ptr<TPipedTransport> trans(new TPipedTransport(underlying, pipe));