-
Notifications
You must be signed in to change notification settings - Fork 103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[testlib] Add fs mount options check to hpctstlib #2958
Conversation
The filesystem_option_check check if the file systems were properly configured based on their type
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2958 +/- ##
========================================
Coverage 86.70% 86.70%
========================================
Files 61 61
Lines 11996 11996
========================================
Hits 10401 10401
Misses 1595 1595 ☔ View full report in Codecov by Sentry. |
This newer version of test is not parameterised. This implies that one should see a single test that processes all filesystems. This version also adds to the output the reference options used by the test.
Hello @victorusu, 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 2023-10-23 21:45:43 UTC |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor style suggestions.
Co-authored-by: Vasileios Karakasis <vkarak@gmail.com>
Co-authored-by: Vasileios Karakasis <vkarak@gmail.com>
Co-authored-by: Vasileios Karakasis <vkarak@gmail.com>
Co-authored-by: Vasileios Karakasis <vkarak@gmail.com>
The filesystem_option_check check if the file systems were properly configured based on their type