[THRIFT-3609] Remove or replace TestPortFixture.h

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.10.0
• Component/s: C++ - Library, Test Suite
• Assignee: John Sirois
• Reporter: John Sirois
• Created: 10/Feb/16 17:29
• Updated: 19/Feb/16 02:17
• Resolved: 17/Feb/16 15:01

Description

Today 6 cpp lib tests use TestPortFixture.h. THRIFT-3608 experimented with correcting one of these cases in lib/cpp/test/SecurityTest.cpp, but a general cleanup should remove all uses of static ports in the cpp tests to improve robustness and open the path to parallel testing - at least in the cpp libs.

Issue Links

• relates to: THRIFT-3608 lib/cpp/test/SecurityTest is flaky in jenkins Thrift-precommit build.

Sub-Tasks

• Fix lib/cpp/test/SecurityTest.cpp to use ephemeral ports: Closed
• Fix Fix cpp/lib/test/TSSLSocketInterruptTest.cpp to use ephemeral ports: Closed
• Fix lib/cpp/test/TServerSocketTest.cpp to use ephemeral ports: Closed
• Kill unused #include "TestPortFixture.h" in lib/cpp/test/TServerTransportTest.cpp.: Closed
• Fix lib/cpp/test/TSocketInterruptTest.cpp to use ephemeral ports.: Closed
• Fix lib/cpp/test/TServerIntegrationTest.cpp to use ephemeral ports: Closed