Skip to content

Conversation

@rsarm
Copy link
Contributor

@rsarm rsarm commented Sep 5, 2018

Closes #402

@rsarm rsarm added this to the ReFrame sprint 2018w35 milestone Sep 5, 2018
@rsarm rsarm self-assigned this Sep 5, 2018
@rsarm rsarm requested a review from vkarak September 5, 2018 06:45
@vkarak vkarak changed the title Set skipping nonexistent file message as warning [bugfix] Convert message for inexistent test files to warning Sep 5, 2018
@vkarak vkarak added bugfix and removed bug labels Sep 5, 2018
if not os.path.exists(d):
printer.info("%s: path `%s' does not exist. Skipping...\n" %
printer.warning("%s: path `%s' does not exist. Skipping..." %
(argparser.prog, d))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please align correctly this line, too.

@vkarak
Copy link
Contributor

vkarak commented Sep 5, 2018

@jenkins-cscs retry daint

@codecov-io
Copy link

Codecov Report

Merging #440 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #440   +/-   ##
=======================================
  Coverage   91.23%   91.23%           
=======================================
  Files          70       70           
  Lines        8611     8611           
=======================================
  Hits         7856     7856           
  Misses        755      755
Impacted Files Coverage Δ
reframe/frontend/cli.py 78.03% <100%> (ø) ⬆️
reframe/core/config.py 82.72% <0%> (-1.82%) ⬇️
reframe/frontend/printer.py 89.33% <0%> (+2.66%) ⬆️

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 404f4a7...972ab84. Read the comment docs.

@vkarak vkarak merged commit 0e26e8c into reframe-hpc:master Sep 5, 2018
@rsarm rsarm deleted the frontend/inexistent-test-file branch November 26, 2019 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants