omaciel/robottelo — pull request #1154 by Ichimonji10 (2014-08-07 17:43:24+00:00)
Title: Create and use decorator `skip_if_bug_open`

This decorator can be used wherever `skip_if_bz_bug_open` or
`skip_if_rm_bug_open` are used. Therefore, also replaces all occurences of them
with `skip_if_bug_open`.

This change is, by itself, simply a refactoring of existing code. It replaces
two enormously complicated functions with a single class that (by virtue of
having an `__init__` and `__call__` method) better manages complexity. This
alone is a good and worthwhile change. However, a second benefit is that the new
decorator can be more easily parameterized with additional arguments. This
should make it possible to, say, adapt this decorator for use within module
`tests.forem.api.test_multiple_paths`.

Also, demote two logging statements from "info" priority to "debug" priority.
It's mildly useful to know when a bug is first being fetched; it's less useful
to know every single time that bug is fetched from a local cache.