Skip to content

Conversation

@vkarak
Copy link
Contributor

@vkarak vkarak commented Feb 8, 2023

When listing fixtures, we also print the variable name to which they are bound, so that users are able to select them in the -S option without having to inspect the test file. I have also enhanced the docs. This is how the new listing looks like:

- osu_latency_test /d2c978ad
    ^build_osu_benchmarks ~generic:default+builtin 'osu_binaries /5cf701b0
      ^fetch_osu_benchmarks ~generic 'osu_benchmarks /9fc7952e

Closes #2763.

@codecov-commenter
Copy link

codecov-commenter commented Feb 8, 2023

Codecov Report

Base: 86.61% // Head: 86.57% // Decreases project coverage by -0.05% ⚠️

Coverage data is based on head (23554ad) compared to base (d74dd1f).
Patch coverage: 30.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2783      +/-   ##
==========================================
- Coverage   86.61%   86.57%   -0.05%     
==========================================
  Files          60       60              
  Lines       11218    11226       +8     
==========================================
+ Hits         9717     9719       +2     
- Misses       1501     1507       +6     
Impacted Files Coverage Δ
reframe/frontend/cli.py 73.22% <30.00%> (-0.62%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@vkarak vkarak merged commit 32fb479 into reframe-hpc:master Feb 14, 2023
@vkarak vkarak deleted the enhancement/list-fixture-var-names branch February 14, 2023 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants