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

Feature/583 offline nodes metrics #584

Closed
wants to merge 2 commits into from

Fix broken tests

15e6084
Select commit
Loading
Failed to load commit list.
Closed

Feature/583 offline nodes metrics #584

Fix broken tests
15e6084
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
GitHub Advanced Security / Pmd (reported by Codacy) succeeded Oct 26, 2023 in 4s

2 new alerts

New alerts in code changed by this pull request

  • 2 notes

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 21 in src/test/java/org/jenkinsci/plugins/prometheus/collectors/jenkins/NodesOfflineGaugeTest.java

See this annotation in the file changed.

Code scanning / Pmd (reported by Codacy)

JUnit tests should include assert() or fail() Note test

JUnit tests should include assert() or fail()

Check notice on line 71 in src/test/java/org/jenkinsci/plugins/prometheus/collectors/jenkins/NodesOfflineGaugeTest.java

See this annotation in the file changed.

Code scanning / Pmd (reported by Codacy)

You should not modify visibility of constructors, methods or fields using setAccessible() Note test

You should not modify visibility of constructors, methods or fields using setAccessible()