[THRIFT-4016] testInsanity() impl does not conform to test spec in ThriftTest.thrift

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.11.0
• Component/s: C# - Library, Delphi - Library, Haxe - Library
• Patch Info: Patch Available
• Assignee: Jens Geyer
• Reporter: Jens Geyer
• Created: 28/Dec/16 14:07
• Updated: 14/Jan/17 09:04
• Resolved: 10/Jan/17 21:53

Description

The Delphi impl of the testInsanity() method does not conform to test spec in ThriftTest.thrift:

  /**
   * So you think you've got this all worked, out eh?
   *
   * Creates a the returned map with these values and prints it out:
   *   { 1 => { 2 => argument,
   *            3 => argument,
   *          },
   *     2 => { 6 => <empty Insanity struct>, },
   *   }
   * @return map<UserId, map<Numberz,Insanity>> - a map with the above values
   */

The server delivered more data and the client tested that. Running the client against (e.g.) a C++ server will incorrectly render an error about the values returned by that server being incorrect. However, the only thing wrong here is the Delphi impl of that test.