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

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

    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
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: