summaryrefslogtreecommitdiffstats
path: root/vespajlib/src/test/java/com/yahoo/concurrent/maintenance/JobControlTest.java
diff options
context:
space:
mode:
Diffstat (limited to 'vespajlib/src/test/java/com/yahoo/concurrent/maintenance/JobControlTest.java')
-rw-r--r--vespajlib/src/test/java/com/yahoo/concurrent/maintenance/JobControlTest.java16
1 files changed, 4 insertions, 12 deletions
diff --git a/vespajlib/src/test/java/com/yahoo/concurrent/maintenance/JobControlTest.java b/vespajlib/src/test/java/com/yahoo/concurrent/maintenance/JobControlTest.java
index 01560c050ff..139a2901cd3 100644
--- a/vespajlib/src/test/java/com/yahoo/concurrent/maintenance/JobControlTest.java
+++ b/vespajlib/src/test/java/com/yahoo/concurrent/maintenance/JobControlTest.java
@@ -3,8 +3,6 @@ package com.yahoo.concurrent.maintenance;
import org.junit.Test;
-import java.util.concurrent.atomic.AtomicLong;
-
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
@@ -21,8 +19,9 @@ public class JobControlTest {
String job1 = "Job1";
String job2 = "Job2";
- TestMaintainer maintainer1 = new TestMaintainer(job1, jobControl, new NoopJobMetrics());
- TestMaintainer maintainer2 = new TestMaintainer(job2, jobControl, new NoopJobMetrics());
+ JobMetrics metrics = new JobMetrics((job, instant) -> {});
+ TestMaintainer maintainer1 = new TestMaintainer(job1, jobControl, metrics);
+ TestMaintainer maintainer2 = new TestMaintainer(job2, jobControl, metrics);
assertEquals(2, jobControl.jobs().size());
assertTrue(jobControl.jobs().contains(job1));
assertTrue(jobControl.jobs().contains(job2));
@@ -63,7 +62,7 @@ public class JobControlTest {
public void testJobControlMayDeactivateJobs() {
JobControlStateMock state = new JobControlStateMock();
JobControl jobControl = new JobControl(state);
- TestMaintainer mockMaintainer = new TestMaintainer(null, jobControl, new NoopJobMetrics());
+ TestMaintainer mockMaintainer = new TestMaintainer(null, jobControl, new JobMetrics((job, instant) -> {}));
assertTrue(jobControl.jobs().contains("TestMaintainer"));
@@ -81,11 +80,4 @@ public class JobControlTest {
assertEquals(2, mockMaintainer.totalRuns());
}
- private static class NoopJobMetrics extends JobMetrics {
-
- @Override
- protected void recordCompletion(String job, Long incompleteRuns, double successFactor) { }
-
- }
-
}