diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-07-11 11:54:20 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-07-16 21:26:15 +0100 |
commit | b618261811c48ff3b98eab1b340a8cd09ef183c6 (patch) | |
tree | 24f3fe70856d432f9c01fa3b51ef0590c67ed808 /meta/lib/oeqa/core/decorator/depends.py | |
parent | a98ab5e560e73b6988512fbae5cefe9e42ceed53 (diff) | |
download | openembedded-core-contrib-b618261811c48ff3b98eab1b340a8cd09ef183c6.tar.gz |
oeqa/runner: Simplify code
There doesn't appear to be any reason we need this _results indirection
any more so remove it.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oeqa/core/decorator/depends.py')
-rw-r--r-- | meta/lib/oeqa/core/decorator/depends.py | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/meta/lib/oeqa/core/decorator/depends.py b/meta/lib/oeqa/core/decorator/depends.py index 99eccc1268..69c604d8f4 100644 --- a/meta/lib/oeqa/core/decorator/depends.py +++ b/meta/lib/oeqa/core/decorator/depends.py @@ -63,12 +63,10 @@ def _order_test_case_by_depends(cases, depends): return [cases[case_id] for case_id in cases_ordered] def _skipTestDependency(case, depends): - results = case.tc._results - skipReasons = ['errors', 'failures', 'skipped'] for reason in skipReasons: - for test, _ in results[reason]: + for test, _ in getattr(case.tc.results, reason): if test.id() in depends: raise SkipTest("Test case %s depends on %s and was in %s." \ % (case.id(), test.id(), reason)) |