You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Jens Geyer (JIRA)" <ji...@apache.org> on 2016/12/28 14:16:58 UTC

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

     [ https://issues.apache.org/jira/browse/THRIFT-4016?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jens Geyer resolved THRIFT-4016.
--------------------------------
       Resolution: Fixed
    Fix Version/s: 0.11.0

> testInsanity() impl does not conform to test spec in ThriftTest.thrift
> ----------------------------------------------------------------------
>
>                 Key: THRIFT-4016
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4016
>             Project: Thrift
>          Issue Type: Bug
>          Components: Delphi - Library
>            Reporter: Jens Geyer
>            Assignee: Jens Geyer
>             Fix For: 0.11.0
>
>         Attachments: 0001-THRIFT-4016-testInsanity-impl-does-not-conform-to-te.patch
>
>
> The Delphi impl of the testInsanity() method does not conform to test spec in ThriftTest.thrift:
> {code}
>   /**
>    * 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
>    */
> {code}
> 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.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)