Cleanup

faucetsdn · Dec 9, 2022 · 9887316 · 9887316
1 parent 972a2c9
commit 9887316
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.github/workflows/testing.yml b/.github/workflows/testing.yml
@@ -4,7 +4,7 @@ on:
   push:
   pull_request:
   schedule:
-    - cron: '40 * * * *'
+    - cron: '0 13 * * *'
 
 jobs:
   unit:

diff --git a/validator/src/main/java/com/google/daq/mqtt/sequencer/SequenceBase.java b/validator/src/main/java/com/google/daq/mqtt/sequencer/SequenceBase.java
@@ -113,6 +113,7 @@ public class SequenceBase {
   private static final String SYSTEM_LOG = "system.log";
   private static final String SEQUENCE_MD = "sequence.md";
   private static final long CLEAN_START_DELAY_MS = 20 * 1000;
+  private static final String WAITING_FOR_GODOT = "nothing";
   protected static Metadata deviceMetadata;
   protected static String projectId;
   protected static String cloudRegion;
@@ -403,9 +404,10 @@ public void valid_serial_no() {
   }
 
   private void recordResult(String result, String methodName, String message) {
-    notice(String.format(RESULT_FORMAT, result, methodName, message));
+    String resultString = String.format(RESULT_FORMAT, result, methodName, message);
+    notice(resultString);
     try (PrintWriter log = new PrintWriter(new FileOutputStream(resultSummary, true))) {
-      log.printf(RESULT_FORMAT, result, methodName, message);
+      log.print(resultString);
     } catch (Exception e) {
       throw new RuntimeException("While writing report summary " + resultSummary.getAbsolutePath(),
           e);
@@ -738,7 +740,7 @@ private void messageEvaluateLoop(Supplier<Boolean> evaluator) {
       processMessage();
     }
     info(String.format("finished %s after %s", waitingCondition, timeSinceStart()));
-    waitingCondition = "nothing";
+    waitingCondition = WAITING_FOR_GODOT;
   }
 
   private void recordSequence(String step) {