Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Register test groups, mitigate warning #424

Merged
merged 3 commits into from
Nov 29, 2023

Conversation

ankona
Copy link
Contributor

@ankona ankona commented Nov 28, 2023

The pytest groups used to split execution of the test cases for parallelization currently result in a warning message displayed during test runs.

=== warnings summary ===
tests/test_cli.py:49
  /lus/cls01029/mcbridch/ss/tests/test_cli.py:49: PytestUnknownMarkWarning: Unknown pytest.mark.group_a - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html
    pytestmark = pytest.mark.group_a

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=== 52 passed, 1 warning in 0.53s ===

This fix adds group registrations to pyproject.toml to avoid those warnings:

=== 52 passed in 0.48s ===

Copy link

codecov bot commented Nov 28, 2023

Codecov Report

Merging #424 (c9407fb) into develop (15ba145) will decrease coverage by 0.06%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #424      +/-   ##
===========================================
- Coverage    89.65%   89.60%   -0.06%     
===========================================
  Files           59       59              
  Lines         3617     3617              
===========================================
- Hits          3243     3241       -2     
- Misses         374      376       +2     

see 1 file with indirect coverage changes

Copy link
Contributor

@billschereriii billschereriii left a comment

Choose a reason for hiding this comment

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

Thank you for catching and fixing this oversight!

@ankona ankona merged commit 508cba3 into CrayLabs:develop Nov 29, 2023
@ankona ankona deleted the marker-warnings branch November 29, 2023 16:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants