Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-4016

testInsanity() impl does not conform to test spec in ThriftTest.thrift

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Patch Info:
      Patch Available

      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.

        Attachments

          Activity

            People

            • Assignee:
              jensg Jens Geyer
              Reporter:
              jensg Jens Geyer

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment