[IMPALA-2730] skipif decorator logs wrong message

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Not A Bug
• Affects Version/s: Impala 2.3.0
• Component/s: Infrastructure
• Labels: test-infra
• Target Version: Impala 2.5.0
• Assignee: Michael Brown
• Reporter: Vlad Berindei
• Created: 02/Dec/15 00:27
• Updated: 04/Feb/16 00:42
• Resolved: 04/Feb/16 00:42

Description

When multiple SkipIf decorators are used and the test is skipped due to one of them, the message that is logged is the message of the first SkipIf decorator (even if the condition of that decorator is false).

Example:

$ export TARGET_FILESYSTEM="s3"
$ run-tests.py metadata/test_hms_integration.py -k test_hive_db_hive_table_add_partition

This will output "Hive not tested with Isilon" to tests/results/TEST-impala-serial.log.
Changing the order of the decorators of TestHmsIntegration in metadata/test_hms_integration.py changes the log message.