refactor: disableReconnect added to TestClient.java

Changed modifier of disableReconnect in TestClientImpl.java to public
Draegerwerk · Jan 20, 2025 · 8f70435 · 8f70435
1 parent 1c6c16f
commit 8f70435
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 17 deletions.
diff --git a/sdccc/src/main/java/com/draeger/medical/sdccc/sdcri/testclient/TestClient.java b/sdccc/src/main/java/com/draeger/medical/sdccc/sdcri/testclient/TestClient.java
@@ -59,6 +59,11 @@ public interface TestClient {
      */
     void enableReconnect();
 
+    /**
+     * Disable reconnection attempts on connection loss.
+     */
+    void disableReconnect();
+
     /**
      * Disconnects the SDC client from the target.
      *

diff --git a/sdccc/src/main/java/com/draeger/medical/sdccc/sdcri/testclient/TestClientImpl.java b/sdccc/src/main/java/com/draeger/medical/sdccc/sdcri/testclient/TestClientImpl.java
@@ -126,19 +126,19 @@ public class TestClientImpl extends AbstractIdleService implements TestClient, W
     /**
      * Creates an SDCri consumer instance.
      *
-     * @param targetDeviceEpr EPR address to filter for
-     * @param targetDeviceFacility facility to filter for
-     * @param targetDeviceBuilding building to filter for
+     * @param targetDeviceEpr         EPR address to filter for
+     * @param targetDeviceFacility    facility to filter for
+     * @param targetDeviceBuilding    building to filter for
      * @param targetDevicePointOfCare point of care to filter for
-     * @param targetDeviceFloor floor to filter for
-     * @param targetDeviceRoom room to filter for
-     * @param targetDeviceBed bed to filter for
-     * @param adapterAddress  ip of the network interface to bind to
-     * @param maxWait         max waiting time to find and connect to target device
-     * @param reconnectTries               number of tries a reconnection is attempted
-     * @param reconnectWait                the wait time between reconnection attempts in seconds
-     * @param testClientUtil  test client utility
-     * @param testRunObserver observer for invalidating test runs on unexpected errors
+     * @param targetDeviceFloor       floor to filter for
+     * @param targetDeviceRoom        room to filter for
+     * @param targetDeviceBed         bed to filter for
+     * @param adapterAddress          ip of the network interface to bind to
+     * @param maxWait                 max waiting time to find and connect to target device
+     * @param reconnectTries          number of tries a reconnection is attempted
+     * @param reconnectWait           the wait time between reconnection attempts in seconds
+     * @param testClientUtil          test client utility
+     * @param testRunObserver         observer for invalidating test runs on unexpected errors
      */
     @Inject
     public TestClientImpl(
@@ -408,6 +408,12 @@ public void enableReconnect() {
         this.reconnectEnabled.set(true);
     }
 
+    @Override
+    public void disableReconnect() {
+        LOG.info(TriggerOnErrorOrWorseLogAppender.RESET_WHITELIST_MARKER, "Disable reconnect feature.");
+        this.reconnectEnabled.set(false);
+    }
+
     @Override
     public synchronized void disconnect() throws TimeoutException {
         disconnect(true);
@@ -562,11 +568,6 @@ private List<String> buildThreadNameWhiteList() {
                 convertToRegex(WATCHDOG_SCHEDULED_EXECUTOR_NAME_FORMAT));
     }
 
-    private void disableReconnect() {
-        LOG.info(TriggerOnErrorOrWorseLogAppender.RESET_WHITELIST_MARKER, "Disable reconnect feature.");
-        this.reconnectEnabled.set(false);
-    }
-
     private String convertToRegex(final String pattern) {
         return pattern.replaceAll("%d", "[\\\\d]+");
     }