Repository: wicket Updated Branches: refs/heads/wicket-7.x ce14514a5 -> 7d3b7a866
WICKET-6236 honoring the javadoc by putting a wait only after the 10th failed attempt to delete a file Project: http://git-wip-us.apache.org/repos/asf/wicket/repo Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/7d3b7a86 Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/7d3b7a86 Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/7d3b7a86 Branch: refs/heads/wicket-7.x Commit: 7d3b7a8660801c7024b863c971c701c7336fefa9 Parents: ce14514 Author: Pedro Henrique Oliveira dos Santos <[email protected]> Authored: Mon Sep 5 04:31:23 2016 -0300 Committer: Pedro Henrique Oliveira dos Santos <[email protected]> Committed: Mon Sep 5 04:57:12 2016 -0300 ---------------------------------------------------------------------- .../java/org/apache/wicket/util/file/Files.java | 17 ++++++++--------- .../org/apache/wicket/util/file/FilesTest.java | 20 ++++++++++++++++++++ 2 files changed, 28 insertions(+), 9 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/wicket/blob/7d3b7a86/wicket-util/src/main/java/org/apache/wicket/util/file/Files.java ---------------------------------------------------------------------- diff --git a/wicket-util/src/main/java/org/apache/wicket/util/file/Files.java b/wicket-util/src/main/java/org/apache/wicket/util/file/Files.java index 9a2204c..ca61c6f 100644 --- a/wicket-util/src/main/java/org/apache/wicket/util/file/Files.java +++ b/wicket-util/src/main/java/org/apache/wicket/util/file/Files.java @@ -128,15 +128,14 @@ public class Files { return true; } - - try - { - Thread.sleep(100); - } - catch (InterruptedException ix) - { - Thread.currentThread().interrupt(); - } + } + try + { + Thread.sleep(100); + } + catch (InterruptedException ix) + { + Thread.currentThread().interrupt(); } } } http://git-wip-us.apache.org/repos/asf/wicket/blob/7d3b7a86/wicket-util/src/test/java/org/apache/wicket/util/file/FilesTest.java ---------------------------------------------------------------------- diff --git a/wicket-util/src/test/java/org/apache/wicket/util/file/FilesTest.java b/wicket-util/src/test/java/org/apache/wicket/util/file/FilesTest.java index 2884c30..75fdab8 100644 --- a/wicket-util/src/test/java/org/apache/wicket/util/file/FilesTest.java +++ b/wicket-util/src/test/java/org/apache/wicket/util/file/FilesTest.java @@ -16,6 +16,11 @@ */ package org.apache.wicket.util.file; +import static java.lang.System.currentTimeMillis; +import static org.hamcrest.Matchers.lessThan; +import static org.mockito.Mockito.mock; +import static org.mockito.Mockito.when; + import java.io.IOException; import java.net.URL; @@ -63,6 +68,21 @@ public class FilesTest extends Assert } /** + * WICKET-6236 - honoring the javadoc by putting a wait only after the 10th failed attempt to delete a file + */ + @Test + public void dontWaitTooMuchIfCantDelete(){ + java.io.File f = mock(java.io.File.class); + when(f.isFile()).thenReturn(true); + when(f.delete()).thenReturn(false); + long start = currentTimeMillis(); + Files.remove(f); + long end = currentTimeMillis(); + assertThat(end - start, lessThan(5000l)); + } + + + /** * Tests for {@link Files#removeFolder(java.io.File)} * * @throws Exception
