[ 
https://issues.apache.org/jira/browse/THRIFT-2668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14101085#comment-14101085
 ] 

ASF GitHub Bot commented on THRIFT-2668:
----------------------------------------

Github user bufferoverflow commented on a diff in the pull request:

    https://github.com/apache/thrift/pull/194#discussion_r16374767
  
    --- Diff: test/cpp/src/TestClient.cpp ---
    @@ -698,5 +756,28 @@ int main(int argc, char** argv) {
       printf("Max time: %" PRIu64 " us\n", time_max);
       printf("Avg time: %" PRIu64 " us\n", time_avg);
     
    +  int ret = 255 - TEST_NOTUSED;
    +  if(!test_basetypes_fails){
    +    ret = ret - TEST_BASETYPES;
    +  }
    +  if(!test_structs_fails){
    +    ret = ret - TEST_STRUCTS;
    +  }
    +  if(!test_containers_fails){
    +    ret = ret - TEST_CONTAINERS;
    +  }
    +  if(!test_exceptions_fails){
    +    ret = ret - TEST_EXCEPTIONS;
    +  }
    +  if(!test_complex_fails){
    +    ret = ret - TEST_COMPLEX;
    +  }
    +  if(!test_enumtypedef_fails){
    +    ret = ret - TEST_ENUMTYPEDEF;
    +  }
    +  if(!test_oneway_fails){
    +    ret = ret - TEST_ONEWAY;
    +  }
    +
       return failCount;
    --- End diff --
    
        return ret;


> TestSuite: detailed result on passed tests by feature
> -----------------------------------------------------
>
>                 Key: THRIFT-2668
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2668
>             Project: Thrift
>          Issue Type: Sub-task
>          Components: Test Suite
>            Reporter: Roger Meier
>
> return code shall be 0 on success or an integer in the range 1 - 255 on error
> as mentioned http://thrift.apache.org/test
> We can define bit fields for the byte we pass as return code:
> {noformat}
> #define TEST_BASETYPES     1  // 0000 0001
> #define TEST_STRUCTS       2  // 0000 0010
> #define TEST_CONTAINERS    4  // 0000 0100
> #define TEST_EXCEPTIONS    8  // 0000 1000
> #define TEST_NOTUSED     240  // 1111 0000 
> {noformat}
> this will help us to see which tests passed 
> a TestClient will do this:
> {noformat}
> ret = 255 - TEST_NOTUSED
> if testBaseTypes == passed
>   ret = ret - TEST_BASETYPES
> if testStructs == passed
>   ret = ret - TEST_STRUCTS
> if testContainers == passed
>   ret = ret - TEST_CONTAINERS
> if testExceptions == passed
>   ret = ret - TEST_EXCEPTIONS
> return ret
> {noformat}
> test suite reporting can take care on the individual test status



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to