Prevent logging the same message multiple times in runKeyword

Author: psaikkonen

src/main/java/JavaFXLibrary.java

@@ -81,6 +81,7 @@ public Object runKeyword(String keywordName, Object[] args) {
         AtomicReference<RuntimeException> retExcep = new AtomicReference<>();
 
         try {
+            RobotLog.ignoreDuplicates();
             // timeout + 500 ms so that underlying timeout has a chance to expire first
             WaitForAsyncUtils.waitFor(getWaitUntilTimeout(TimeUnit.MILLISECONDS) + 500, TimeUnit.MILLISECONDS, () -> {
 
@@ -100,6 +101,7 @@ public Object runKeyword(String keywordName, Object[] args) {
                 }
             });
         } catch (TimeoutException te) {
+            RobotLog.reset();
             RuntimeException e = retExcep.get();
             runOnFailure.runOnFailure();
 
@@ -113,10 +115,12 @@ public Object runKeyword(String keywordName, Object[] args) {
                 RobotLog.trace("JavaFXLibrary: Caught JavaFXLibrary RUNTIME exception: \n" + Throwables.getStackTraceAsString(e));
                 throw e;
             }
-        } catch (JavaFXLibraryTimeoutException jfxte){
+        } catch (JavaFXLibraryTimeoutException jfxte) {
+            RobotLog.reset();
             RobotLog.trace("JavaFXLibrary: Caught JavaFXLibrary TIMEOUT exception: \n" + Throwables.getStackTraceAsString(jfxte));
             throw jfxte;
         }
+        RobotLog.reset();
         return retval.get();
     }
 

src/main/java/javafxlibrary/utils/RobotLog.java

@@ -1,24 +1,57 @@
 package javafxlibrary.utils;
 
+import java.util.LinkedList;
+
 public class RobotLog {
 
+    private static boolean ignoreDuplicates = false;
+    private static LinkedList<String> loggedMessages = new LinkedList<>();
+
+
+    public static void ignoreDuplicates() {
+        ignoreDuplicates = true;
+    }
+
+    public static void reset() {
+        ignoreDuplicates = false;
+        loggedMessages.clear();
+    }
+
     public static void info(String message) {
-        System.out.println("*INFO* " + message);
+        if (shouldLogMessage(message))
+            System.out.println("*INFO* " + message);
     }
 
     public static void debug(String message) {
-        System.out.println("*DEBUG* " + message);
+        if (shouldLogMessage(message))
+            System.out.println("*DEBUG* " + message);
     }
 
     public static void trace(String message) {
-        System.out.println("*TRACE* " + message);
+        if (shouldLogMessage(message))
+            System.out.println("*TRACE* " + message);
     }
 
     public static void warn(String message) {
-        System.out.println("*WARN* " + message);
+        if (shouldLogMessage(message))
+            System.out.println("*WARN* " + message);
     }
 
     public static void error(String message) {
-        System.out.println("*ERROR* " + message);
+        if (shouldLogMessage(message))
+            System.out.println("*ERROR* " + message);
+    }
+
+    private static boolean shouldLogMessage(String message) {
+        if (ignoreDuplicates) {
+            if (loggedMessages.contains(message)) {
+                return false;
+            } else {
+                loggedMessages.add(message);
+                return true;
+            }
+        } else {
+            return true;
+        }
     }
 }