Issue #203: Refactor-Extensions-code-to-check-protocol-efficiently

Modify OsCommandExtensionTest and WbemExtensionTest
sentrysoftware · Jun 4, 2024 · 0681744 · 0681744
1 parent 65a46de
commit 0681744
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/...c/test/java/org/sentrysoftware/metricshub/extension/oscommand/OsCommandExtensionTest.java b/...c/test/java/org/sentrysoftware/metricshub/extension/oscommand/OsCommandExtensionTest.java
@@ -480,7 +480,7 @@ void testCheckSshNoHealthWhenMustCheckFalse() {
 		Optional<Boolean> result = osCommandExtension.checkProtocol(telemetryManager);
 
 		// Assert the result
-		assertEquals(Optional.empty(),result);
+		assertEquals(Optional.empty(), result);
 	}
 
 	@Test
@@ -497,7 +497,7 @@ void testCheckSshNoHealthWhenNoConfiguration() {
 		Optional<Boolean> result = osCommandExtension.checkProtocol(telemetryManager);
 
 		// Assert the result
-		assertEquals(Optional.empty(),result);
+		assertEquals(Optional.empty(), result);
 	}
 
 	@Test

diff --git a/...tension/src/test/java/org/sentrysoftware/metricshub/extension/wbem/WbemExtensionTest.java b/...tension/src/test/java/org/sentrysoftware/metricshub/extension/wbem/WbemExtensionTest.java
@@ -144,9 +144,9 @@ void testCheckWbemDownHealth() throws ClientException {
 			.doWbemQuery(anyString(), any(WbemConfiguration.class), anyString(), anyString());
 
 		// Start the WBEM Health Check strategy
-		boolean result = wbemExtension.checkProtocol(telemetryManager);
+		Optional<Boolean> result = wbemExtension.checkProtocol(telemetryManager);
 
-		assertFalse(result);
+		assertFalse(result.get());
 	}
 
 	@Test