From: James Procter <j.procter@dundee.ac.uk>
Date: Tue, 12 Dec 2023 17:20:04 +0000 (+0000)
Subject: JAL-4353 wait for files to be written fully before comparing size with older file
X-Git-Url: http://source.jalview.org/gitweb/?p=jalview.git;a=commitdiff_plain;h=360ef45fd26bf78323fff8f0a3f7a8632cd9db8e

JAL-4353 wait for files to be written fully before comparing size with older file
---

diff --git a/test/jalview/bin/CommandsTest.java b/test/jalview/bin/CommandsTest.java
index 2d54093..7b42737 100644
--- a/test/jalview/bin/CommandsTest.java
+++ b/test/jalview/bin/CommandsTest.java
@@ -191,13 +191,17 @@ public class CommandsTest
         Assert.assertTrue(Files.size(file.toPath()) > 0, "File '" + filename
                 + "' has no content from '" + cmdLine + "'");
         // make sure the successive output files get bigger!
-		if (lastFile != null) {
-			Assert.assertTrue(Files.size(file.toPath()) > Files.size(lastFile.toPath()));
-			System.out.println("this file: " + file + " +" + Files.size(file.toPath()) + " greater than "
-					+ Files.size(lastFile.toPath()));
-		}
-		// remember it for next file
-		lastFile = file;
+        if (lastFile != null)
+        {
+          waitForLastWrite(file,25);
+          
+          if (Files.size(file.toPath()) > Files
+                  .size(lastFile.toPath()))
+          Assert.assertTrue(Files.size(file.toPath()) > Files
+                  .size(lastFile.toPath()));
+        }
+        // remember it for next file
+        lastFile = file;
       }
     } catch (Exception x)
     {
@@ -210,7 +214,22 @@ public class CommandsTest
     }
   }
 
+  private static long waitForLastWrite(File file, int i) throws IOException
+  {
+    long lastSize,stableSize =Files.size(file.toPath());
+    // wait around until we are sure the file has been completely written.
+    do {
+      lastSize = stableSize;
+      try {
+        Thread.sleep(i);
+      } catch (Exception x) {}
+      stableSize=Files.size(file.toPath());
+    } while (stableSize!=lastSize);
+    return stableSize;
+  }
+
   @Test(groups = "Functional", dataProvider = "argfileOutputFiles", singleThreaded = true)
+
   public void argFilesGlobAndSubstitutionsTest(String cmdLine,
           String[] filenames) throws IOException
   {