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.

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
Closed

Feature/583 offline nodes metrics #584

wants to merge 2 commits into from

Conversation

aleksei-scanbot
Copy link

Fixes #583

Changes proposed

  • ...
  • ...
  • ...

Checklist

  • Includes tests covering the new functionality?
  • Ready for review
  • Follows CONTRIBUTING rules

Notify

@Waschndolos

public class NodesOfflineGaugeTest extends MockedJenkinsTest {

@Test
public void testCollectResult() throws Exception {

Check notice

Code scanning / Pmd (reported by Codacy)

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

JUnit tests should include assert() or fail()
}

static void setFinalStatic(Field field, Object newValue) throws Exception {
field.setAccessible(true);

Check notice

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()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add default_jenkins_nodes_offline
1 participant