Disable broken test

Summary: See comment

Reviewed By: phlalx

Differential Revision: D47334480

fbshipit-source-id: 3f30b884c9a4e3f1bffbcfcdb1e577bb011c0816

cpp-channel/thrift-cpp-channel.cabal

@@ -132,6 +132,11 @@ flag tests_use_ipv4
      default: False
      manual: True
 
+flag include_broken_tests
+     description: Include broken tests
+     default: False
+     manual: True
+
 test-suite header-channel
   import: fb-haskell, fb-cpp
   type: exitcode-stdio-1.0
@@ -144,8 +149,8 @@ test-suite header-channel
                test/if/gen-cpp2/Calculator.cpp
                test/if/gen-cpp2/AdderAsyncClient.cpp
                test/if/gen-cpp2/Adder.cpp
-               test/if/gen-cpp2/math_data.cpp
                test/if/gen-cpp2/math_types.cpp
+               test/if/gen-cpp2/math_data.cpp
                test/if/gen-cpp2/math_metadata.cpp
   extra-libraries:
         thriftmetadata
@@ -177,3 +182,14 @@ test-suite header-channel
                   unordered-containers,
                   deepseq,
                   STMonadTrans
+
+  -- This test is broken when built from github. For some reason the
+  -- requests timeout. It seems to be a subtle timing issue of some
+  -- kind inside fbthrift, because I tried tracing through the code
+  -- using gdb and when using breakpoints the test will sometimes
+  -- pass. Without a deeper understanding of the internals of fbthrift
+  -- I can't get any further, so just ignoring the test for now. It
+  -- passes on the internal CI so I'm pretty sure it's not an issue
+  -- with the test itself.
+  if !flag(include_broken_tests)
+     buildable: False