Convert common to JUnit 5 51/138351/1
authorwaynedunican <wayne.dunican@est.tech>
Thu, 27 Jun 2024 15:57:49 +0000 (16:57 +0100)
committerwaynedunican <wayne.dunican@est.tech>
Mon, 1 Jul 2024 11:16:18 +0000 (12:16 +0100)
Review for utils-test

Issue-ID: POLICY-5043
Change-Id: I5269fca2b71ca2e814bbb050cd2a5505f47419fa
Signed-off-by: waynedunican <wayne.dunican@est.tech>
25 files changed:
utils-test/pom.xml
utils-test/src/main/java/org/onap/policy/common/utils/gson/GsonTestUtils.java
utils-test/src/main/java/org/onap/policy/common/utils/test/ExceptionsTester.java
utils-test/src/main/java/org/onap/policy/common/utils/test/ThrowablesTester.java
utils-test/src/main/java/org/onap/policy/common/utils/time/TestTimeMulti.java
utils-test/src/test/java/org/onap/policy/common/utils/gson/GsonSerializerTest.java
utils-test/src/test/java/org/onap/policy/common/utils/gson/GsonTestUtilsBuilderTest.java
utils-test/src/test/java/org/onap/policy/common/utils/gson/GsonTestUtilsTest.java
utils-test/src/test/java/org/onap/policy/common/utils/io/SerializerTest.java
utils-test/src/test/java/org/onap/policy/common/utils/security/SelfSignedKeyStoreTest.java
utils-test/src/test/java/org/onap/policy/common/utils/test/ConstructionErrorTest.java
utils-test/src/test/java/org/onap/policy/common/utils/test/ErrorsTesterTest.java
utils-test/src/test/java/org/onap/policy/common/utils/test/ExceptionsTesterTest.java
utils-test/src/test/java/org/onap/policy/common/utils/test/ThrowablesTesterTest.java
utils-test/src/test/java/org/onap/policy/common/utils/test/log/logback/ExtractAppenderTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/PeriodicItemTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/PseudoExecutorTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/PseudoScheduledExecutorServiceTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/PseudoScheduledFutureTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/PseudoTimerTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/RunnableItemTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/SleepItemTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/TestTimeMultiTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/TestTimeTest.java
utils-test/src/test/java/org/onap/policy/common/utils/time/WorkItemTest.java

index 1b00a69..a659062 100644 (file)
@@ -68,8 +68,8 @@
             <scope>compile</scope>
         </dependency>
         <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-api</artifactId>
             <scope>compile</scope>
         </dependency>
         <dependency>
@@ -79,7 +79,7 @@
         </dependency>
         <dependency>
             <groupId>org.mockito</groupId>
-            <artifactId>mockito-core</artifactId>
+            <artifactId>mockito-junit-jupiter</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>
             <artifactId>spring-test</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.hamcrest</groupId>
+            <artifactId>hamcrest-core</artifactId>
+            <version>2.2</version>
+            <scope>compile</scope>
+        </dependency>
     </dependencies>
 </project>
index 8276ea4..da62569 100644 (file)
@@ -21,7 +21,7 @@
 
 package org.onap.policy.common.utils.gson;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import com.google.gson.Gson;
 import com.google.gson.JsonArray;
index 1ab1f28..34dc568 100644 (file)
@@ -3,6 +3,7 @@
  * Common Utils-Test
  * =============================================================================
  * Copyright (C) 2018, 2021 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * =============================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,9 +21,9 @@
 
 package org.onap.policy.common.utils.test;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.lang.reflect.Constructor;
 
index 06ca804..62584c7 100644 (file)
@@ -3,6 +3,7 @@
  * Common Utils-Test
  * =============================================================================
  * Copyright (C) 2018, 2020-2021 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * =============================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.test;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.lang.reflect.Constructor;
 import java.lang.reflect.InvocationTargetException;
index 9e61eaa..8ffcd36 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2018-2021 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,7 +21,7 @@
 
 package org.onap.policy.common.utils.time;
 
-import static org.junit.Assert.fail;
+import static org.junit.jupiter.api.Assertions.fail;
 
 import java.util.Iterator;
 import java.util.LinkedList;
index 82f5ede..aa70143 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -26,12 +27,12 @@ import com.google.gson.stream.JsonReader;
 import com.google.gson.stream.JsonWriter;
 import java.io.IOException;
 import java.io.StringReader;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
-public class GsonSerializerTest {
+class GsonSerializerTest {
 
     @Test
-    public void testReadJsonReader() {
+    void testReadJsonReader() {
         JsonReader rdr = new JsonReader(new StringReader("10"));
 
         GsonSerializer<Object> ser = new GsonSerializer<Object>() {
index 466041e..d85653d 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,7 +21,7 @@
 
 package org.onap.policy.common.utils.gson;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
@@ -30,20 +31,20 @@ import com.google.gson.TypeAdapterFactory;
 import com.google.gson.reflect.TypeToken;
 import com.google.gson.stream.JsonWriter;
 import java.io.IOException;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class GsonTestUtilsBuilderTest {
+class GsonTestUtilsBuilderTest {
 
     private GsonTestUtils utils;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         utils = new MyBuilder().build();
     }
 
     @Test
-    public void testBuilderAddMock() {
+    void testBuilderAddMock() {
         PreMock pre = mock(PreMock.class);
         when(pre.getId()).thenReturn(2000);
 
index 51522ec..d355823 100644 (file)
@@ -23,9 +23,9 @@ package org.onap.policy.common.utils.gson;
 
 import static org.assertj.core.api.Assertions.assertThatCode;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
 
 import com.google.gson.JsonArray;
 import com.google.gson.JsonElement;
@@ -38,26 +38,26 @@ import java.io.IOException;
 import lombok.Getter;
 import lombok.Setter;
 import lombok.ToString;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class GsonTestUtilsTest {
+class GsonTestUtilsTest {
     private static final String HELLO = "hello";
 
     private GsonTestUtils utils;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         utils = new GsonTestUtils();
     }
 
     @Test
-    public void testGetGson() {
+    void testGetGson() {
         assertNotNull(utils.getGson());
     }
 
     @Test
-    public void testGsonRoundTrip() {
+    void testGsonRoundTrip() {
         Data data = new Data();
         data.setId(500);
 
@@ -71,7 +71,7 @@ public class GsonTestUtilsTest {
     }
 
     @Test
-    public void testCompareGsonObjectClass_testCompareGsonObjectFile() {
+    void testCompareGsonObjectClass_testCompareGsonObjectFile() {
         Data data = new Data();
         data.setId(500);
         data.setText(HELLO);
@@ -98,7 +98,7 @@ public class GsonTestUtilsTest {
     }
 
     @Test
-    public void testCompareGsonObjectString() {
+    void testCompareGsonObjectString() {
         Data data = new Data();
         data.setId(600);
         data.setText(HELLO);
@@ -108,7 +108,7 @@ public class GsonTestUtilsTest {
     }
 
     @Test
-    public void testCompareGsonObjectJsonElement() {
+    void testCompareGsonObjectJsonElement() {
         Data data = new Data();
         data.setId(650);
         data.setText(HELLO);
@@ -125,7 +125,7 @@ public class GsonTestUtilsTest {
     }
 
     @Test
-    public void testApplyScripts() {
+    void testApplyScripts() {
         Data data = new Data();
         data.setId(700);
         data.setText(HELLO);
@@ -144,7 +144,7 @@ public class GsonTestUtilsTest {
     }
 
     @Test
-    public void testReorderJsonObject() {
+    void testReorderJsonObject() {
         // insert properties in a non-alphabetical order
         JsonObject inner = new JsonObject();
         inner.addProperty("objBint", 100);
@@ -168,7 +168,7 @@ public class GsonTestUtilsTest {
     }
 
     @Test
-    public void testReorderJsonArray() {
+    void testReorderJsonArray() {
         // insert properties in a non-alphabetical order
         JsonObject inner = new JsonObject();
         inner.add("objCNull", JsonNull.INSTANCE);
@@ -185,7 +185,7 @@ public class GsonTestUtilsTest {
     }
 
     @Test
-    public void testReorderJsonElement() {
+    void testReorderJsonElement() {
         // null element
         JsonElement jsonEl = null;
         assertNull(utils.reorder(jsonEl));
index 203c78e..25461da 100644 (file)
@@ -3,7 +3,7 @@
  * ONAP Policy Engine - Common Modules
  * ================================================================================
  * Copyright (C) 2018-2020 AT&T Intellectual Property. All rights reserved.
- * Modificaitons Copyright (C) 2023 Nordix Foundation.
+ * Modificaitons Copyright (C) 2023-2024 Nordix Foundation.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,9 +22,9 @@
 package org.onap.policy.common.utils.io;
 
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.Mockito.doThrow;
 import static org.mockito.Mockito.mock;
 
@@ -37,14 +37,14 @@ import java.io.Serializable;
 import java.nio.charset.Charset;
 import java.nio.charset.StandardCharsets;
 import java.util.Arrays;
-import org.junit.AfterClass;
-import org.junit.Before;
-import org.junit.BeforeClass;
-import org.junit.Test;
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 import org.onap.policy.common.utils.io.Serializer.Factory;
 import org.springframework.test.util.ReflectionTestUtils;
 
-public class SerializerTest {
+class SerializerTest {
     private static final String FACTORY = "factory";
 
     /**
@@ -52,28 +52,28 @@ public class SerializerTest {
      */
     private static Factory saveFactory;
 
-    @BeforeClass
+    @BeforeAll
     public static void setUpBeforeClass() {
         saveFactory = (Factory) ReflectionTestUtils.getField(Serializer.class, FACTORY);
     }
 
-    @AfterClass
+    @AfterAll
     public static void tearDownAfterClass() {
         ReflectionTestUtils.setField(Serializer.class, FACTORY, saveFactory);
     }
 
-    @Before
+    @BeforeEach
     public void setUp() {
         setFactory(saveFactory);
     }
 
     @Test
-    public void testFactory() {
+    void testFactory() {
         assertNotNull(saveFactory);
     }
 
     @Test
-    public void testSerialize() throws Exception {
+    void testSerialize() throws Exception {
         MyObject obj1 = new MyObject(3);
         byte[] data = Serializer.serialize(obj1);
         assertTrue(data.length > 0);
@@ -85,13 +85,14 @@ public class SerializerTest {
         assertEquals(obj1.value, obj2.value);
     }
 
-    @Test(expected = java.io.NotSerializableException.class)
-    public void testSerialize_Ex() throws Exception {
-        Serializer.serialize(new NotSerializable());
+    @Test
+    void testSerialize_Ex() {
+        assertThatThrownBy(() -> Serializer.serialize(new NotSerializable()))
+            .isInstanceOf(java.io.NotSerializableException.class);
     }
 
     @Test
-    public void testSerialize_ArrayCloseEx() {
+    void testSerialize_ArrayCloseEx() {
         IOException ex = new IOException("testSerialize_ArrayCloseEx");
 
         /*
@@ -125,7 +126,7 @@ public class SerializerTest {
     }
 
     @Test
-    public void testSerialize_ObjectWriteEx() {
+    void testSerialize_ObjectWriteEx() {
         IOException ex = new IOException("testSerialize_ObjectWriteEx");
 
         /*
@@ -142,7 +143,7 @@ public class SerializerTest {
     }
 
     @Test
-    public void testSerialize_ObjectCloseEx() throws Exception {
+    void testSerialize_ObjectCloseEx() throws Exception {
         IOException ex = new IOException("testSerialize_ObjectCloseEx");
         ObjectOutputStream oos = mock(ObjectOutputStream.class);
         doThrow(ex).when(oos).close();
@@ -168,7 +169,7 @@ public class SerializerTest {
     }
 
     @Test
-    public void testSerialize_BothCloseEx() throws Exception {
+    void testSerialize_BothCloseEx() throws Exception {
         IOException ex = new IOException("testSerialize_BothCloseEx");
         IOException ex2 = new IOException("testSerialize_BothCloseEx_2");
         ObjectOutputStream oos = mock(ObjectOutputStream.class);
@@ -216,14 +217,14 @@ public class SerializerTest {
     }
 
     @Test
-    public void testDeserialize() throws Exception {
+    void testDeserialize() throws Exception {
         MyObject obj1 = new MyObject(3);
         MyObject obj2 = Serializer.roundTrip(obj1);
         assertEquals(obj1.value, obj2.value);
     }
 
     @Test
-    public void testDeserialize_ArrayCloseEx() throws Exception {
+    void testDeserialize_ArrayCloseEx() {
         IOException ex = new IOException("testSerialize_ObjectWriteEx");
 
         /*
@@ -252,7 +253,7 @@ public class SerializerTest {
     }
 
     @Test
-    public void testDeserialize_ObjectReadEx() throws Exception {
+    void testDeserialize_ObjectReadEx() {
         IOException ex = new IOException("testDeserialize_ObjectReadEx");
 
         /*
@@ -269,7 +270,7 @@ public class SerializerTest {
     }
 
     @Test
-    public void testDeserialize_ObjectRead_ClassEx() throws Exception {
+    void testDeserialize_ObjectRead_ClassEx() throws Exception {
         MyObject obj1 = new MyObject(200);
 
         // must use binary character set
@@ -302,7 +303,7 @@ public class SerializerTest {
     }
 
     @Test
-    public void testDeserialize_ObjectCloseEx() throws Exception {
+    void testDeserialize_ObjectCloseEx() {
         IOException ex = new IOException("testDeserialize_ObjectCloseEx");
 
         /*
@@ -325,7 +326,7 @@ public class SerializerTest {
     }
 
     @Test
-    public void testDeserialize_BothCloseEx() throws Exception {
+    void testDeserialize_BothCloseEx() {
         IOException ex = new IOException("testDeserialize_BothCloseEx");
         IOException ex2 = new IOException("testDeserialize_BothCloseEx_2");
 
@@ -359,16 +360,17 @@ public class SerializerTest {
     }
 
     @Test
-    public void testRoundTrip() throws Exception {
+    void testRoundTrip() throws Exception {
         MyObject obj1 = new MyObject(3);
 
         MyObject obj2 = Serializer.roundTrip(obj1);
         assertEquals(obj1.value, obj2.value);
     }
 
-    @Test(expected = java.io.NotSerializableException.class)
-    public void testRoundTrip_Ex() throws Exception {
-        Serializer.roundTrip(new NotSerializable());
+    @Test
+    void testRoundTrip_Ex() {
+        assertThatThrownBy(() -> Serializer.roundTrip(new NotSerializable()))
+            .isInstanceOf(java.io.NotSerializableException.class);
     }
 
     /**
index 27bceac..8a3dfe3 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2021 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -27,9 +28,9 @@ import java.io.File;
 import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.util.concurrent.TimeUnit;
-import org.junit.Before;
-import org.junit.BeforeClass;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 public class SelfSignedKeyStoreTest {
     private static final String USER_DIR_PROP = "user.dir";
@@ -42,14 +43,14 @@ public class SelfSignedKeyStoreTest {
     /**
      * Saves the user.dir property and initializes static fields.
      */
-    @BeforeClass
+    @BeforeAll
     public static void setUpBeforeClass() {
         saveUserDir = System.getProperty(USER_DIR_PROP);
         defaultName = saveUserDir + "/" + SelfSignedKeyStore.RELATIVE_PATH;
         defaultKeystore = new File(defaultName);
     }
 
-    @Before
+    @BeforeEach
     public void setUp() {
         System.setProperty(USER_DIR_PROP, saveUserDir);
         delete(defaultKeystore);
@@ -108,7 +109,7 @@ public class SelfSignedKeyStoreTest {
      * Tests the constructor, when the SAN file is not found.
      */
     @Test
-    public void testSelfSignedKeyStoreStringNoSanFile() throws Exception {
+    public void testSelfSignedKeyStoreStringNoSanFile() {
         assertThatThrownBy(() -> new SelfSignedKeyStore() {
             @Override
             protected String getKeystoreSanName() {
@@ -121,7 +122,7 @@ public class SelfSignedKeyStoreTest {
      * Tests the constructor, when write fails.
      */
     @Test
-    public void testSelfSignedKeyStoreStringWriteFailure() throws Exception {
+    public void testSelfSignedKeyStoreStringWriteFailure() {
         assertThatThrownBy(() -> new SelfSignedKeyStore("target/unknown/path/to/keystore"))
                         .isInstanceOf(IOException.class);
     }
index 8a60a5d..95bba10 100644 (file)
@@ -3,6 +3,7 @@
  * Common Utils-Test
  * ================================================================================
  * Copyright (C) 2018-2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.test;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
-public class ConstructionErrorTest extends ErrorsTester {
+class ConstructionErrorTest extends ErrorsTester {
 
     @Test
-    public void test() {
+    void test() {
         assertEquals(4, testAllError(ConstructionError.class));
     }
 
index 2009e29..b6d91f0 100644 (file)
@@ -3,6 +3,7 @@
  * Common Utils-Test
  * ================================================================================
  * Copyright (C) 2018 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.test;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
-public class ErrorsTesterTest {
+class ErrorsTesterTest {
 
     @Test
-    public void test() {
+    void test() {
         assertEquals(2, new ErrorsTester().testAllError(SimpleError.class));
         assertEquals(5, new ErrorsTester().testAllError(StaticError.class));
     }
index 89ff3ba..81a8503 100644 (file)
@@ -3,6 +3,7 @@
  * Common Utils-Test
  * ================================================================================
  * Copyright (C) 2018 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.test;
 
-import static org.junit.Assert.assertEquals;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
-public class ExceptionsTesterTest {
+class ExceptionsTesterTest {
 
     @Test
-    public void test() {
+    void test() {
         assertEquals(2, new ExceptionsTester().test(SimpleException.class));
         assertEquals(8, new ExceptionsTester().test(StaticException.class));
     }
 
-    @Test(expected = AssertionError.class)
-    public void testNoConstructorsException() {
-        new ExceptionsTester().test(NoConstructorsException.class);
+    @Test
+    void testNoConstructorsException() {
+        ExceptionsTester tester = new ExceptionsTester();
+        assertThatThrownBy(() -> tester.test(NoConstructorsException.class))
+            .isInstanceOf(AssertionError.class);
     }
 
     /**
index 3cb53d8..7775ef3 100644 (file)
@@ -3,6 +3,7 @@
  * Common Utils-Test
  * ================================================================================
  * Copyright (C) 2018-2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.test;
 
-import static org.junit.Assert.assertEquals;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
-public class ThrowablesTesterTest {
+class ThrowablesTesterTest {
 
     @Test
-    public void test() {
+    void test() {
         assertEquals(2, new ThrowablesTester().testAllThrowable(SimpleThrowable.class));
         assertEquals(5, new ThrowablesTester().testAllThrowable(StaticThrowable.class));
     }
 
     @Test
-    public void testNoConstructorsThrowable() {
+    void testNoConstructorsThrowable() {
         // this will not throw an error, but it should return 0, as there are
         // no matching constructors
         assertEquals(0, new ThrowablesTester().testAllThrowable(NoConstructorsThrowable.class));
     }
 
-    @Test(expected = AssertionError.class)
-    public void testIgnoreMessageThrowable() {
-        new ThrowablesTester().testAllThrowable(IgnoreMessageThrowable.class);
+    @Test
+    void testIgnoreMessageThrowable() {
+        ThrowablesTester tester = new ThrowablesTester();
+        assertThatThrownBy(() -> tester.testAllThrowable(IgnoreMessageThrowable.class))
+            .isInstanceOf(AssertionError.class);
     }
 
-    @Test(expected = AssertionError.class)
-    public void testIgnoreCauseThrowable() {
-        new ThrowablesTester().testAllThrowable(IgnoreCauseThrowable.class);
+    @Test
+    void testIgnoreCauseThrowable() {
+        ThrowablesTester tester = new ThrowablesTester();
+        assertThatThrownBy(() -> tester.testAllThrowable(IgnoreCauseThrowable.class))
+            .isInstanceOf(AssertionError.class);
     }
 
-    @Test(expected = AssertionError.class)
-    public void testAlwaysSuppressThrowable() {
-        new ThrowablesTester().testAllThrowable(AlwaysSuppressThrowable.class);
+    @Test
+    void testAlwaysSuppressThrowable() {
+        ThrowablesTester tester = new ThrowablesTester();
+        assertThatThrownBy(() -> tester.testAllThrowable(AlwaysSuppressThrowable.class))
+            .isInstanceOf(AssertionError.class);
     }
 
-    @Test(expected = AssertionError.class)
-    public void testNeverSuppressThrowable() {
-        new ThrowablesTester().testAllThrowable(NeverSuppressThrowable.class);
+    @Test
+    void testNeverSuppressThrowable() {
+        ThrowablesTester tester = new ThrowablesTester();
+        assertThatThrownBy(() -> tester.testAllThrowable(NeverSuppressThrowable.class))
+            .isInstanceOf(AssertionError.class);
     }
 
-    @Test(expected = AssertionError.class)
-    public void testAlwaysWritableThrowable() {
-        new ThrowablesTester().testAllThrowable(AlwaysWritableThrowable.class);
+    @Test
+    void testAlwaysWritableThrowable() {
+        ThrowablesTester tester = new ThrowablesTester();
+        assertThatThrownBy(() -> tester.testAllThrowable(AlwaysWritableThrowable.class))
+            .isInstanceOf(AssertionError.class);
     }
 
-    @Test(expected = AssertionError.class)
-    public void testNeverWritableThrowable() {
-        new ThrowablesTester().testAllThrowable(NeverWritableThrowable.class);
+    @Test
+    void testNeverWritableThrowable() {
+        ThrowablesTester tester = new ThrowablesTester();
+        assertThatThrownBy(() -> tester.testAllThrowable(NeverWritableThrowable.class))
+            .isInstanceOf(AssertionError.class);
     }
 
-    @Test(expected = ConstructionError.class)
-    public void testThrowInstantiationException() {
-        new ThrowablesTester().testAllThrowable(ThrowInstantiationThrowable.class);
+    @Test
+    void testThrowInstantiationException() {
+        ThrowablesTester tester = new ThrowablesTester();
+        assertThatThrownBy(() -> tester.testAllThrowable(ThrowInstantiationThrowable.class))
+            .isInstanceOf(AssertionError.class);
     }
 
     /**
index 4c2237f..9367209 100644 (file)
@@ -3,6 +3,7 @@
  * Common Utils-Test
  * ================================================================================
  * Copyright (C) 2018-2020 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,8 +21,8 @@
 
 package org.onap.policy.common.utils.test.log.logback;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 
 import ch.qos.logback.classic.Level;
 import ch.qos.logback.classic.Logger;
@@ -31,13 +32,13 @@ import java.util.LinkedList;
 import java.util.List;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicInteger;
-import org.junit.After;
-import org.junit.Before;
-import org.junit.BeforeClass;
-import org.junit.Test;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 import org.slf4j.LoggerFactory;
 
-public class ExtractAppenderTest {
+class ExtractAppenderTest {
     private static final String ABC_DIGIT = "abc[0-9]";
     private static final String ABC_DIGIT1 = "abc[1-9]";
     private static final String DEF_DIGIT = "def[0-9]";
@@ -58,13 +59,13 @@ public class ExtractAppenderTest {
 
     private List<Thread> threads;
 
-    @BeforeClass
+    @BeforeAll
     public static void setUpBeforeClass() {
         logger = (Logger) LoggerFactory.getLogger(ExtractAppenderTest.class);
         logger.setLevel(Level.INFO);
     }
 
-    @Before
+    @BeforeEach
     public void setUp() {
         threads = new LinkedList<>();
     }
@@ -72,7 +73,7 @@ public class ExtractAppenderTest {
     /**
      * Tear down all appenders and threads.
      */
-    @After
+    @AfterEach
     public void tearDown() throws Exception {
         logger.detachAndStopAllAppenders();
 
@@ -83,7 +84,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testExtractAppender() {
+    void testExtractAppender() {
         AtomicInteger count = new AtomicInteger(0);
 
         ExtractAppender appender = new ExtractAppender() {
@@ -117,7 +118,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testExtractAppenderStringArray() {
+    void testExtractAppenderStringArray() {
         AtomicInteger count = new AtomicInteger(0);
 
         ExtractAppender appender = new ExtractAppender(ABC_DIGIT, DEF_DIGIT) {
@@ -156,7 +157,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testExtractAppenderQueueStringArray() {
+    void testExtractAppenderQueueStringArray() {
         // no. of matches allowed in the list
         int nallowed = 3;
 
@@ -204,7 +205,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testAppendILoggingEvent_NoPatterns() {
+    void testAppendILoggingEvent_NoPatterns() {
         ExtractAppender appender = makeAppender();
 
         logger.info(HELLO);
@@ -214,7 +215,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testAppendILoggingEvent_Formatted() {
+    void testAppendILoggingEvent_Formatted() {
         ExtractAppender appender = makeAppender();
 
         logger.info("hello {} world{}", "there", "!");
@@ -223,7 +224,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testAppendILoggingEvent_MatchFirstPattern() {
+    void testAppendILoggingEvent_MatchFirstPattern() {
         ExtractAppender appender = makeAppender(ABC_DIGIT, DEF_DIGIT);
 
         logger.info("hello abc1");
@@ -233,7 +234,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testAppendILoggingEvent_MatchLastPattern() {
+    void testAppendILoggingEvent_MatchLastPattern() {
         ExtractAppender appender = makeAppender(ABC_DIGIT, DEF_DIGIT);
 
         logger.info("hello def1");
@@ -243,7 +244,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testAppendILoggingEvent_Group1() {
+    void testAppendILoggingEvent_Group1() {
         ExtractAppender appender = makeAppender("hello (abc)|(xyz)", DEF_DIGIT);
 
         logger.info("hello abc, world!");
@@ -253,7 +254,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testAppendILoggingEvent_Group3() {
+    void testAppendILoggingEvent_Group3() {
         ExtractAppender appender = makeAppender("hello (abc)|(pdq)|(xyz)", DEF_DIGIT);
 
         logger.info("say hello xyz, world!");
@@ -263,7 +264,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testAppendILoggingEvent_NoGroup() {
+    void testAppendILoggingEvent_NoGroup() {
         ExtractAppender appender = makeAppender(HELLO_ABC);
 
         logger.info("say hello abc, world!");
@@ -273,7 +274,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testGetExtracted() {
+    void testGetExtracted() {
         ExtractAppender appender = makeAppender(ABC_DIGIT1);
 
         logger.info(HELLO_ABC1_WORLD);
@@ -289,7 +290,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testClearExtractions() {
+    void testClearExtractions() {
         final ExtractAppender appender = makeAppender(ABC_DIGIT1);
 
         logger.info(HELLO_ABC1_WORLD);
@@ -312,7 +313,7 @@ public class ExtractAppenderTest {
     }
 
     @Test
-    public void testSetPattern() {
+    void testSetPattern() {
         final ExtractAppender appender = makeAppender(ABC_DIGIT1);
 
         logger.info(HELLO_ABC1_WORLD);
@@ -334,7 +335,7 @@ public class ExtractAppenderTest {
      * Launches threads doing everything in parallel to ensure nothing crashes.
      */
     @Test
-    public void test_MultiThreaded() throws Exception {
+    void test_MultiThreaded() throws Exception {
         // when to stop
         long tend = System.currentTimeMillis() + 250;
 
index 3e64edf..14223b9 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 package org.onap.policy.common.utils.time;
 
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertSame;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class PeriodicItemTest {
+class PeriodicItemTest {
     private static final long DELAY_MS = 100L;
     private static final long PERIOD_MS = 200L;
     private static final Object ASSOCIATE = new Object();
@@ -41,7 +42,7 @@ public class PeriodicItemTest {
     /**
      * Sets up objects, including {@link #item}.
      */
-    @Before
+    @BeforeEach
     public void setUp() {
         currentTime = new TestTime();
         count = 0;
@@ -49,18 +50,18 @@ public class PeriodicItemTest {
     }
 
     @Test
-    public void testBumpNextTime() {
+    void testBumpNextTime() {
         assertTrue(item.bumpNextTime());
         assertEquals(currentTime.getMillis() + PERIOD_MS, item.getNextMs());
     }
 
     @Test
-    public void testToString() {
+    void testToString() {
         assertNotNull(item.toString());
     }
 
     @Test
-    public void testPeriodicItem() {
+    void testPeriodicItem() {
         assertSame(ASSOCIATE, item.getAssociate());
         assertNotNull(item.getAction());
         assertEquals(currentTime.getMillis() + DELAY_MS, item.getNextMs());
index 0046f79..d3c00da 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2020 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.time;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class PseudoExecutorTest {
+class PseudoExecutorTest {
     private int invoked;
     private PseudoExecutor executor;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         invoked = 0;
         executor = new PseudoExecutor();
     }
 
     @Test
-    public void test() {
+    void test() {
         assertEquals(0, executor.getQueueLength());
         assertEquals(0, executor.getTasks().size());
         assertTrue(executor.runAll(0));
index 745e989..5969f1f 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 package org.onap.policy.common.utils.time;
 
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.util.Collections;
 import java.util.List;
@@ -32,10 +33,10 @@ import java.util.concurrent.Callable;
 import java.util.concurrent.Future;
 import java.util.concurrent.ScheduledFuture;
 import java.util.concurrent.TimeUnit;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class PseudoScheduledExecutorServiceTest {
+class PseudoScheduledExecutorServiceTest {
     private static final long DELAY_MS = 100L;
     private static final long PERIOD_MS = 200L;
     private static final List<Callable<Object>> EMPTY_CALLABLES = Collections.emptyList();
@@ -48,7 +49,7 @@ public class PseudoScheduledExecutorServiceTest {
     /**
      * Sets up objects, including {@link #svc}.
      */
-    @Before
+    @BeforeEach
     public void setUp() {
         ran = 0;
         called = 0;
@@ -57,7 +58,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testShutdown() {
+    void testShutdown() {
         // submit some tasks
         svc.submit(new MyRun());
         svc.schedule(new MyRun(), 1L, TimeUnit.SECONDS);
@@ -70,7 +71,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testShutdownNow() {
+    void testShutdownNow() {
         // submit some tasks
         svc.submit(new MyRun());
         svc.schedule(new MyRun(), 1L, TimeUnit.SECONDS);
@@ -83,7 +84,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testIsShutdown_testIsTerminated() {
+    void testIsShutdown_testIsTerminated() {
         assertFalse(svc.isShutdown());
         assertFalse(svc.isTerminated());
 
@@ -93,7 +94,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testAwaitTermination() throws InterruptedException {
+    void testAwaitTermination() throws InterruptedException {
         assertFalse(svc.awaitTermination(1L, TimeUnit.SECONDS));
 
         svc.shutdown();
@@ -101,7 +102,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testSubmitCallableOfT() throws Exception {
+    void testSubmitCallableOfT() throws Exception {
         Future<Integer> future = svc.submit(new MyCallable());
         currentTime.runOneTask(0);
 
@@ -113,7 +114,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testSubmitRunnableT() throws Exception {
+    void testSubmitRunnableT() throws Exception {
         Future<Integer> future = svc.submit(new MyRun(), 2);
         currentTime.runOneTask(0);
 
@@ -125,7 +126,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testSubmitRunnable() throws Exception {
+    void testSubmitRunnable() throws Exception {
         assertNotNull(svc.submit(new MyRun()));
         currentTime.runOneTask(0);
 
@@ -136,31 +137,31 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testInvokeAllCollectionOfQextendsCallableOfT() {
+    void testInvokeAllCollectionOfQextendsCallableOfT() {
         assertThatThrownBy(() -> svc.invokeAll(EMPTY_CALLABLES))
                         .isInstanceOf(UnsupportedOperationException.class);
     }
 
     @Test
-    public void testInvokeAllCollectionOfQextendsCallableOfTLongTimeUnit() {
+    void testInvokeAllCollectionOfQextendsCallableOfTLongTimeUnit() {
         assertThatThrownBy(() -> svc.invokeAll(EMPTY_CALLABLES, 1, TimeUnit.MILLISECONDS))
                         .isInstanceOf(UnsupportedOperationException.class);
     }
 
     @Test
-    public void testInvokeAnyCollectionOfQextendsCallableOfT() {
+    void testInvokeAnyCollectionOfQextendsCallableOfT() {
         assertThatThrownBy(() -> svc.invokeAny(EMPTY_CALLABLES))
                         .isInstanceOf(UnsupportedOperationException.class);
     }
 
     @Test
-    public void testInvokeAnyCollectionOfQextendsCallableOfTLongTimeUnit() {
+    void testInvokeAnyCollectionOfQextendsCallableOfTLongTimeUnit() {
         assertThatThrownBy(() -> svc.invokeAny(EMPTY_CALLABLES, 1, TimeUnit.MILLISECONDS))
                         .isInstanceOf(UnsupportedOperationException.class);
     }
 
     @Test
-    public void testExecute() throws InterruptedException {
+    void testExecute() throws InterruptedException {
         svc.execute(new MyRun());
         currentTime.runOneTask(0);
 
@@ -171,7 +172,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testScheduleRunnableLongTimeUnit() throws InterruptedException {
+    void testScheduleRunnableLongTimeUnit() throws InterruptedException {
         assertNotNull(svc.schedule(new MyRun(), DELAY_MS, TimeUnit.MILLISECONDS));
 
         assertEquals(DELAY_MS, oneTaskElapsedTime());
@@ -182,7 +183,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testScheduleCallableOfVLongTimeUnit() throws Exception {
+    void testScheduleCallableOfVLongTimeUnit() throws Exception {
         ScheduledFuture<Integer> future = svc.schedule(new MyCallable(), DELAY_MS, TimeUnit.MILLISECONDS);
 
         assertEquals(DELAY_MS, oneTaskElapsedTime());
@@ -194,7 +195,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testScheduleAtFixedRate() throws InterruptedException {
+    void testScheduleAtFixedRate() throws InterruptedException {
         final ScheduledFuture<?> future =
                         svc.scheduleAtFixedRate(new MyRun(), DELAY_MS, PERIOD_MS, TimeUnit.MILLISECONDS);
 
@@ -218,7 +219,7 @@ public class PseudoScheduledExecutorServiceTest {
     }
 
     @Test
-    public void testScheduleWithFixedDelay() throws InterruptedException {
+    void testScheduleWithFixedDelay() throws InterruptedException {
         final ScheduledFuture<?> future =
                         svc.scheduleWithFixedDelay(new MyRun(), DELAY_MS, PERIOD_MS, TimeUnit.MILLISECONDS);
 
index 248edf6..31156af 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2019, 2021 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.time;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertSame;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.mockito.Mockito.lenient;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
 import java.util.concurrent.Delayed;
 import java.util.concurrent.TimeUnit;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
 
-@RunWith(MockitoJUnitRunner.class)
-public class PseudoScheduledFutureTest {
+@ExtendWith(MockitoExtension.class)
+class PseudoScheduledFutureTest {
     private static final long DELAY_MS = 1000L;
 
     private int count;
@@ -49,9 +51,9 @@ public class PseudoScheduledFutureTest {
     /**
      * Sets up objects, including {@link #future}.
      */
-    @Before
+    @BeforeEach
     public void setUp() {
-        when(work.getDelay()).thenReturn(DELAY_MS);
+        lenient().when(work.getDelay()).thenReturn(DELAY_MS);
 
         count = 0;
         future = new PseudoScheduledFuture<>(() -> ++count, true);
@@ -59,7 +61,7 @@ public class PseudoScheduledFutureTest {
     }
 
     @Test
-    public void testRun() {
+    void testRun() {
         // verify with a periodic task - should execute twice
         count = 0;
         future.run();
@@ -75,7 +77,7 @@ public class PseudoScheduledFutureTest {
     }
 
     @Test
-    public void testPseudoScheduledFutureRunnableTBoolean() throws Exception {
+    void testPseudoScheduledFutureRunnableTBoolean() throws Exception {
         final Integer result = 100;
         future = new PseudoScheduledFuture<>(() -> ++count, result, true);
         assertTrue(future.isPeriodic());
@@ -94,7 +96,7 @@ public class PseudoScheduledFutureTest {
     }
 
     @Test
-    public void testPseudoScheduledFutureCallableOfTBoolean() throws Exception {
+    void testPseudoScheduledFutureCallableOfTBoolean() throws Exception {
         assertTrue(future.isPeriodic());
         future.run();
         future.run();
@@ -111,13 +113,13 @@ public class PseudoScheduledFutureTest {
     }
 
     @Test
-    public void testGetDelay() {
+    void testGetDelay() {
         assertEquals(DELAY_MS, future.getDelay(TimeUnit.MILLISECONDS));
         assertEquals(TimeUnit.MILLISECONDS.toSeconds(DELAY_MS), future.getDelay(TimeUnit.SECONDS));
     }
 
     @Test
-    public void testCompareTo() {
+    void testCompareTo() {
         Delayed delayed = mock(Delayed.class);
         when(delayed.getDelay(TimeUnit.MILLISECONDS)).thenReturn(DELAY_MS + 1);
 
@@ -125,18 +127,18 @@ public class PseudoScheduledFutureTest {
     }
 
     @Test
-    public void testIsPeriodic() {
+    void testIsPeriodic() {
         assertTrue(future.isPeriodic());
         assertFalse(new PseudoScheduledFuture<>(() -> ++count, false).isPeriodic());
     }
 
     @Test
-    public void testGetWorkItem() {
+    void testGetWorkItem() {
         assertSame(work, future.getWorkItem());
     }
 
     @Test
-    public void testSetWorkItem() {
+    void testSetWorkItem() {
         work = mock(WorkItem.class);
         future.setWorkItem(work);
         assertSame(work, future.getWorkItem());
index d7316c1..69e5452 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 package org.onap.policy.common.utils.time;
 
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.util.Date;
 import java.util.TimerTask;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class PseudoTimerTest {
+class PseudoTimerTest {
     private static final long DELAY_MS = 1000L;
     private static final long PERIOD_MS = 2000L;
 
@@ -41,7 +42,7 @@ public class PseudoTimerTest {
     /**
      * Sets up objects, including {@link #timer}.
      */
-    @Before
+    @BeforeEach
     public void setUp() {
         count = 0;
         currentTime = new TestTimeMulti();
@@ -49,7 +50,7 @@ public class PseudoTimerTest {
     }
 
     @Test
-    public void testCancel() {
+    void testCancel() {
         // schedule two tasks
         timer.scheduleAtFixedRate(new MyTask(), DELAY_MS, PERIOD_MS);
         timer.schedule(new MyTask(), DELAY_MS);
@@ -64,13 +65,13 @@ public class PseudoTimerTest {
     }
 
     @Test
-    public void testPurge() {
+    void testPurge() {
         assertEquals(0, timer.purge());
         assertEquals(0, timer.purge());
     }
 
     @Test
-    public void testScheduleTimerTaskLong() throws InterruptedException {
+    void testScheduleTimerTaskLong() throws InterruptedException {
         timer.schedule(new MyTask(), DELAY_MS);
         assertFalse(currentTime.isEmpty());
 
@@ -82,14 +83,14 @@ public class PseudoTimerTest {
     }
 
     @Test
-    public void testScheduleTimerTaskDate() {
+    void testScheduleTimerTaskDate() {
         MyTask task = new MyTask();
         Date curdate = new Date();
         assertThatThrownBy(() -> timer.schedule(task, curdate)).isInstanceOf(UnsupportedOperationException.class);
     }
 
     @Test
-    public void testScheduleTimerTaskLongLong() throws InterruptedException {
+    void testScheduleTimerTaskLongLong() throws InterruptedException {
         timer.schedule(new MyTask(), DELAY_MS, PERIOD_MS);
         assertFalse(currentTime.isEmpty());
 
@@ -105,14 +106,14 @@ public class PseudoTimerTest {
     }
 
     @Test
-    public void testScheduleTimerTaskDateLong() {
+    void testScheduleTimerTaskDateLong() {
         MyTask task = new MyTask();
         Date curdate = new Date();
         assertThatThrownBy(() -> timer.schedule(task, curdate, 1L)).isInstanceOf(UnsupportedOperationException.class);
     }
 
     @Test
-    public void testScheduleAtFixedRateTimerTaskLongLong() throws InterruptedException {
+    void testScheduleAtFixedRateTimerTaskLongLong() throws InterruptedException {
         timer.scheduleAtFixedRate(new MyTask(), DELAY_MS, PERIOD_MS);
         assertFalse(currentTime.isEmpty());
 
@@ -128,7 +129,7 @@ public class PseudoTimerTest {
     }
 
     @Test
-    public void testScheduleAtFixedRateTimerTaskDateLong() {
+    void testScheduleAtFixedRateTimerTaskDateLong() {
         MyTask task = new MyTask();
         Date curdate = new Date();
         assertThatThrownBy(() -> timer.scheduleAtFixedRate(task, curdate, 1L))
index e7bbd01..e35c58f 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.time;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertSame;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.util.concurrent.FutureTask;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class RunnableItemTest {
+class RunnableItemTest {
     private static final long DELAY_MS = 100L;
     private static final Object ASSOCIATE = new Object();
 
@@ -41,7 +42,7 @@ public class RunnableItemTest {
     /**
      * Sets up objects, including {@link #item}.
      */
-    @Before
+    @BeforeEach
     public void setUp() {
         currentTime = new TestTime();
         count = 0;
@@ -49,7 +50,7 @@ public class RunnableItemTest {
     }
 
     @Test
-    public void testWasCancelled() {
+    void testWasCancelled() {
         assertFalse(item.wasCancelled());
 
         FutureTask<Object> future = new FutureTask<>(() -> count++);
@@ -61,13 +62,13 @@ public class RunnableItemTest {
     }
 
     @Test
-    public void testIsAssociatedWith() {
+    void testIsAssociatedWith() {
         assertFalse(item.isAssociatedWith(this));
         assertTrue(item.isAssociatedWith(ASSOCIATE));
     }
 
     @Test
-    public void testFire() {
+    void testFire() {
         item.fire();
         assertEquals(1, count);
 
@@ -78,7 +79,7 @@ public class RunnableItemTest {
     }
 
     @Test
-    public void testRunnableItem_testGetAssociate_testGetAction() {
+    void testRunnableItem_testGetAssociate_testGetAction() {
         assertSame(ASSOCIATE, item.getAssociate());
         assertNotNull(item.getAction());
         assertEquals(currentTime.getMillis() + DELAY_MS, item.getNextMs());
@@ -96,7 +97,7 @@ public class RunnableItemTest {
     }
 
     @Test
-    public void testToString() {
+    void testToString() {
         assertNotNull(item.toString());
     }
 }
index dbd5478..ad8f2fb 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.time;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class SleepItemTest {
+class SleepItemTest {
     private static final int SLEEP_MS = 250;
     private static final long MAX_WAIT_MS = 5000L;
 
@@ -44,7 +45,7 @@ public class SleepItemTest {
     /**
      * Sets up objects, including {@link #item}.
      */
-    @Before
+    @BeforeEach
     public void setUp() {
         currentTime = new TestTime();
         started = new CountDownLatch(1);
@@ -70,7 +71,7 @@ public class SleepItemTest {
     }
 
     @Test
-    public void testInterrupt() throws InterruptedException {
+    void testInterrupt() throws InterruptedException {
         startThread();
 
         item.interrupt();
@@ -80,7 +81,7 @@ public class SleepItemTest {
     }
 
     @Test
-    public void testFire_testAwait() throws InterruptedException {
+    void testFire_testAwait() throws InterruptedException {
         startThread();
 
         // verify that it hasn't finished yet
@@ -95,12 +96,12 @@ public class SleepItemTest {
     }
 
     @Test
-    public void testSleepItem() {
+    void testSleepItem() {
         assertEquals(currentTime.getMillis() + SLEEP_MS, item.getNextMs());
     }
 
     @Test
-    public void testToString() {
+    void testToString() {
         assertNotNull(item.toString());
     }
 
index 8b6501a..f6d1e6a 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2018-2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,11 +23,11 @@ package org.onap.policy.common.utils.time;
 
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.awaitility.Awaitility.await;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.util.concurrent.Callable;
 import java.util.concurrent.CountDownLatch;
@@ -35,23 +36,23 @@ import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicInteger;
 import java.util.concurrent.atomic.AtomicReference;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class TestTimeMultiTest {
+class TestTimeMultiTest {
     private static final long SHORT_WAIT_MS = 100L;
     private static final long DELAY_MS = 500L;
     private static final long MAX_WAIT_MS = 5000L;
 
     private TestTimeMulti multi;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         multi = new TestTimeMulti();
     }
 
     @Test
-    public void testSleep() throws InterruptedException {
+    void testSleep() throws InterruptedException {
         // negative sleep time
         final long tbegin = multi.getMillis();
         MyThread thread = new MyThread(-5);
@@ -81,17 +82,17 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testTestTimeMulti() {
+    void testTestTimeMulti() {
         assertTrue(multi.getMaxWaitMs() > 0);
     }
 
     @Test
-    public void testTestTimeMultiLong() {
+    void testTestTimeMultiLong() {
         assertEquals(200, new TestTimeMulti(200).getMaxWaitMs());
     }
 
     @Test
-    public void testIsEmpty_testQueueLength() throws InterruptedException {
+    void testIsEmpty_testQueueLength() throws InterruptedException {
         assertTrue(multi.isEmpty());
 
         // queue up two items
@@ -114,7 +115,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testDestroy() throws InterruptedException {
+    void testDestroy() throws InterruptedException {
         // this won't interrupt
         multi.enqueue(new WorkItem(multi, DELAY_MS));
 
@@ -143,7 +144,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testRunOneTask() throws InterruptedException {
+    void testRunOneTask() throws InterruptedException {
         // nothing in the queue yet
         assertFalse(multi.runOneTask(0));
 
@@ -163,7 +164,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testWaitFor() throws InterruptedException {
+    void testWaitFor() throws InterruptedException {
         // queue up a couple of items
         multi.enqueue(new WorkItem(multi, DELAY_MS));
         multi.enqueue(new WorkItem(multi, DELAY_MS * 2));
@@ -179,7 +180,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testWaitFor_EmptyQueue() throws InterruptedException {
+    void testWaitFor_EmptyQueue() throws InterruptedException {
         multi = new TestTimeMulti(SHORT_WAIT_MS);
 
         final long realBegin = System.currentTimeMillis();
@@ -192,7 +193,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testWaitUntilCallable() throws InterruptedException {
+    void testWaitUntilCallable() throws InterruptedException {
         multi.enqueue(new WorkItem(multi, DELAY_MS));
         multi.enqueue(new WorkItem(multi, DELAY_MS * 2));
         multi.enqueue(new WorkItem(multi, DELAY_MS * 3));
@@ -209,7 +210,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testWaitUntilCallable_InterruptEx() throws InterruptedException {
+    void testWaitUntilCallable_InterruptEx() throws InterruptedException {
         multi = new TestTimeMulti();
 
         Callable<Boolean> callable = () -> {
@@ -237,7 +238,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testWaitUntilCallable_ConditionThrowsEx() throws InterruptedException {
+    void testWaitUntilCallable_ConditionThrowsEx() throws InterruptedException {
         multi = new TestTimeMulti();
 
         Callable<Boolean> callable = () -> {
@@ -252,7 +253,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testWaitUntilCallable_NeverSatisfied() throws InterruptedException {
+    void testWaitUntilCallable_NeverSatisfied() throws InterruptedException {
         multi = new TestTimeMulti(SHORT_WAIT_MS);
 
         final long realBegin = System.currentTimeMillis();
@@ -262,7 +263,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testWaitUntilLongTimeUnitCallable() throws InterruptedException {
+    void testWaitUntilLongTimeUnitCallable() throws InterruptedException {
         multi.enqueue(new WorkItem(multi, DELAY_MS));
         multi.enqueue(new WorkItem(multi, DELAY_MS * 2));
         multi.enqueue(new WorkItem(multi, DELAY_MS * 3));
@@ -279,7 +280,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testWaitUntilLongTimeUnitCallable_PseudoTimeExpires() throws InterruptedException {
+    void testWaitUntilLongTimeUnitCallable_PseudoTimeExpires() throws InterruptedException {
         multi.enqueue(new WorkItem(multi, DELAY_MS));
         multi.enqueue(new WorkItem(multi, DELAY_MS * 2));
         multi.enqueue(new WorkItem(multi, DELAY_MS * 3));
@@ -291,7 +292,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testRunItem() throws InterruptedException {
+    void testRunItem() throws InterruptedException {
         AtomicBoolean fired = new AtomicBoolean(false);
         multi.enqueue(new MyWorkItem(fired));
 
@@ -305,7 +306,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testRunItem_Rescheduled() throws InterruptedException {
+    void testRunItem_Rescheduled() throws InterruptedException {
         AtomicBoolean fired = new AtomicBoolean(false);
 
         multi.enqueue(new MyWorkItem(fired) {
@@ -326,7 +327,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testRunItem_Canceled() throws InterruptedException {
+    void testRunItem_Canceled() throws InterruptedException {
         AtomicBoolean fired = new AtomicBoolean(false);
 
         multi.enqueue(new MyWorkItem(fired) {
@@ -354,7 +355,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testEnqueue() throws InterruptedException {
+    void testEnqueue() throws InterruptedException {
         CountDownLatch started = new CountDownLatch(1);
         CountDownLatch finished = new CountDownLatch(1);
         AtomicReference<InterruptedException> ex = new AtomicReference<>();
@@ -390,7 +391,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testCancelItems() throws InterruptedException {
+    void testCancelItems() throws InterruptedException {
         AtomicBoolean fired1 = new AtomicBoolean();
         multi.enqueue(new MyWorkItem(fired1));
 
@@ -420,7 +421,7 @@ public class TestTimeMultiTest {
     }
 
     @Test
-    public void testPurgeItems() throws InterruptedException {
+    void testPurgeItems() throws InterruptedException {
         AtomicBoolean fired = new AtomicBoolean();
 
         // queue up two that are canceled, one that is not
index d2cf678..c3b69eb 100644 (file)
@@ -3,6 +3,7 @@
  * Common Utils-Test
  * ================================================================================
  * Copyright (C) 2018-2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package org.onap.policy.common.utils.time;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 public class TestTimeTest {
 
index d2fc8d5..f0229ca 100644 (file)
@@ -3,6 +3,7 @@
  * ONAP
  * ================================================================================
  * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2024 Nordix Foundation
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,17 +23,17 @@ package org.onap.policy.common.utils.time;
 
 import static org.assertj.core.api.Assertions.assertThatCode;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 public class WorkItemTest {
     private TestTime currentTime;
     private WorkItem item;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         currentTime = new TestTime();
         item = new WorkItem(currentTime, 0);