[THRIFT-3360] Improve cross test servers and clients further

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.10.0
• Assignee: Nobuaki Sukegawa
• Reporter: Nobuaki Sukegawa
• Created: 27/Sep/15 14:47
• Updated: 18/Mar/16 17:54
• Resolved: 18/Mar/16 17:54

Description

This is follow up fixes to THRIFT-3347
All except C# cross tests in tests.json are mostly fixed now.

Java

• Add missing client calls
• Add missing client assersions
• Semantic return code
• Fix insane server handler

Note that because of newly introduced Java testBinary and assertions, many failures are discovered and listed.
Binary incompatibility is tracked in THRIFT-3359 and I'm working on it.

Python

• Fix testException
• Add testMultiException

Javascript

• Fix testInsanity

C++

• Do not return "basic type error" when connection etc. failed

c_glib

• Fix client testInsanity (I forgot to include this to THRIFT-3347)
• Fix server insane handler

Return code

I want to introduce new semantice return values, so updated test/README.md in the pull request.
The timeout bit is already used by test.py.

+ #define TEST_UNKNOWN 64 // 0100 0000 (Failed to prepare environemt etc.)
+ #define TEST_TIMEOUT 128 // 1000 0000

Issue Links

• is related to: THRIFT-3373 Various fixes for cross test servers and clients
• is related to: THRIFT-3495 Minor enhancements and fixes for cross test