[ 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)