gh-2435 PR review

gchq · Aug 25, 2021 · 62ff201 · 62ff201
1 parent cae6d6b
commit 62ff201
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/...ava/uk/gov/gchq/gaffer/federatedstore/integration/AbstractStandaloneFederatedStoreIT.java b/...ava/uk/gov/gchq/gaffer/federatedstore/integration/AbstractStandaloneFederatedStoreIT.java
@@ -16,7 +16,7 @@
 
 package uk.gov.gchq.gaffer.federatedstore.integration;
 
-import org.junit.AfterClass;
+import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeEach;
 
 import uk.gov.gchq.gaffer.cache.CacheServiceLoader;
@@ -38,7 +38,7 @@ public void setUp() throws Exception {
         _setUp();
     }
 
-    @AfterClass
+    @AfterAll
     public void tearDown() throws Exception {
         CacheServiceLoader.shutdown();
         _tearDown();

diff --git a/...d-store/src/test/java/uk/gov/gchq/gaffer/federatedstore/integration/FederatedAdminIT.java b/...d-store/src/test/java/uk/gov/gchq/gaffer/federatedstore/integration/FederatedAdminIT.java
@@ -58,8 +58,7 @@ public class FederatedAdminIT extends AbstractStandaloneFederatedStoreIT {
     public static final User ADMIN_USER = new User("admin", Collections.EMPTY_SET, Sets.newHashSet("AdminAuth"));
     public static final User NOT_ADMIN_USER = new User("admin", Collections.EMPTY_SET, Sets.newHashSet("NotAdminAuth"));
 
-    private static Class currentClass = new Object() {
-    }.getClass().getEnclosingClass();
+    private static Class currentClass = new Object() {}.getClass().getEnclosingClass();
     private static final AccumuloProperties ACCUMULO_PROPERTIES = AccumuloProperties.loadStoreProperties(
             StreamUtil.openStream(currentClass, "properties/singleUseAccumuloStore.properties"));
 

diff --git a/...rc/test/java/uk/gov/gchq/gaffer/federatedstore/integration/FederatedStoreRecursionIT.java b/...rc/test/java/uk/gov/gchq/gaffer/federatedstore/integration/FederatedStoreRecursionIT.java
@@ -48,6 +48,7 @@
 import uk.gov.gchq.koryphe.impl.predicate.IsEqual;
 
 import java.util.ArrayList;
+import java.util.concurrent.TimeUnit;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
@@ -64,7 +65,7 @@ public class FederatedStoreRecursionIT {
     private User user = new User();
 
     @Test
-    @Timeout(value = 20000)
+    @Timeout(value = 1, unit = TimeUnit.MINUTES)
     public void shouldNotInfinityLoopWhenAddingElements() throws Exception {
         /*
          * Structure: