[feat] Implement test dependency API #769

vkarak · 2019-04-18T23:03:04Z

Extend RegressionTest API to allow defining and retrieving dependencies between tests
Build dependency graph of test cases
Corresponding unit tests

- Extend RegressionTest API to allow defining and retrieving dependencies between tests - Build dependency graph of test cases - Corresponding unit tests

pep8speaks · 2019-04-18T23:03:09Z

Hello @vkarak, Thank you for updating!

Cheers! There are no PEP8 issues in this Pull Request!Do see the ReFrame Coding Style Guide

Comment last updated at 2019-05-22 11:43:28 UTC

reframe/frontend/dependency.py

reframe/core/pipeline.py

reframe/frontend/executors/__init__.py

unittests/test_dependencies.py

unittests/test_policies.py

- Fix documentation of build_deps() - Remove stale test file - Extend unit tests to a couple of corner cases

vkarak · 2019-05-05T17:18:34Z

@jenkins-cscs retry dom

reframe/core/pipeline.py

unittests/resources/settings.py

reframe/frontend/executors/__init__.py

reframe/frontend/dependency.py

codecov-io · 2019-05-19T20:54:46Z

Codecov Report

Merging #769 into master will increase coverage by 0.12%.
The diff coverage is 95.65%.

@@            Coverage Diff             @@
##           master     #769      +/-   ##
==========================================
+ Coverage   91.51%   91.64%   +0.12%     
==========================================
  Files          76       78       +2     
  Lines        9551     9775     +224     
==========================================
+ Hits         8741     8958     +217     
- Misses        810      817       +7

Impacted Files	Coverage Δ
unittests/resources/settings.py	`100% <ø> (ø)`	⬆️
reframe/frontend/executors/__init__.py	`98.23% <100%> (+1.04%)`	⬆️
reframe/core/exceptions.py	`82.17% <100%> (+0.13%)`	⬆️
unittests/test_config.py	`97.53% <100%> (ø)`	⬆️
...s/resources/checks_unlisted/dependencies/normal.py	`100% <100%> (ø)`
reframe/core/pipeline.py	`93.28% <86.95%> (-0.17%)`	⬇️
reframe/frontend/dependency.py	`93.47% <93.47%> (ø)`
unittests/test_policies.py	`98.33% <96.63%> (-0.84%)`	⬇️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 61698aa...37951d7. Read the comment docs.

vkarak · 2019-05-22T11:36:16Z

@jenkins-cscs retry none

…t/testdep-api

Implement test dependency API

7f6e5d3

- Extend RegressionTest API to allow defining and retrieving dependencies between tests - Build dependency graph of test cases - Corresponding unit tests

vkarak added prio: normal enhancement labels Apr 18, 2019

vkarak added this to the ReFrame sprint 2019w15 milestone Apr 18, 2019

vkarak requested review from teojgo and victorusu April 18, 2019 23:03

vkarak self-assigned this Apr 18, 2019