[system-tests] Update all SkipTest() with corresponding error message
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Fuel for OpenStack |
Fix Committed
|
Medium
|
Unassigned | ||
Mitaka |
Won't Fix
|
Medium
|
Fuel QA Team | ||
Newton |
Fix Committed
|
Medium
|
Unassigned |
Bug Description
Now we are using SkipTest without any error message - so it's difficult to figure out why the test was skipped (previous test in dependency tree failed or the env configured incorrectly). We should add error message for all SkipTest() calls for improving readability of our logs.
For example, in [0] we are skipping the test because the env configured incorrectly;
in [1] we suppress the error message (even if exists it will be removed);
in [2] we check 'should we run the test' and if not - we are skipping it without any message.
[0] https:/
[1] https:/
[2] https:/
Changed in fuel: | |
status: | New → Confirmed |
tags: | added: testrail |
Changed in fuel: | |
assignee: | Maksym Strukov (unbelll) → nobody |
Related fix proposed to branch: master /review. openstack. org/244798
Review: https:/