THRIFT-356. Improve CompactProtocolTestStruct
authorDavid Reiss <dreiss@apache.org>
Tue, 7 Apr 2009 20:51:48 +0000 (20:51 +0000)
committerDavid Reiss <dreiss@apache.org>
Tue, 7 Apr 2009 20:51:48 +0000 (20:51 +0000)
commitfe5b77d728feec094a086e516d3a185796397072
tree188a44d82331ca19f9f65dfc3481ab86c65aa6ef
parent2bcf3995ef7ff3db554c550b2d760d870a64d389
THRIFT-356. Improve CompactProtocolTestStruct

- Remove the default values from CompactProtocolTestStruct
- Add a const CompactProtocolTestStruct that contains those values

This has a short-term benefit and a long-term benefit.

The short-term benefit is that it removes the list-as-map-key
(etc.) values from the "types" generated file and puts it into
the "constants" generated file.  This gets them out of the way
of the Python tests, so they pass again.

The long term benefit is that is strengthens the Java TCompactProtocol
test.  Before, it was reading the "full" CompactProtocolTestStruct into
a CompactProtocolTestStruct that was just constructed, but the latter
was already almost equal to the former.  This change makes it much
easier to create an empty CompactProtocolTestStruct to read the serialized
data into.

git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@762957 13f79535-47bb-0310-9956-ffa450edef68
lib/java/test/org/apache/thrift/test/Fixtures.java
test/DebugProtoTest.thrift