Ignore Linux debug files, and remove logs

Author: a-maurice

scripts/gha/desktop_tester.py

@@ -65,7 +65,7 @@ def main(argv):
   for file_dir, _, file_names in os.walk(testapp_dir):
     for file_name in file_names:
       # match Testapp names, e.g. "Firebase Analytics Unity Testapp"
-      if testapp_name in file_name.lower():
+      if testapp_name in file_name.lower() and file_name.endswith("_s.debug"):
         testapps.append(os.path.join(file_dir, file_name))
 
   if not testapps:
@@ -123,15 +123,13 @@ def run(self):
     open_process = subprocess.Popen(args=args)
     test_running = True
     time_until_timeout = 300  # Timeout of 300 seconds, or 5 minutes.
-    logging.info("=====  log_path: %s", log_path)
     while test_running and time_until_timeout > 0:
       time.sleep(5)
       time_until_timeout -= 5
       if os.path.exists(log_path):
         with open(log_path) as f:
           self.logs = f.read()
         test_running = "All tests finished" not in self.logs
-    logging.info("=====  Did I timeout? %d", time_until_timeout)
     open_process.kill()
     if platform.system() == 'Linux':
       # Linux seems to have a problem printing out too much information, so truncate it

scripts/gha/integration_testing/automated_testapp/AutomatedTestRunner.cs

@@ -102,8 +102,6 @@ private static string FormatLog(string condition, string stackTrace, LogType typ
       return string.Format("[{0}]({1}): {2}\n{3}\n", DateTime.Now, type, condition, stackTrace);
     }
 
-    float lastLogTime = 0;
-
     // Runs through the tests, checking if the current one is done.
     // If so, logs the relevant information, and proceeds to the next test.
     public void Update() {
@@ -118,14 +116,9 @@ public void Update() {
 
       if (currentTestIndex >= tests.Count) {
         if (Finished) {
-          if (Time.time > lastLogTime + 5f) {
-            UnityEngine.Debug.Log("=====  Test was finished, waiting for some time to pass: " + Time.time + "  " + endTime);
-            lastLogTime = Time.time;
-          }
           // No tests left to run.
           // Wait 5 seconds before notifying test lab manager so video can capture end of test.
           if (Time.time > endTime + 5f) {
-            UnityEngine.Debug.Log("=====  Enough time passed, marking stuff as complete");
             testLabManager.NotifyHarnessTestIsComplete();
           }
         } else {
@@ -143,7 +136,6 @@ public void Update() {
           }
           Finished = true;
           endTime = Time.time;
-          UnityEngine.Debug.Log("=====  Test was finished, starting to wait: " + Time.time + "  " + endTime);
         }
         return;
       }

scripts/gha/integration_testing/automated_testapp/ftl_testapp_files/DesktopTestLabManager.cs

@@ -37,7 +37,6 @@ private DesktopTestLabManager(int scenario, StreamWriter logWriter) {
     }
 
     protected override void OnFinishTest() {
-      UnityEngine.Debug.Log("=====  TestLabManager OnFinishTest called");
       logWriter.Close();
       Application.Quit();
     }
@@ -61,7 +60,6 @@ public static TestLabManager Create() {
       }
       // Hard-coded scenario number. May be replaced with a number supplied via command line
       // flag if multiple scenarios become required for desktop.
-      UnityEngine.Debug.Log("======  Making a TestLabManager with logPath: " + logPath);
       return new DesktopTestLabManager(1, File.AppendText(logPath));
     }