Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Commit

Permalink
Merge pull request #494 from ArgusMonitoring/nikhil_code_coverage
Browse files Browse the repository at this point in the history
testing refresh thread tests undo
  • Loading branch information
Nikhil Wahi authored and GitHub Enterprise committed Jun 21, 2019
2 parents c379fab + 0ab76c5 commit 7419d58
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
import static org.junit.Assert.fail;
import static org.junit.Assert.assertEquals;

@Ignore("These new tests are failing in the build pipeline due to persistence issues. @TODO: remove @ignore when pipeline issues are resolved")
//@Ignore("These new tests are failing in the build pipeline due to persistence issues. @TODO: remove @ignore when pipeline issues are resolved")
public class AlertDefinitionsCacheRefresherThreadTest {
private static final String EXPRESSION =
"DIVIDE(-1h:argus.jvm:file.descriptor.open{host=unknown-host}:avg, -1h:argus.jvm:file.descriptor.max{host=unknown-host}:avg)";
Expand Down Expand Up @@ -109,9 +109,8 @@ public void testInitializeCache() {
assertTrue(actualAlert.getNotifications().toArray()[0].equals(notification));
}

@Test
@Ignore
public void testRefreshCache() {
SystemMain system = TestUtils.getInstance();
AlertDefinitionsCache cache = new AlertDefinitionsCache(alertService, false);
AlertDefinitionsCacheRefresherThread refresherThread = new AlertDefinitionsCacheRefresherThread(cache, alertService);

Expand Down

0 comments on commit 7419d58

Please sign in to comment.