From 068b0e772899ac95051e10d611568b331e8b1c01 Mon Sep 17 00:00:00 2001 From: Frederic Bregier Date: Thu, 7 Oct 2021 21:36:17 +0200 Subject: [PATCH] Fix tests to pass on CICD under heavy load Some tests were done by assuming too much the CICD host was not used by other CICD workloads. Unfortunately those tests, while valid in real condition, were assuming too short times to pass. The tests are adapted a bit to be more tolerant on CPU usage. Also update dependencies. --- .../waarp/common/utility/ProcessesTest.java | 2 +- .../NetworkClientCompressionVs35Test.java | 46 +++++++++++---- .../NetworkClientOnlyBothCompressionTest.java | 46 +++++++++++---- .../NetworkClientOnlyOneCompressionTest.java | 46 +++++++++++---- .../protocol/junit/NetworkClientTest.java | 57 ++++++++++++++----- .../monitoring/HttpMonitoringAbstract.java | 42 +++++++++++--- doc/waarp-r66/source/changes.rst | 4 +- .../source/interface/rest/general.rst | 2 +- pom.xml | 45 ++++++++------- 9 files changed, 213 insertions(+), 77 deletions(-) diff --git a/WaarpCommon/src/test/java/org/waarp/common/utility/ProcessesTest.java b/WaarpCommon/src/test/java/org/waarp/common/utility/ProcessesTest.java index 1094ab161a..f49c2580b4 100644 --- a/WaarpCommon/src/test/java/org/waarp/common/utility/ProcessesTest.java +++ b/WaarpCommon/src/test/java/org/waarp/common/utility/ProcessesTest.java @@ -39,7 +39,7 @@ public class ProcessesTest { public boolean waitForPidDone(int pid) { int i; - for (i = 0; i < 10; i++) { + for (i = 0; i < 50; i++) { try { Thread.sleep(TIMEMS); } catch (InterruptedException ignored) {//NOSONAR diff --git a/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientCompressionVs35Test.java b/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientCompressionVs35Test.java index 93f4cea0a4..66189ca366 100644 --- a/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientCompressionVs35Test.java +++ b/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientCompressionVs35Test.java @@ -900,7 +900,7 @@ public void test6_RecvUsingTrafficShaping() throws IOException { totestBig.delete(); } - private void test_Spooled(final SpooledThread spooledThread, final int factor) + private void test_Spooled(final SpooledThread spooledThread) throws IOException, InterruptedException { final int size = 200; Configuration.configuration.changeNetworkLimit(0, 0, 0, 0, 1000); @@ -912,25 +912,51 @@ private void test_Spooled(final SpooledThread spooledThread, final int factor) spooledThread.future = future; spooledThread.start(); Thread.sleep(200); + int cpt = 0; logger.warn("1rst file"); final File totestBig = generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } logger.warn("1rst file delete"); totestBig.delete(); Thread.sleep(2000); logger.warn("Second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); - logger.warn("Third file"); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + logger.warn("Third file modifying second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size + 1, "abcdefghij"); - Thread.sleep(4000 / factor); + if (spooledThread.ignoreAlreadyUsed) { + Thread.sleep(2000); + } else { + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + } logger.warn("Third file deleted"); totestBig.delete(); - Thread.sleep(2000 / factor); + Thread.sleep(1000); generateOutFile(stop.getAbsolutePath(), 10); future.awaitOrInterruptible(); assertTrue(future.isSuccess()); @@ -948,7 +974,7 @@ public void test70_Spooled() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = false; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -959,7 +985,7 @@ public void test70_SpooledSubmit() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -970,7 +996,7 @@ public void test70_SpooledIgnore() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = true; spooledThread.submit = false; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(2, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -982,7 +1008,7 @@ public void test70_SpooledSubmitIgnore() SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = true; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(2, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } diff --git a/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientOnlyBothCompressionTest.java b/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientOnlyBothCompressionTest.java index 13dc53df5f..06578e2027 100644 --- a/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientOnlyBothCompressionTest.java +++ b/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientOnlyBothCompressionTest.java @@ -1108,7 +1108,7 @@ public void test6_RecvUsingTrafficShapingNoCompress() throws IOException { totestBig.delete(); } - private void test_Spooled(final SpooledThread spooledThread, final int factor) + private void test_Spooled(final SpooledThread spooledThread) throws IOException, InterruptedException { final int size = 200; Configuration.configuration.changeNetworkLimit(0, 0, 0, 0, 1000); @@ -1120,25 +1120,51 @@ private void test_Spooled(final SpooledThread spooledThread, final int factor) spooledThread.future = future; spooledThread.start(); Thread.sleep(200); + int cpt = 0; logger.warn("1rst file"); final File totestBig = generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } logger.warn("1rst file delete"); totestBig.delete(); Thread.sleep(2000); logger.warn("Second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); - logger.warn("Third file"); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + logger.warn("Third file modifying second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size + 1, "abcdefghij"); - Thread.sleep(4000 / factor); + if (spooledThread.ignoreAlreadyUsed) { + Thread.sleep(2000); + } else { + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + } logger.warn("Third file deleted"); totestBig.delete(); - Thread.sleep(2000 / factor); + Thread.sleep(1000); generateOutFile(stop.getAbsolutePath(), 10); future.awaitOrInterruptible(); assertTrue(future.isSuccess()); @@ -1156,7 +1182,7 @@ public void test70_Spooled() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = false; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1167,7 +1193,7 @@ public void test70_SpooledSubmit() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1178,7 +1204,7 @@ public void test70_SpooledIgnore() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = true; spooledThread.submit = false; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(2, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1190,7 +1216,7 @@ public void test70_SpooledSubmitIgnore() SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = true; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(2, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } diff --git a/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientOnlyOneCompressionTest.java b/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientOnlyOneCompressionTest.java index ba00273cb4..b2933009cc 100644 --- a/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientOnlyOneCompressionTest.java +++ b/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientOnlyOneCompressionTest.java @@ -1025,7 +1025,7 @@ public void test6_RecvUsingTrafficShaping() throws IOException { totestBig.delete(); } - private void test_Spooled(final SpooledThread spooledThread, final int factor) + private void test_Spooled(final SpooledThread spooledThread) throws IOException, InterruptedException { final int size = 200; Configuration.configuration.changeNetworkLimit(0, 0, 0, 0, 1000); @@ -1037,25 +1037,51 @@ private void test_Spooled(final SpooledThread spooledThread, final int factor) spooledThread.future = future; spooledThread.start(); Thread.sleep(200); + int cpt = 0; logger.warn("1rst file"); final File totestBig = generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } logger.warn("1rst file delete"); totestBig.delete(); Thread.sleep(2000); logger.warn("Second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); - logger.warn("Third file"); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + logger.warn("Third file modifying second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size + 1, "abcdefghij"); - Thread.sleep(4000 / factor); + if (spooledThread.ignoreAlreadyUsed) { + Thread.sleep(2000); + } else { + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + } logger.warn("Third file deleted"); totestBig.delete(); - Thread.sleep(2000 / factor); + Thread.sleep(1000); generateOutFile(stop.getAbsolutePath(), 10); future.awaitOrInterruptible(); assertTrue(future.isSuccess()); @@ -1073,7 +1099,7 @@ public void test70_Spooled() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = false; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1084,7 +1110,7 @@ public void test70_SpooledSubmit() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1095,7 +1121,7 @@ public void test70_SpooledIgnore() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = true; spooledThread.submit = false; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(2, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1107,7 +1133,7 @@ public void test70_SpooledSubmitIgnore() SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = true; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(2, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } diff --git a/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientTest.java b/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientTest.java index 286e2b98c7..286757ce3b 100644 --- a/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientTest.java +++ b/WaarpR66/src/test/java/org/waarp/openr66/protocol/junit/NetworkClientTest.java @@ -1279,7 +1279,7 @@ public void test6_RecvUsingTrafficShaping() throws IOException { totestBig.delete(); } - private void test_Spooled(final SpooledThread spooledThread, final int factor) + private void test_Spooled(final SpooledThread spooledThread) throws IOException, InterruptedException { final int size = 200; Configuration.configuration.changeNetworkLimit(0, 0, 0, 0, 1000); @@ -1291,25 +1291,54 @@ private void test_Spooled(final SpooledThread spooledThread, final int factor) spooledThread.future = future; spooledThread.start(); Thread.sleep(200); + int cpt = 0; logger.warn("1rst file"); final File totestBig = generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() + + spooledThread.spooledDirectoryTransfer.getError() != cpt) { + Thread.sleep(200); + } else { + break; + } + } logger.warn("1rst file delete"); totestBig.delete(); Thread.sleep(2000); logger.warn("Second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); - logger.warn("Third file"); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() + + spooledThread.spooledDirectoryTransfer.getError() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + logger.warn("Third file modifying second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size + 1, "abcdefghij"); - Thread.sleep(4000 / factor); + if (spooledThread.ignoreAlreadyUsed) { + Thread.sleep(2000); + } else { + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() + + spooledThread.spooledDirectoryTransfer.getError() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + } logger.warn("Third file deleted"); totestBig.delete(); - Thread.sleep(2000 / factor); + Thread.sleep(1000); generateOutFile(stop.getAbsolutePath(), 10); future.awaitOrInterruptible(); assertTrue(future.isSuccess()); @@ -1328,7 +1357,7 @@ public void test70_Spooled() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = false; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); if (spooledThread.spooledDirectoryTransfer.getSent() != 3) { Thread.sleep(100); if (spooledThread.spooledDirectoryTransfer.getSent() != 3) { @@ -1348,7 +1377,7 @@ public void test70_SpooledSubmit() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1359,7 +1388,7 @@ public void test70_SpooledIgnore() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = true; spooledThread.submit = false; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(2, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1371,7 +1400,7 @@ public void test70_SpooledSubmitIgnore() SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = true; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(2, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1385,7 +1414,7 @@ public void test70_SpooledWrongHost() spooledThread.submit = false; spooledThread.host = "hostbs"; // spooledThread.waarpHost = null; - test_Spooled(spooledThread, 1); + test_Spooled(spooledThread); assertEquals(0, spooledThread.spooledDirectoryTransfer.getSent()); assertTrue(3 <= spooledThread.spooledDirectoryTransfer.getError()); } @@ -1398,7 +1427,7 @@ public void test70_SpooledSubmitWrongHost() spooledThread.ignoreAlreadyUsed = false; spooledThread.host = "hostbs"; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -1412,7 +1441,7 @@ public void test70_SpooledWrongHostIgnore() spooledThread.host = "hostbs"; spooledThread.submit = false; //spooledThread.waarpHost = null; - test_Spooled(spooledThread, 1); + test_Spooled(spooledThread); assertEquals(0, spooledThread.spooledDirectoryTransfer.getSent()); assertTrue(3 <= spooledThread.spooledDirectoryTransfer.getError()); } @@ -1425,7 +1454,7 @@ public void test70_SpooledSubmitWrongHostIgnore() spooledThread.ignoreAlreadyUsed = true; spooledThread.host = "hostbs"; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(2, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } diff --git a/WaarpR66/src/test/java/org/waarp/openr66/protocol/monitoring/HttpMonitoringAbstract.java b/WaarpR66/src/test/java/org/waarp/openr66/protocol/monitoring/HttpMonitoringAbstract.java index 76bfeb5b11..3f3ce43226 100644 --- a/WaarpR66/src/test/java/org/waarp/openr66/protocol/monitoring/HttpMonitoringAbstract.java +++ b/WaarpR66/src/test/java/org/waarp/openr66/protocol/monitoring/HttpMonitoringAbstract.java @@ -299,7 +299,7 @@ public void test6_RecvUsingTrafficShaping() throws IOException { numberTransfer++; } - private void test_Spooled(final SpooledThread spooledThread, final int factor) + private void test_Spooled(final SpooledThread spooledThread) throws IOException, InterruptedException { final int size = 200; Configuration.configuration.changeNetworkLimit(0, 0, 0, 0, 1000); @@ -311,25 +311,51 @@ private void test_Spooled(final SpooledThread spooledThread, final int factor) spooledThread.future = future; spooledThread.start(); Thread.sleep(200); + int cpt = 0; logger.warn("1rst file"); final File totestBig = generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } logger.warn("1rst file delete"); totestBig.delete(); Thread.sleep(2000); logger.warn("Second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size); - Thread.sleep(4000 / factor); - logger.warn("Third file"); + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + logger.warn("Third file modifying second file"); generateOutFile( SpooledThread.TMP_R_66_TEST_OUT_EXAMPLE + "/testTaskBig.txt", size + 1, "abcdefghij"); - Thread.sleep(4000 / factor); + if (spooledThread.ignoreAlreadyUsed) { + Thread.sleep(2000); + } else { + cpt++; + for (int i = 0; i < 100; i++) { + if (spooledThread.spooledDirectoryTransfer.getSent() != cpt) { + Thread.sleep(200); + } else { + break; + } + } + } logger.warn("Third file deleted"); totestBig.delete(); - Thread.sleep(2000 / factor); + Thread.sleep(1000); generateOutFile(stop.getAbsolutePath(), 10); future.awaitOrInterruptible(); assertTrue(future.isSuccess()); @@ -348,7 +374,7 @@ public void test70_Spooled() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = false; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } @@ -359,7 +385,7 @@ public void test70_SpooledSubmit() throws IOException, InterruptedException { SpooledThread spooledThread = new SpooledThread(); spooledThread.ignoreAlreadyUsed = false; spooledThread.submit = true; - test_Spooled(spooledThread, 2); + test_Spooled(spooledThread); assertEquals(3, spooledThread.spooledDirectoryTransfer.getSent()); assertEquals(0, spooledThread.spooledDirectoryTransfer.getError()); } diff --git a/doc/waarp-r66/source/changes.rst b/doc/waarp-r66/source/changes.rst index 264a8f873a..68104eefdc 100644 --- a/doc/waarp-r66/source/changes.rst +++ b/doc/waarp-r66/source/changes.rst @@ -7,7 +7,7 @@ La procédure de mise à jour est disponible ici: :any:`upgrade` Non publié ========== -Waarp R66 3.6.1 (2021-09-20) +Waarp R66 3.6.1 (2021-12-10) ============================ @@ -41,6 +41,8 @@ Correctifs (issue [`#103 `__]) - Amélioration de la documentation et de sa génération [`#98 `__] +- Corrections de tests non passant sur des environnements chargés par ailleurs + (issue [`#104 `__]) - Mise à jour des dépendances diff --git a/doc/waarp-r66/source/interface/rest/general.rst b/doc/waarp-r66/source/interface/rest/general.rst index 38dd4f9a98..3c50d50912 100644 --- a/doc/waarp-r66/source/interface/rest/general.rst +++ b/doc/waarp-r66/source/interface/rest/general.rst @@ -48,7 +48,7 @@ requêtée selon l'algorithme suivant : sont utilisés, sont convertis en minuscules et triés dans l'ordre alphabétique des identifiants pour reconstruire une nouvelle chaine de requête. 2. Le paramètre ``X-Auth-InternalKey`` est ajouté à la chaîne de requête - obtenue à l'étape precedente, avec pour valeur le mot de passe de + obtenue à l'étape précédente, avec pour valeur le mot de passe de l'utilisateur. 3. La chaîne de caractère utilisée pour le calcul de la signature est construite en concaténant le chemin de l'url et la chaîne de requête obtenue diff --git a/pom.xml b/pom.xml index 394c2e5fd1..39e660778a 100644 --- a/pom.xml +++ b/pom.xml @@ -269,8 +269,8 @@ test - 4.1.68.Final - 2.0.43.Final + 4.1.69.Final + 2.0.44.Final 1.0.3 1.5.0 @@ -289,16 +289,16 @@ 2.12.1 0.3.6 2.8.8 - 9.4.43.v20210629 + 9.4.44.v20210927 0.9.3-1 4.5.13 4.4.14 1.7.2 - 2.10.11 + 2.10.12 1.5.0-4 1.2.0 - 1.11.18 + 1.11.20 5.9.0 3.1.10 4.13.2 @@ -310,7 +310,7 @@ 2.53.0 4.4.0 3.12.4 - 30.1.1-jre + 31.0.1-jre 1.8.0 @@ -355,11 +355,12 @@ 2.7 1.2.7 4.2.3 - 4.4.1 + 4.4.2 2.5.2 1.9 2.2.0 3.0.0 + 1.3 @@ -1200,7 +1201,7 @@ org.codehaus.mojo extra-enforcer-rules - 1.3 + ${extra-enforcer-rules.version} org.codehaus.mojo @@ -1288,7 +1289,7 @@ ${waarp.version}-jre8 WaarpPackaging - 30.1.1-jre + 31.0.1-jre 1.2.4 1.21 2.11.0 @@ -1301,8 +1302,8 @@ 2.5.4 8.0.20 42.2.24 - 2.12.5 - 2.12.5 + 2.13.0 + 2.13.0 2.1.3 4.7.1.Final 2.1.1 @@ -1314,10 +1315,10 @@ 2.3.5 2.3.0.1 10.6 - 2.34 + 2.35 2.14.1 - 8.3.0 - 4.9.1 + 8.3.1 + 4.9.2 2.10.0 1.5.31 7.15.0 @@ -1355,7 +1356,7 @@ org.codehaus.mojo extra-enforcer-rules - 1.3 + ${extra-enforcer-rules.version} org.codehaus.mojo @@ -1480,7 +1481,7 @@ ${waarp.version}-jre11 WaarpPackaging - 30.1.1-jre + 31.0.1-jre 1.2.4 1.21 2.11.0 @@ -1492,8 +1493,8 @@ 1.4.200 2.5.4 8.0.20 - 2.12.5 - 2.12.5 + 2.13.0 + 2.13.0 42.2.24 2.1.3 4.7.2.Final @@ -1506,10 +1507,10 @@ 2.3.5 2.3.0.1 10.6 - 2.34 + 2.35 2.14.1 - 8.3.0 - 4.9.1 + 8.3.1 + 4.9.2 2.10.0 1.5.31 7.15.0 @@ -1532,7 +1533,7 @@ org.codehaus.mojo extra-enforcer-rules - 1.3 + ${extra-enforcer-rules.version} org.codehaus.mojo