Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1418 I think it's fine as-is, except for adding to ThriftTest in this case since that is a global test thrift file, and you need one local to lib/cpp/test. The fact you took the time to add a unit test and did it in a very TDD way makes me quite happy. :)
---