Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove double test counting #63

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Remove double test counting #63

wants to merge 1 commit into from
+4 −2

Conversation

jrha
Copy link
Member

@jrha jrha commented Sep 11, 2024 •
edited
Loading

Split out from #61.

  • had_no_warnings caused double test in older versions

Probably requires newer versions of Test::More and Test::NoWarnings than are shipped in EL8.

@jrha jrha changed the title Stdweirdt unitfile Remove double test counting Sep 11, 2024
@jrha jrha force-pushed the stdweirdt-unitfile branch from f5f3e24 to 85af950 Compare November 12, 2024 12:51
@stdweird @jrha
had_no_warnings caused double test in older versions
85af950 
One extra test was accounted for in older versions of Test::More+Test::NoWarnings. Replaced without test counting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jrha @stdweird