Formatting and style fixes
Uncomment ThriftTest functionality; use SKIP-FLAKY to control execution

Problem

ThriftTest has testing functionality that was marked flaky but currently
passes.  I am unsure if these are desired tests but they pass and may be
useful.

Solution

Given that there is a convention for including flaky tests by default
- and disabling via ENV variable - I reenabled test paths that pass.  For
paths that do not pass I created a skipThriftTest method.

Result

Potentially higher test coverage in the package.