Fix some warnings in the test code

Author: sungshik

src/test/java/engineering/swat/watch/APIErrorsTests.java

@@ -37,7 +37,7 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
-public class APIErrorsTests {
+class APIErrorsTests {
 
     private TestDirectory testDir;
 

src/test/java/engineering/swat/watch/DeleteLockTests.java

@@ -51,7 +51,7 @@ void setup() throws IOException {
     }
 
     @AfterEach
-    void cleanup() throws IOException {
+    void cleanup() {
         if (testDir != null) {
             testDir.close();
         }

src/test/java/engineering/swat/watch/RecursiveWatchTests.java

@@ -31,7 +31,6 @@
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
-import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicReference;
 
@@ -56,14 +55,14 @@ void setup() throws IOException {
     }
 
     @AfterEach
-    void cleanup() throws IOException {
+    void cleanup() {
         if (testDir != null) {
             testDir.close();
         }
     }
 
     @BeforeAll
-    static void setupEverything() throws IOException {
+    static void setupEverything() {
         Awaitility.setDefaultTimeout(TestHelper.NORMAL_WAIT);
     }
 
@@ -123,7 +122,7 @@ void correctRelativePathIsReported() throws IOException {
     }
 
     @Test
-    void deleteOfFileInDirectoryShouldBeVisible() throws IOException, InterruptedException {
+    void deleteOfFileInDirectoryShouldBeVisible() throws IOException {
         var target = testDir.getTestFiles()
             .stream()
             .filter(p -> !p.getParent().equals(testDir.getTestDirectory()))

src/test/java/engineering/swat/watch/SingleDirectoryTests.java

@@ -30,8 +30,6 @@
 
 import java.io.IOException;
 import java.nio.file.Files;
-import java.time.Duration;
-import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
 
 import org.awaitility.Awaitility;
@@ -42,7 +40,7 @@
 
 import engineering.swat.watch.WatchEvent.Kind;
 
-public class SingleDirectoryTests {
+class SingleDirectoryTests {
     private TestDirectory testDir;
 
     @BeforeEach
@@ -51,7 +49,7 @@ void setup() throws IOException {
     }
 
     @AfterEach
-    void cleanup() throws IOException {
+    void cleanup() {
         if (testDir != null) {
             testDir.close();
         }
@@ -63,7 +61,7 @@ static void setupEverything() {
     }
 
     @Test
-    void deleteOfFileInDirectoryShouldBeVisible() throws IOException, InterruptedException {
+    void deleteOfFileInDirectoryShouldBeVisible() throws IOException {
         var target = testDir.getTestFiles().get(0);
         var seenDelete = new AtomicBoolean(false);
         var seenCreate = new AtomicBoolean(false);
@@ -91,7 +89,7 @@ void deleteOfFileInDirectoryShouldBeVisible() throws IOException, InterruptedExc
     }
 
     @Test
-    void alternativeAPITest() throws IOException, InterruptedException {
+    void alternativeAPITest() throws IOException {
         var target = testDir.getTestFiles().get(0);
         var seenDelete = new AtomicBoolean(false);
         var seenCreate = new AtomicBoolean(false);

src/test/java/engineering/swat/watch/SingleFileTests.java

@@ -40,7 +40,7 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
-public class SingleFileTests {
+class SingleFileTests {
     private TestDirectory testDir;
 
     @BeforeEach
@@ -49,7 +49,7 @@ void setup() throws IOException {
     }
 
     @AfterEach
-    void cleanup() throws IOException {
+    void cleanup() {
         if (testDir != null) {
             testDir.close();
         }

src/test/java/engineering/swat/watch/SmokeTests.java

@@ -32,7 +32,6 @@
 
 import java.io.IOException;
 import java.nio.file.Files;
-import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
 
 import org.awaitility.Awaitility;
@@ -51,7 +50,7 @@ void setup() throws IOException {
     }
 
     @AfterEach
-    void cleanup() throws IOException {
+    void cleanup() {
         if (testDir != null) {
             testDir.close();
         }
@@ -63,7 +62,7 @@ static void setupEverything() {
     }
 
     @Test
-    void watchDirectory() throws IOException, InterruptedException {
+    void watchDirectory() throws IOException {
         var changed = new AtomicBoolean(false);
         var target = testDir.getTestFiles().get(0);
         var watchConfig = Watcher.watch(testDir.getTestDirectory(), WatchScope.PATH_AND_CHILDREN)
@@ -77,7 +76,7 @@ void watchDirectory() throws IOException, InterruptedException {
     }
 
     @Test
-    void watchRecursiveDirectory() throws IOException, InterruptedException {
+    void watchRecursiveDirectory() throws IOException {
         var changed = new AtomicBoolean(false);
         var target = testDir.getTestFiles().stream()
             .filter(p -> !p.getParent().equals(testDir.getTestDirectory()))