Skip to content

Comments

add support for retrieving stats from ModuleLoadTest#44

Merged
shahzebsiddiqui merged 4 commits intomasterfrom
moduleloadtest_stats
Mar 15, 2023
Merged

add support for retrieving stats from ModuleLoadTest#44
shahzebsiddiqui merged 4 commits intomasterfrom
moduleloadtest_stats

Conversation

@shahzebsiddiqui
Copy link
Member

No description provided.

…nary with

results such as passed, failed, total and rate.

change logic for include and exclude to work, both can be specified. If after include and exclude
there are no modules we return and raise SystemExit.

If count == 0 we raise exception SystemExit

refactor regression test and increase coverage
@codecov
Copy link

codecov bot commented Mar 15, 2023

Codecov Report

Patch coverage: 94.73% and project coverage change: -2.72 ⚠️

Comparison is base (5225863) 98.30% compared to head (3a1d950) 95.58%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #44      +/-   ##
==========================================
- Coverage   98.30%   95.58%   -2.72%     
==========================================
  Files           4        4              
  Lines         177      204      +27     
  Branches       39       51      +12     
==========================================
+ Hits          174      195      +21     
  Misses          1        1              
- Partials        2        8       +6     
Impacted Files Coverage Δ
lmod/moduleloadtest.py 97.82% <94.73%> (-2.18%) ⬇️

... and 1 file with indirect coverage changes

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 in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@shahzebsiddiqui shahzebsiddiqui merged commit dfa7e18 into master Mar 15, 2023
@shahzebsiddiqui shahzebsiddiqui deleted the moduleloadtest_stats branch March 15, 2023 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant