diff options
author | waynedunican <wayne.dunican@est.tech> | 2024-06-27 15:04:03 +0100 |
---|---|---|
committer | waynedunican <wayne.dunican@est.tech> | 2024-07-01 11:51:37 +0100 |
commit | 0c17b552764c5cc876aad4363ebe24dc7c25a952 (patch) | |
tree | 661dfe61618a5b8f3cdf8f10ae775b02863ccbdc /utils/src | |
parent | 61eb8b47308a859a64b9538222445534dda3a34e (diff) |
Convert common to JUnit 5
Review for utils
Issue-ID: POLICY-5043
Change-Id: Ic5f1d30ef6bfb02fd09270127b19180b1f989084
Signed-off-by: waynedunican <wayne.dunican@est.tech>
Diffstat (limited to 'utils/src')
38 files changed, 596 insertions, 538 deletions
diff --git a/utils/src/test/java/org/onap/policy/common/utils/cmd/TestCommandLineArguments.java b/utils/src/test/java/org/onap/policy/common/utils/cmd/TestCommandLineArguments.java index b45f107d..82ae6636 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/cmd/TestCommandLineArguments.java +++ b/utils/src/test/java/org/onap/policy/common/utils/cmd/TestCommandLineArguments.java @@ -1,6 +1,6 @@ /*- * ============LICENSE_START======================================================= - * Copyright (C) 2021, 2023 Nordix Foundation. + * Copyright (C) 2021-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. @@ -23,15 +23,15 @@ package org.onap.policy.common.utils.cmd; import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatCode; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertTrue; -import static org.junit.Assert.fail; +import static org.junit.jupiter.api.Assertions.assertFalse; +import static org.junit.jupiter.api.Assertions.assertTrue; +import static org.junit.jupiter.api.Assertions.fail; import org.apache.commons.cli.Option; import org.apache.commons.cli.Options; -import org.junit.Test; +import org.junit.jupiter.api.Test; -public class TestCommandLineArguments { +class TestCommandLineArguments { private static final String FAKE_HELP_CLASS = "org.onap.policy.HelpClass"; private static final String FAKE_COMPONENT = "fake policy cpm"; private static final String TEST_CONFIG_FILE = "cmdFiles/configuration.json"; @@ -43,19 +43,19 @@ public class TestCommandLineArguments { CommandLineArgumentsHandler testCmd = new CommandLineArgumentsHandler(FAKE_HELP_CLASS, FAKE_COMPONENT); @Test - public void testVersion() throws CommandLineException { + void testVersion() throws CommandLineException { String[] version = {"-v"}; assertThat(testCmd.parse(version)).startsWith("ONAP Version test."); } @Test - public void testHelp() throws CommandLineException { + void testHelp() throws CommandLineException { String[] help = {"-h"}; assertThat(testCmd.parse(help)).startsWith("usage: org.onap.policy.HelpClass [options...]"); } @Test - public void testParse() throws CommandLineException { + void testParse() throws CommandLineException { String[] args = {"-c", TEST_CONFIG_FILE}; testCmd.parse(args); @@ -64,14 +64,14 @@ public class TestCommandLineArguments { } @Test - public void testParse_ShouldThrowExceptionWithInvalidArguments() { + void testParse_ShouldThrowExceptionWithInvalidArguments() { String[] invalidArgs = {"-a"}; assertThatThrownBy(() -> testCmd.parse(invalidArgs)).hasMessage(ERR_MSG_INVALID_ARGS) .hasRootCauseMessage("Unrecognized option: -a"); } @Test - public void testParse_ShouldThrowExceptionWithExtraArguments() { + void testParse_ShouldThrowExceptionWithExtraArguments() { String[] remainingArgs = {"-c", TEST_CONFIG_FILE, "extraArgs"}; String expectedErrorMsg = "too many command line arguments specified: [-c, cmdFiles/configuration.json, extraArgs]"; @@ -79,55 +79,55 @@ public class TestCommandLineArguments { } @Test - public void testParse_ShouldThrowExceptionWhenFileNameNull() { + void testParse_ShouldThrowExceptionWhenFileNameNull() { String[] nullArgs = {"-c", null}; assertThatThrownBy(() -> testCmd.parse(nullArgs)).hasMessage(ERR_MSG_INVALID_ARGS); } @Test - public void testValidate() throws CommandLineException { + void testValidate() throws CommandLineException { String[] validConfigArgs = {"-c", TEST_CONFIG_FILE}; testCmd.parse(validConfigArgs); assertThatCode(() -> testCmd.validate()).doesNotThrowAnyException(); } @Test - public void testValidate_ShouldThrowExceptionWhenConfigFileNotPresent() throws CommandLineException { + void testValidate_ShouldThrowExceptionWhenConfigFileNotPresent() throws CommandLineException { String[] versionArgs = {"-v"}; testCmd.parse(versionArgs); assertValidate(versionArgs, ERR_MSG_POLICY_CONFIG_FILE); } @Test - public void testValidate_ShouldThrowExceptionWhenFileNameEmpty() { + void testValidate_ShouldThrowExceptionWhenFileNameEmpty() { String[] argsOnlyKeyNoValue = {"-c", ""}; assertValidate(argsOnlyKeyNoValue, ERR_MSG_POLICY_CONFIG_FILE); assertFalse(testCmd.checkSetConfigurationFilePath()); } @Test - public void testValidate_ShouldThrowExceptionWhenFileNameEmptySpace() { + void testValidate_ShouldThrowExceptionWhenFileNameEmptySpace() { String[] argsOnlyKeyNoValue = {"-c", " "}; assertValidate(argsOnlyKeyNoValue, ERR_MSG_POLICY_CONFIG_FILE); assertFalse(testCmd.checkSetConfigurationFilePath()); } @Test - public void testValidate_ShouldThrowExceptionWhenFileNameDoesNotExist() { + void testValidate_ShouldThrowExceptionWhenFileNameDoesNotExist() { String[] fileNameNotExistentArgs = {"-c", "someFileName.json"}; assertValidate(fileNameNotExistentArgs, "fake policy cpm configuration file \"someFileName.json\" does not exist"); } @Test - public void testValidate_ShouldThrowExceptionWhenFileNameIsNotFile() { + void testValidate_ShouldThrowExceptionWhenFileNameIsNotFile() { String[] folderAsFileNameArgs = {"-c", "src/test/resources"}; assertValidate(folderAsFileNameArgs, "fake policy cpm configuration file \"src/test/resources\" is not a normal file"); } @Test - public void testAddExtraOptions() throws CommandLineException { + void testAddExtraOptions() throws CommandLineException { Option extra = Option.builder("p").longOpt("property-file") .desc("the full path to the topic property file to use, the property file contains the " + FAKE_COMPONENT + " properties") @@ -149,7 +149,7 @@ public class TestCommandLineArguments { } @Test - public void testNewOptions() throws CommandLineException { + void testNewOptions() throws CommandLineException { Options newOptions = new Options(); newOptions.addOption( Option.builder("a").longOpt("fake-option").desc("the fake property to check command line parse") @@ -163,7 +163,7 @@ public class TestCommandLineArguments { assertTrue(testCmdExtraOpt.getCommandLine().hasOption("a")); - // should raise exception as -c is not present on options; + // should raise exception as -c is not present on options // default options should've been replaced by constructor parameter. String[] argsError = {"-c", "aaaa.json"}; assertThatThrownBy(() -> testCmdExtraOpt.parse(argsError)).hasMessage(ERR_MSG_INVALID_ARGS) diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/CoderExceptionTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/CoderExceptionTest.java index cecc4265..9e82c5d8 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/coder/CoderExceptionTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/coder/CoderExceptionTest.java @@ -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,30 +21,31 @@ package org.onap.policy.common.utils.coder; -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.assertNull; +import static org.junit.jupiter.api.Assertions.assertSame; +import static org.junit.jupiter.api.Assertions.assertTrue; -import org.junit.Test; +import org.junit.jupiter.api.Test; -public class CoderExceptionTest { +class CoderExceptionTest { private static final String STRING_VALUE = "My String"; private static final Throwable CAUSE = new Throwable(); private CoderException exc; @Test - public void testCoderException() { + void testCoderException() { exc = new CoderException(); - assertEquals(null, exc.getMessage()); + assertNull(exc.getMessage()); assertSame(null, exc.getCause()); assertNotNull(exc.toString()); } @Test - public void testCoderExceptionString() { + void testCoderExceptionString() { exc = new CoderException(STRING_VALUE); assertEquals(STRING_VALUE, exc.getMessage()); @@ -52,7 +54,7 @@ public class CoderExceptionTest { } @Test - public void testCoderExceptionThrowable() { + void testCoderExceptionThrowable() { exc = new CoderException(CAUSE); assertEquals(CAUSE.toString(), exc.getMessage()); @@ -61,7 +63,7 @@ public class CoderExceptionTest { } @Test - public void testCoderExceptionStringThrowable() { + void testCoderExceptionStringThrowable() { exc = new CoderException(STRING_VALUE, CAUSE); assertEquals(STRING_VALUE, exc.getMessage()); diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/CoderTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/CoderTest.java index 01821504..fe72292a 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/coder/CoderTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/coder/CoderTest.java @@ -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. @@ -20,18 +21,18 @@ package org.onap.policy.common.utils.coder; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNull; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNull; import java.io.File; import java.io.InputStream; import java.io.OutputStream; import java.io.Reader; import java.io.Writer; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class CoderTest { +class CoderTest { private static final Long LONG = 10L; private static final Integer INTEGER = 10; private static final String INT_TEXT = INTEGER.toString(); @@ -41,13 +42,13 @@ public class CoderTest { private MyCoder coder; - @Before - public void setUp() { + @BeforeEach + void setUp() { coder = new MyCoder(); } @Test - public void testConvert() throws CoderException { + void testConvert() throws CoderException { assertNull(coder.convert(null, String.class)); // same class of object diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/PropertyCoderTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/PropertyCoderTest.java index 86f8a1b1..d45f43b0 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/coder/PropertyCoderTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/coder/PropertyCoderTest.java @@ -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. @@ -20,17 +21,17 @@ package org.onap.policy.common.utils.coder; -import static org.junit.Assert.assertEquals; +import static org.junit.jupiter.api.Assertions.assertEquals; import com.google.gson.annotations.SerializedName; import java.io.Reader; import java.io.StringReader; import java.util.List; import lombok.Getter; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class PropertyCoderTest { +class PropertyCoderTest { private PropertyCoder propertyCoder = null; private static final String AES_ENCRYPTION_KEY = "aes_encryption_key"; @@ -64,13 +65,13 @@ public class PropertyCoderTest { * * @throws Exception if an error occurs */ - @Before + @BeforeEach public void setUp() throws Exception { propertyCoder = new PropertyCoder(); } @Test - public void testPropertyCoder() { + void testPropertyCoder() { MyClass data = propertyCoder.decode(json, AES_ENCRYPTION_KEY, MyClass.class); assertEquals("alpha", data.getPdpRestPass()); assertEquals("hello", data.servers.get(0).pass); @@ -80,7 +81,7 @@ public class PropertyCoderTest { } @Test - public void testPropertyCoderReader() { + void testPropertyCoderReader() { Reader reader = new StringReader(json); MyClass data = propertyCoder.decode(reader, AES_ENCRYPTION_KEY, MyClass.class); assertEquals("alpha", data.getPdpRestPass()); diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderInstantAsMillisTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderInstantAsMillisTest.java index ec977da6..f13ddd9a 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderInstantAsMillisTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderInstantAsMillisTest.java @@ -3,6 +3,7 @@ * ONAP PAP * ================================================================================ * 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,8 +23,8 @@ package org.onap.policy.common.utils.coder; import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotNull; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNotNull; import com.google.gson.JsonElement; import java.io.StringReader; @@ -32,22 +33,22 @@ import java.time.Instant; import java.util.LinkedHashMap; import java.util.Map; 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 StandardCoderInstantAsMillisTest { +class StandardCoderInstantAsMillisTest { private static final long INSTANT_MILLIS = 1583249713500L; private static final String INSTANT_TEXT = String.valueOf(INSTANT_MILLIS); private StandardCoder coder; - @Before + @BeforeEach public void setUp() { coder = new StandardCoderInstantAsMillis(); } @Test - public void testConvert() throws CoderException { + void testConvert() throws CoderException { MyObject obj = makeObject(); @SuppressWarnings("unchecked") @@ -60,7 +61,7 @@ public class StandardCoderInstantAsMillisTest { } @Test - public void testEncodeDecode() throws CoderException { + void testEncodeDecode() throws CoderException { MyObject obj = makeObject(); assertThat(coder.encode(obj, false)).contains(INSTANT_TEXT); assertThat(coder.encode(obj, true)).contains(INSTANT_TEXT); @@ -80,13 +81,13 @@ public class StandardCoderInstantAsMillisTest { } @Test - public void testJson() { + void testJson() { MyObject obj = makeObject(); assertThat(coder.toPrettyJson(obj)).contains(INSTANT_TEXT); } @Test - public void testToJsonTree_testFromJsonJsonElementClassT() throws Exception { + void testToJsonTree_testFromJsonJsonElementClassT() throws Exception { MyMap map = new MyMap(); map.props = new LinkedHashMap<>(); map.props.put("jel keyA", "jel valueA"); @@ -102,7 +103,7 @@ public class StandardCoderInstantAsMillisTest { } @Test - public void testConvertFromDouble() throws Exception { + void testConvertFromDouble() throws Exception { String text = "[listA, {keyA=100}, 200]"; assertEquals(text, coder.decode(text, Object.class).toString()); @@ -111,7 +112,7 @@ public class StandardCoderInstantAsMillisTest { } @Test - public void testToStandard() throws Exception { + void testToStandard() throws Exception { MyObject obj = makeObject(); StandardCoderObject sco = coder.toStandard(obj); assertNotNull(sco.getData()); @@ -122,7 +123,7 @@ public class StandardCoderInstantAsMillisTest { } @Test - public void testFromStandard() throws Exception { + void testFromStandard() throws Exception { MyObject obj = new MyObject(); obj.abc = "pdq"; StandardCoderObject sco = coder.toStandard(obj); diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderObjectTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderObjectTest.java index 1748aed3..c597b7f6 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderObjectTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderObjectTest.java @@ -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. @@ -21,16 +22,16 @@ package org.onap.policy.common.utils.coder; 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.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.Gson; import com.google.gson.JsonElement; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class StandardCoderObjectTest { +class StandardCoderObjectTest { private static final Gson gson = new Gson(); private static final String PROP1 = "abc"; @@ -48,24 +49,24 @@ public class StandardCoderObjectTest { * * @throws Exception if an error occurs */ - @Before + @BeforeEach public void setUp() throws Exception { sco = new StandardCoderObject(gson.fromJson(JSON, JsonElement.class)); } @Test - public void testStandardCoderObject() { + void testStandardCoderObject() { assertNull(new StandardCoderObject().getData()); } @Test - public void testStandardCoderObjectJsonElement() { + void testStandardCoderObjectJsonElement() { assertNotNull(sco.getData()); assertEquals(JSON, gson.toJson(sco.getData())); } @Test - public void testGetString() throws Exception { + void testGetString() throws Exception { // one field assertEquals(VAL1, sco.getString(PROP1)); @@ -93,7 +94,7 @@ public class StandardCoderObjectTest { } @Test - public void testGetFieldFromObject() { + void testGetFieldFromObject() { // not an object assertNull(sco.getFieldFromObject(fromJson("[]"), PROP1)); @@ -105,7 +106,7 @@ public class StandardCoderObjectTest { } @Test - public void testGetItemFromArray() { + void testGetItemFromArray() { // not an array assertNull(sco.getItemFromArray(fromJson("{}"), 0)); diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderTest.java index 33c7331e..b4ac0395 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderTest.java @@ -3,6 +3,7 @@ * ONAP PAP * ================================================================================ * 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. @@ -21,10 +22,10 @@ package org.onap.policy.common.utils.coder; 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.Assert.assertSame; +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.assertSame; import static org.mockito.Mockito.doThrow; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.when; @@ -53,10 +54,10 @@ import java.util.List; import java.util.Map; import java.util.TreeMap; 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 StandardCoderTest { +class StandardCoderTest { private static final String EXPECTED_EXCEPTION = "expected exception"; private static final JsonParseException jpe = new JsonParseException(EXPECTED_EXCEPTION); @@ -64,13 +65,13 @@ public class StandardCoderTest { private StandardCoder coder; - @Before + @BeforeEach public void setUp() { coder = new StandardCoder(); } @Test - public void testConvert() throws CoderException { + void testConvert() throws CoderException { // null source assertNull(coder.convert(null, StandardCoderObject.class)); @@ -99,7 +100,7 @@ public class StandardCoderTest { } @Test - public void testEncodeObject() throws Exception { + void testEncodeObject() throws Exception { List<Integer> arr = Arrays.asList(1100, 1110); assertEquals("[1100,1110]", coder.encode(arr)); @@ -110,7 +111,7 @@ public class StandardCoderTest { } @Test - public void testEncodeObjectBoolean() throws Exception { + void testEncodeObjectBoolean() throws Exception { final List<Integer> arr = Arrays.asList(1100, 1110); /* @@ -136,7 +137,7 @@ public class StandardCoderTest { } @Test - public void testEncodeWriterObject() throws Exception { + void testEncodeWriterObject() throws Exception { List<Integer> arr = Arrays.asList(1200, 1210); StringWriter wtr = new StringWriter(); coder.encode(wtr, arr); @@ -149,7 +150,7 @@ public class StandardCoderTest { } @Test - public void testEncodeOutputStreamObject() throws Exception { + void testEncodeOutputStreamObject() throws Exception { List<Integer> arr = Arrays.asList(1300, 1310); ByteArrayOutputStream stream = new ByteArrayOutputStream(); coder.encode(stream, arr); @@ -171,7 +172,7 @@ public class StandardCoderTest { } @Test - public void testEncodeFileObject() throws Exception { + void testEncodeFileObject() throws Exception { File file = new File(getClass().getResource(StandardCoder.class.getSimpleName() + ".json").getFile() + "X"); file.deleteOnExit(); List<Integer> arr = Arrays.asList(1400, 1410); @@ -195,7 +196,7 @@ public class StandardCoderTest { } @Test - public void testDecodeStringClass() throws Exception { + void testDecodeStringClass() throws Exception { String text = "[2200,2210]"; assertEquals(text, coder.decode(text, JsonElement.class).toString()); @@ -207,7 +208,7 @@ public class StandardCoderTest { } @Test - public void testDecodeReaderClass() throws Exception { + void testDecodeReaderClass() throws Exception { String text = "[2300,2310]"; assertEquals(text, coder.decode(new StringReader(text), JsonElement.class).toString()); @@ -219,7 +220,7 @@ public class StandardCoderTest { } @Test - public void testDecodeInputStreamClass() throws Exception { + void testDecodeInputStreamClass() throws Exception { String text = "[2400,2410]"; assertEquals(text, coder.decode(new ByteArrayInputStream(text.getBytes(StandardCharsets.UTF_8)), JsonElement.class) @@ -236,7 +237,7 @@ public class StandardCoderTest { } @Test - public void testDecodeFileClass() throws Exception { + void testDecodeFileClass() throws Exception { File file = new File(getClass().getResource(StandardCoder.class.getSimpleName() + ".json").getFile()); String text = new String(Files.readAllBytes(file.toPath()), StandardCharsets.UTF_8); assertEquals(text, coder.decode(file, JsonElement.class).toString()); @@ -263,7 +264,7 @@ public class StandardCoderTest { } @Test - public void testToJsonTree_testFromJsonJsonElementClassT() throws Exception { + void testToJsonTree_testFromJsonJsonElementClassT() throws Exception { MyMap map = new MyMap(); map.props = new LinkedHashMap<>(); map.props.put("jel keyA", "jel valueA"); @@ -279,7 +280,7 @@ public class StandardCoderTest { } @Test - public void testConvertFromDouble() throws Exception { + void testConvertFromDouble() throws Exception { String text = "[listA, {keyA=100}, 200]"; assertEquals(text, coder.decode(text, Object.class).toString()); @@ -288,7 +289,7 @@ public class StandardCoderTest { } @Test - public void testToStandard() throws Exception { + void testToStandard() throws Exception { MyObject obj = new MyObject(); obj.abc = "xyz"; StandardCoderObject sco = coder.toStandard(obj); @@ -300,7 +301,7 @@ public class StandardCoderTest { } @Test - public void testFromStandard() throws Exception { + void testFromStandard() throws Exception { MyObject obj = new MyObject(); obj.abc = "pdq"; StandardCoderObject sco = coder.toStandard(obj); @@ -313,7 +314,7 @@ public class StandardCoderTest { } @Test - public void testStandardTypeAdapter() throws Exception { + void testStandardTypeAdapter() throws Exception { String json = "{'abc':'def'}".replace('\'', '"'); StandardCoderObject sco = coder.fromJson(json, StandardCoderObject.class); assertNotNull(sco.getData()); @@ -327,7 +328,7 @@ public class StandardCoderTest { } @Test - public void testMapDouble() throws Exception { + void testMapDouble() throws Exception { MyMap map = new MyMap(); map.props = new HashMap<>(); map.props.put("plainString", "def"); @@ -352,7 +353,7 @@ public class StandardCoderTest { } @Test - public void testListDouble() throws Exception { + void testListDouble() throws Exception { @SuppressWarnings("unchecked") List<Object> list = coder.decode("[10, 20.1, 30]", LinkedList.class); assertEquals("[10, 20.1, 30]", list.toString()); diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardValCoderTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardValCoderTest.java index 2fcdb0dd..45d814a4 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardValCoderTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardValCoderTest.java @@ -1,6 +1,7 @@ /* * ============LICENSE_START======================================================= * 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. @@ -21,11 +22,11 @@ package org.onap.policy.common.utils.coder; 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.assertNotEquals; -import static org.junit.Assert.assertTrue; -import static org.junit.Assert.fail; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertFalse; +import static org.junit.jupiter.api.Assertions.assertNotEquals; +import static org.junit.jupiter.api.Assertions.assertTrue; +import static org.junit.jupiter.api.Assertions.fail; import com.worldturner.medeia.api.ValidationFailedException; import java.io.IOException; @@ -36,10 +37,10 @@ import java.nio.file.Paths; import java.util.List; import lombok.Data; import lombok.NoArgsConstructor; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class StandardValCoderTest { +class StandardValCoderTest { private String jsonSchema; private String validJson; private String missingReqJson; @@ -61,7 +62,7 @@ public class StandardValCoderTest { public List<ValInner> aaCollection; } - @Before + @BeforeEach public void testSetUp() throws Exception { jsonSchema = getJson("src/test/resources/org/onap/policy/common/utils/coder/test.schema.json"); validJson = getJson("src/test/resources/org/onap/policy/common/utils/coder/valid.json"); @@ -70,7 +71,7 @@ public class StandardValCoderTest { } @Test - public void testDecode() throws CoderException { + void testDecode() throws CoderException { StandardValCoder valCoder = new StandardValCoder(jsonSchema, "test-schema"); ValOuter valOuter = valCoder.decode(validJson, ValOuter.class); @@ -110,7 +111,7 @@ public class StandardValCoderTest { } @Test - public void testEncode() throws CoderException { + void testEncode() throws CoderException { StandardValCoder valCoder = new StandardValCoder(jsonSchema, "test-schema"); ValOuter valOuter = valCoder.decode(validJson, ValOuter.class); @@ -127,7 +128,7 @@ public class StandardValCoderTest { } @Test - public void testPretty() throws CoderException { + void testPretty() throws CoderException { StandardValCoder valCoder = new StandardValCoder(jsonSchema, "test-schema"); ValOuter valOuter = valCoder.decode(validJson, ValOuter.class); @@ -144,7 +145,7 @@ public class StandardValCoderTest { } @Test - public void testConformance() { + void testConformance() { StandardValCoder valCoder = new StandardValCoder(jsonSchema, "test-schema"); assertTrue(valCoder.isConformant(validJson)); assertFalse(valCoder.isConformant(missingReqJson)); diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardYamlCoderTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardYamlCoderTest.java index cadeb055..ff3f6efa 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardYamlCoderTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardYamlCoderTest.java @@ -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. @@ -21,8 +22,8 @@ package org.onap.policy.common.utils.coder; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotNull; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.mockito.Mockito.spy; import static org.mockito.Mockito.when; @@ -30,25 +31,25 @@ import java.io.File; import java.io.StringWriter; import java.nio.charset.StandardCharsets; import java.nio.file.Files; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; import org.onap.policy.common.utils.coder.YamlJsonTranslatorTest.Container; -public class StandardYamlCoderTest { +class StandardYamlCoderTest { private static final File YAML_FILE = new File("src/test/resources/org/onap/policy/common/utils/coder/YamlJsonTranslator.yaml"); private StandardYamlCoder coder; private Container cont; - @Before + @BeforeEach public void setUp() throws CoderException { coder = new StandardYamlCoder(); cont = coder.decode(YAML_FILE, Container.class); } @Test - public void testToPrettyJson() throws CoderException { + void testToPrettyJson() throws CoderException { String expected = coder.encode(cont); assertEquals(expected, coder.encode(cont, false)); @@ -67,7 +68,7 @@ public class StandardYamlCoderTest { } @Test - public void testToJsonObject() throws CoderException { + void testToJsonObject() throws CoderException { String yaml = coder.encode(cont); Container cont2 = coder.decode(yaml, Container.class); @@ -75,7 +76,7 @@ public class StandardYamlCoderTest { } @Test - public void testToJsonWriterObject() throws CoderException { + void testToJsonWriterObject() throws CoderException { StringWriter wtr = new StringWriter(); coder.encode(wtr, cont); String yaml = wtr.toString(); @@ -85,25 +86,25 @@ public class StandardYamlCoderTest { } @Test - public void testFromJsonStringClassOfT() throws Exception { + void testFromJsonStringClassOfT() throws Exception { String yaml = new String(Files.readAllBytes(YAML_FILE.toPath()), StandardCharsets.UTF_8); Container cont2 = coder.decode(yaml, Container.class); assertEquals(cont, cont2); } @Test - public void testFromJsonReaderClassOfT() { + void testFromJsonReaderClassOfT() { YamlJsonTranslatorTest.verify(cont); } @Test - public void testFromJsonDoubleToInteger() throws Exception { + void testFromJsonDoubleToInteger() throws Exception { Object value = coder.decode("20", Object.class); assertEquals(Integer.valueOf(20), value); } @Test - public void testStandardTypeAdapter() throws Exception { + void testStandardTypeAdapter() throws Exception { String yaml = "abc: def\n"; StandardCoderObject sco = coder.fromJson(yaml, StandardCoderObject.class); assertNotNull(sco.getData()); diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/YamlJsonTranslatorTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/YamlJsonTranslatorTest.java index c76b775a..563181c0 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/coder/YamlJsonTranslatorTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/coder/YamlJsonTranslatorTest.java @@ -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. @@ -21,10 +22,10 @@ package org.onap.policy.common.utils.coder; 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.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.io.File; import java.io.FileReader; @@ -35,11 +36,11 @@ import java.nio.file.Files; import java.util.List; import java.util.Map; import lombok.EqualsAndHashCode; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; import org.yaml.snakeyaml.error.YAMLException; -public class YamlJsonTranslatorTest { +class YamlJsonTranslatorTest { private static final File YAML_FILE = new File("src/test/resources/org/onap/policy/common/utils/coder/YamlJsonTranslator.yaml"); @@ -51,7 +52,7 @@ public class YamlJsonTranslatorTest { * * @throws IOException if an error occurs */ - @Before + @BeforeEach public void setUp() throws IOException { translator = new YamlJsonTranslator(); @@ -61,7 +62,7 @@ public class YamlJsonTranslatorTest { } @Test - public void testToYamlObject() { + void testToYamlObject() { String yaml = translator.toYaml(cont); Container cont2 = translator.fromYaml(yaml, Container.class); @@ -69,7 +70,7 @@ public class YamlJsonTranslatorTest { } @Test - public void testToYamlWriterObject() throws IOException { + void testToYamlWriterObject() throws IOException { IOException ex = new IOException("expected exception"); // writer that throws an exception when the write() method is invoked @@ -96,14 +97,14 @@ public class YamlJsonTranslatorTest { } @Test - public void testFromYamlStringClassOfT() throws IOException { + void testFromYamlStringClassOfT() throws IOException { String yaml = new String(Files.readAllBytes(YAML_FILE.toPath()), StandardCharsets.UTF_8); Container cont2 = translator.fromYaml(yaml, Container.class); assertEquals(cont, cont2); } @Test - public void testFromYamlReaderClassOfT() { + void testFromYamlReaderClassOfT() { verify(cont); } diff --git a/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityMgrCloserTest.java b/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityMgrCloserTest.java index 17d7f7ff..8bd2452d 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityMgrCloserTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityMgrCloserTest.java @@ -3,7 +3,7 @@ * Common Utils * ================================================================================ * Copyright (C) 2018-2020 AT&T Intellectual Property. All rights reserved. - * Modifications Copyright (C) 2023 Nordix Foundation. + * Modifications 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. @@ -21,21 +21,21 @@ package org.onap.policy.common.utils.jpa; -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.never; import static org.mockito.Mockito.verify; import jakarta.persistence.EntityManager; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class EntityMgrCloserTest { +class EntityMgrCloserTest { private EntityManager mgr; - @Before + @BeforeEach public void setUp() { mgr = mock(EntityManager.class); } @@ -45,7 +45,7 @@ public class EntityMgrCloserTest { * Verifies that the constructor does not do anything extra before being closed. */ @Test - public void testEntityMgrCloser() { + void testEntityMgrCloser() { EntityMgrCloser entityMgrCloser = new EntityMgrCloser(mgr); assertEquals(mgr, entityMgrCloser.getManager()); @@ -59,7 +59,7 @@ public class EntityMgrCloserTest { } @Test - public void testGetManager() { + void testGetManager() { try (EntityMgrCloser c = new EntityMgrCloser(mgr)) { assertEquals(mgr, c.getManager()); } @@ -69,7 +69,7 @@ public class EntityMgrCloserTest { * Verifies that the manager gets closed when close() is invoked. */ @Test - public void testClose() { + void testClose() { EntityMgrCloser entityMgrCloser = new EntityMgrCloser(mgr); entityMgrCloser.close(); @@ -82,7 +82,7 @@ public class EntityMgrCloserTest { * Ensures that the manager gets closed when "try" block exits normally. */ @Test - public void testClose_TryWithoutExcept() { + void testClose_TryWithoutExcept() { try (EntityMgrCloser entityMgrCloser = new EntityMgrCloser(mgr)) { // No need to do anything in the try block } @@ -94,7 +94,7 @@ public class EntityMgrCloserTest { * Ensures that the manager gets closed when "try" block throws an exception. */ @Test - public void testClose_TryWithExcept() { + void testClose_TryWithExcept() { try { try (EntityMgrCloser c = new EntityMgrCloser(mgr)) { throw new Exception("expected exception"); diff --git a/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityMgrFactoryCloserTest.java b/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityMgrFactoryCloserTest.java index 0e8edca2..aed0937c 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityMgrFactoryCloserTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityMgrFactoryCloserTest.java @@ -3,7 +3,7 @@ * Common Utils * ================================================================================ * Copyright (C) 2018-2020 AT&T Intellectual Property. All rights reserved. - * Modifications Copyright (C) 2023 Nordix Foundation. + * Modifications 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. @@ -21,21 +21,21 @@ package org.onap.policy.common.utils.jpa; -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.never; import static org.mockito.Mockito.verify; import jakarta.persistence.EntityManagerFactory; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class EntityMgrFactoryCloserTest { +class EntityMgrFactoryCloserTest { private EntityManagerFactory factory; - @Before + @BeforeEach public void setUp() { factory = mock(EntityManagerFactory.class); } @@ -45,7 +45,7 @@ public class EntityMgrFactoryCloserTest { * Verifies that the constructor does not do anything extra before being closed. */ @Test - public void testEntityMgrFactoryCloser() { + void testEntityMgrFactoryCloser() { EntityMgrFactoryCloser entityMgrFactoryCloser = new EntityMgrFactoryCloser(factory); assertEquals(factory, entityMgrFactoryCloser.getFactory()); @@ -59,7 +59,7 @@ public class EntityMgrFactoryCloserTest { } @Test - public void testgetFactory() { + void testGetFactory() { try (EntityMgrFactoryCloser c = new EntityMgrFactoryCloser(factory)) { assertEquals(factory, c.getFactory()); } @@ -69,7 +69,7 @@ public class EntityMgrFactoryCloserTest { * Verifies that the manager gets closed when close() is invoked. */ @Test - public void testClose() { + void testClose() { EntityMgrFactoryCloser entityMgrFactoryCloser = new EntityMgrFactoryCloser(factory); entityMgrFactoryCloser.close(); @@ -82,7 +82,7 @@ public class EntityMgrFactoryCloserTest { * Ensures that the manager gets closed when "try" block exits normally. */ @Test - public void testClose_TryWithoutExcept() { + void testClose_TryWithoutExcept() { try (EntityMgrFactoryCloser entityMgrFactoryCloser = new EntityMgrFactoryCloser(factory)) { // No need to do anything in the try block } @@ -94,7 +94,7 @@ public class EntityMgrFactoryCloserTest { * Ensures that the manager gets closed when "try" block throws an exception. */ @Test - public void testClose_TryWithExcept() { + void testClose_TryWithExcept() { try { try (EntityMgrFactoryCloser c = new EntityMgrFactoryCloser(factory)) { throw new Exception("expected exception"); diff --git a/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityTransCloserTest.java b/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityTransCloserTest.java index dc63c673..a864c740 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityTransCloserTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/jpa/EntityTransCloserTest.java @@ -3,7 +3,7 @@ * Common Utils * ================================================================================ * Copyright (C) 2018-2021 AT&T Intellectual Property. All rights reserved. - * Modifications Copyright (C) 2023 Nordix Foundation. + * Modifications 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. @@ -21,7 +21,7 @@ package org.onap.policy.common.utils.jpa; -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.never; import static org.mockito.Mockito.times; @@ -29,17 +29,17 @@ import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; import jakarta.persistence.EntityTransaction; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class EntityTransCloserTest { +class EntityTransCloserTest { private EntityTransaction trans; /** * Set up EntityTransaction mock. */ - @Before + @BeforeEach public void setUp() { trans = mock(EntityTransaction.class); @@ -53,7 +53,7 @@ public class EntityTransCloserTest { * being closed. */ @Test - public void testEntityTransCloser() { + void testEntityTransCloser() { EntityTransCloser entityTransCloser = new EntityTransCloser(trans); assertEquals(trans, entityTransCloser.getTransaction()); @@ -71,7 +71,7 @@ public class EntityTransCloserTest { } @Test - public void testGetTransation() { + void testGetTransaction() { try (EntityTransCloser t = new EntityTransCloser(trans)) { assertEquals(trans, t.getTransaction()); } @@ -82,7 +82,7 @@ public class EntityTransCloserTest { * is active. */ @Test - public void testClose_Active() { + void testClose_Active() { EntityTransCloser entityTransCloser = new EntityTransCloser(trans); when(trans.isActive()).thenReturn(true); @@ -99,7 +99,7 @@ public class EntityTransCloserTest { * when and no transaction is active. */ @Test - public void testClose_Inactive() { + void testClose_Inactive() { EntityTransCloser entityTransCloser = new EntityTransCloser(trans); when(trans.isActive()).thenReturn(false); @@ -116,7 +116,7 @@ public class EntityTransCloserTest { * normally and a transaction is active. */ @Test - public void testClose_TryWithoutExcept_Active() { + void testClose_TryWithoutExcept_Active() { when(trans.isActive()).thenReturn(true); try (EntityTransCloser entityTransCloser = new EntityTransCloser(trans)) { @@ -133,7 +133,7 @@ public class EntityTransCloserTest { * "try" block exits normally and no transaction is active. */ @Test - public void testClose_TryWithoutExcept_Inactive() { + void testClose_TryWithoutExcept_Inactive() { when(trans.isActive()).thenReturn(false); try (EntityTransCloser entityTransCloser = new EntityTransCloser(trans)) { @@ -150,7 +150,7 @@ public class EntityTransCloserTest { * an exception and a transaction is active. */ @Test - public void testClose_TryWithExcept_Active() { + void testClose_TryWithExcept_Active() { when(trans.isActive()).thenReturn(true); try { @@ -172,7 +172,7 @@ public class EntityTransCloserTest { * "try" block throws an exception and no transaction is active. */ @Test - public void testClose_TryWithExcept_Inactive() { + void testClose_TryWithExcept_Inactive() { when(trans.isActive()).thenReturn(false); try { @@ -193,7 +193,7 @@ public class EntityTransCloserTest { * Verifies that commit() only commits, and that the subsequent close() does not re-commit. */ @Test - public void testCommit() { + void testCommit() { EntityTransCloser entityTransCloser = new EntityTransCloser(trans); entityTransCloser.commit(); @@ -205,7 +205,7 @@ public class EntityTransCloserTest { // closed, but not re-committed entityTransCloser.close(); - verify(trans, times(1)).commit(); + verify(trans).commit(); } /** @@ -213,7 +213,7 @@ public class EntityTransCloserTest { * back. */ @Test - public void testRollback() { + void testRollback() { EntityTransCloser entityTransCloser = new EntityTransCloser(trans); entityTransCloser.rollback(); diff --git a/utils/src/test/java/org/onap/policy/common/utils/logging/LoggerUtilsTest.java b/utils/src/test/java/org/onap/policy/common/utils/logging/LoggerUtilsTest.java index 79db2093..6c39c210 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/logging/LoggerUtilsTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/logging/LoggerUtilsTest.java @@ -3,6 +3,7 @@ * ONAP Policy * ================================================================================ * 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. @@ -22,20 +23,20 @@ package org.onap.policy.common.utils.logging; -import static org.junit.Assert.assertTrue; +import static org.junit.jupiter.api.Assertions.assertTrue; -import org.junit.Test; -import org.junit.runner.RunWith; -import org.mockito.junit.MockitoJUnitRunner; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.extension.ExtendWith; +import org.mockito.junit.jupiter.MockitoExtension; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -@RunWith(MockitoJUnitRunner.class) -public class LoggerUtilsTest { +@ExtendWith(MockitoExtension.class) +class LoggerUtilsTest { protected static final Logger logger = LoggerFactory.getLogger(LoggerUtilsTest.class); @Test - public void testMarker() { + void testMarker() { assertTrue(logger.isInfoEnabled()); logger.info("line 1"); logger.info(LoggerUtils.METRIC_LOG_MARKER, "line 1 Metric"); diff --git a/utils/src/test/java/org/onap/policy/common/utils/network/NetworkUtilTest.java b/utils/src/test/java/org/onap/policy/common/utils/network/NetworkUtilTest.java index 4019ca79..ccd67fc4 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/network/NetworkUtilTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/network/NetworkUtilTest.java @@ -3,6 +3,7 @@ * policy-utils * ================================================================================ * 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. @@ -21,11 +22,12 @@ package org.onap.policy.common.utils.network; import static org.assertj.core.api.Assertions.assertThat; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertNotEquals; -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.assertInstanceOf; +import static org.junit.jupiter.api.Assertions.assertNotEquals; +import static org.junit.jupiter.api.Assertions.assertNotNull; +import static org.junit.jupiter.api.Assertions.assertTrue; import java.io.IOException; import java.net.InetSocketAddress; @@ -33,17 +35,17 @@ import java.net.ServerSocket; import java.net.Socket; import javax.net.ssl.TrustManager; import javax.net.ssl.X509TrustManager; -import org.junit.Test; +import org.junit.jupiter.api.Test; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -public class NetworkUtilTest { +class NetworkUtilTest { protected static Logger logger = LoggerFactory.getLogger(NetworkUtilTest.class); private static final String LOCALHOST = "localhost"; @Test - public void test() throws InterruptedException, IOException { + void test() throws InterruptedException, IOException { assertNotNull(NetworkUtil.IPV4_WILDCARD_ADDRESS); assertFalse(NetworkUtil.isTcpPortOpen(LOCALHOST, NetworkUtil.allocPort(), 1, 5)); assertNotNull(NetworkUtil.getHostname()); @@ -51,10 +53,10 @@ public class NetworkUtilTest { } @Test - public void testAlwaysTrustManager() throws Exception { + void testAlwaysTrustManager() throws Exception { TrustManager[] mgrarr = NetworkUtil.getAlwaysTrustingManager(); assertEquals(1, mgrarr.length); - assertTrue(mgrarr[0] instanceof X509TrustManager); + assertInstanceOf(X509TrustManager.class, mgrarr[0]); X509TrustManager mgr = (X509TrustManager) mgrarr[0]; assertNotNull(mgr.getAcceptedIssuers()); @@ -66,7 +68,7 @@ public class NetworkUtilTest { } @Test - public void testAllocPort_testAllocPortString__testAllocPortInetSocketAddress() throws Exception { + void testAllocPort_testAllocPortString__testAllocPortInetSocketAddress() throws Exception { // allocate wild-card port int wildCardPort = NetworkUtil.allocPort(); assertNotEquals(0, wildCardPort); @@ -93,7 +95,7 @@ public class NetworkUtilTest { } @Test - public void testGenUniqueName() { + void testGenUniqueName() { String name = NetworkUtil.genUniqueName(LOCALHOST); assertThat(name).isNotBlank().isNotEqualTo(LOCALHOST); @@ -114,7 +116,7 @@ public class NetworkUtilTest { @Override public void run() { - try (Socket server = socket.accept()) { + try (Socket server = socket.accept()) { //NOSONAR // do nothing } catch (IOException e) { diff --git a/utils/src/test/java/org/onap/policy/common/utils/properties/BeanConfiguratorTest.java b/utils/src/test/java/org/onap/policy/common/utils/properties/BeanConfiguratorTest.java index 7da4eccd..45226b28 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/properties/BeanConfiguratorTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/properties/BeanConfiguratorTest.java @@ -3,6 +3,7 @@ * ONAP - Common Modules * ================================================================================ * 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. @@ -20,16 +21,17 @@ package org.onap.policy.common.utils.properties; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertNull; -import static org.junit.Assert.assertSame; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertFalse; +import static org.junit.jupiter.api.Assertions.assertNull; +import static org.junit.jupiter.api.Assertions.assertSame; +import static org.junit.jupiter.api.Assertions.assertThrows; import java.lang.reflect.Field; import java.lang.reflect.Method; import java.util.Properties; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; import org.onap.policy.common.utils.properties.exception.PropertyAccessException; import org.onap.policy.common.utils.properties.exception.PropertyException; import org.onap.policy.common.utils.properties.exception.PropertyInvalidException; @@ -38,7 +40,7 @@ import org.onap.policy.common.utils.properties.exception.PropertyMissingExceptio /** * Test class for PropertyConfiguration. */ -public class BeanConfiguratorTest { +class BeanConfiguratorTest { private static final String EXPECTED_EXCEPTION = "expected exception"; private static final String FALSE_STRING = "false"; private static final String A_VALUE = "a.value"; @@ -72,14 +74,14 @@ public class BeanConfiguratorTest { private BeanConfigurator beancfg; - @Before + @BeforeEach public void setUp() { props = new Properties(); beancfg = new BeanConfigurator(); } @Test - public void testConfigureFromProperties() throws PropertyException { + void testConfigureFromProperties() throws PropertyException { testStringValueNoDefault(); } @@ -93,7 +95,7 @@ public class BeanConfiguratorTest { } @Test - public void testSetAllFields() throws Exception { + void testSetAllFields() throws Exception { /* * Implements an extra interface, just to see that it doesn't cause issues. @@ -159,7 +161,7 @@ public class BeanConfiguratorTest { } @Test - public void testSetAllFields_NoProperties() throws Exception { + void testSetAllFields_NoProperties() throws Exception { class Config { @@ -180,12 +182,12 @@ public class BeanConfiguratorTest { } @Test - public void testSetValueObjectFieldProperties_FieldSet() throws PropertyException { + void testSetValueObjectFieldProperties_FieldSet() throws PropertyException { testStringValueNoDefault(); } @Test - public void testSetValueObjectFieldProperties_NoAnnotation() throws PropertyException { + void testSetValueObjectFieldProperties_NoAnnotation() throws PropertyException { class Config { private String value; @@ -203,8 +205,8 @@ public class BeanConfiguratorTest { assertNull(cfg.value); } - @Test(expected = PropertyAccessException.class) - public void testSetValueObjectFieldProperties_WrongFieldType() throws PropertyException { + @Test + void testSetValueObjectFieldProperties_WrongFieldType() { class Config { // Cannot set a property into an "Exception" field @@ -218,11 +220,12 @@ public class BeanConfiguratorTest { } props.setProperty(THE_VALUE, STRING_VALUE); - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyAccessException.class, () -> beancfg.configureFromProperties( + new Config(), props)); } - @Test(expected = PropertyAccessException.class) - public void testGetSetter_NoSetter() throws PropertyException { + @Test + void testGetSetter_NoSetter() { class Config { @Property(name = THE_VALUE) @@ -230,16 +233,19 @@ public class BeanConfiguratorTest { } props.setProperty(THE_VALUE, STRING_VALUE); - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyAccessException.class, () -> beancfg.configureFromProperties( + new Config(), props)); } - @Test(expected = PropertyMissingException.class) - public void testSetValueObjectMethodFieldPropertiesProperty_NoProperty_NoDefault() throws PropertyException { - beancfg.configureFromProperties(new PlainStringConfig(), props); + @Test + void testSetValueObjectMethodFieldPropertiesProperty_NoProperty_NoDefault() { + assertThrows(PropertyMissingException.class, () -> beancfg.configureFromProperties( + new PlainStringConfig(), props)); + } - @Test(expected = PropertyInvalidException.class) - public void testSetValueObjectMethodFieldPropertiesProperty_IllegalArgEx() throws PropertyException { + @Test + void testSetValueObjectMethodFieldPropertiesProperty_IllegalArgEx() { props.setProperty(THE_VALUE, STRING_VALUE); beancfg = new BeanConfigurator() { @@ -249,11 +255,12 @@ public class BeanConfiguratorTest { } }; - beancfg.configureFromProperties(new PlainStringConfig(), props); + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new PlainStringConfig(), props)); } - @Test(expected = PropertyAccessException.class) - public void testSetValueObjectMethodFieldPropertiesProperty_MethodEx() throws PropertyException { + @Test + void testSetValueObjectMethodFieldPropertiesProperty_MethodEx() { class Config extends PlainStringConfig { @Override @@ -263,11 +270,12 @@ public class BeanConfiguratorTest { } props.setProperty(THE_VALUE, STRING_VALUE); - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyAccessException.class, () -> beancfg.configureFromProperties( + new Config(), props)); } @Test - public void testGetValue() throws PropertyException { + void testGetValue() throws PropertyException { // this class contains all of the supported field types class Config { @@ -413,8 +421,8 @@ public class BeanConfiguratorTest { assertEquals(10001, cfg.primLongValue); } - @Test(expected = PropertyAccessException.class) - public void testGetValue_UnsupportedType() throws PropertyException { + @Test + void testGetValue_UnsupportedType() { class Config { // Cannot set a property into an "Exception" field @@ -428,12 +436,14 @@ public class BeanConfiguratorTest { } props.setProperty(THE_VALUE, STRING_VALUE); - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyAccessException.class, () -> beancfg.configureFromProperties( + new Config(), props)); + } @Test - public void testCheckModifiable_OtherModifiers() throws PropertyException { - // this class contains all of the supported field types + void testCheckModifiable_OtherModifiers() throws PropertyException { + // this class contains all the supported field types class Config { @Property(name = "public") @@ -473,14 +483,16 @@ public class BeanConfiguratorTest { assertEquals("a protected string", cfg.protectedString); } - @Test(expected = PropertyAccessException.class) - public void testCheckModifiable_Static() throws PropertyException { + @Test + void testCheckModifiable_Static() { props.setProperty(THE_VALUE, STRING_VALUE); - beancfg.configureFromProperties(new StaticPropConfig(), props); + assertThrows(PropertyAccessException.class, () -> beancfg.configureFromProperties( + new StaticPropConfig(), props)); + } - @Test(expected = PropertyAccessException.class) - public void testCheckModifiable_Final() throws PropertyException { + @Test + void testCheckModifiable_Final() { class Config { // Cannot set a property into an "final" field @@ -489,22 +501,26 @@ public class BeanConfiguratorTest { } props.setProperty(THE_VALUE, STRING_VALUE); - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyAccessException.class, () -> beancfg.configureFromProperties( + new Config(), props)); + } - @Test(expected = PropertyAccessException.class) - public void testCheckMethod_Static() throws PropertyException { + @Test + void testCheckMethod_Static() { props.setProperty(THE_VALUE, STRING_VALUE); - beancfg.configureFromProperties(new StaticMethodConfig(), props); + assertThrows(PropertyAccessException.class, () -> beancfg.configureFromProperties( + new StaticMethodConfig(), props)); + } @Test - public void testGetStringValue() throws PropertyException { + void testGetStringValue() throws PropertyException { testStringValueNoDefault(); } @Test - public void testGetBooleanValue_NoDefault() throws PropertyException { + void testGetBooleanValue_NoDefault() throws PropertyException { props.setProperty(THE_VALUE, "true"); PlainBooleanConfig cfg = new PlainBooleanConfig(); beancfg.configureFromProperties(cfg, props); @@ -512,8 +528,8 @@ public class BeanConfiguratorTest { assertEquals(true, cfg.value); } - @Test(expected = PropertyInvalidException.class) - public void testGetBooleanValue_InvalidDefault() throws PropertyException { + @Test + void testGetBooleanValue_InvalidDefault() { class Config { @Property(name = THE_VALUE, defaultValue = INVALID_VALUE) @@ -526,11 +542,12 @@ public class BeanConfiguratorTest { } props.setProperty(THE_VALUE, "true"); - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new Config(), props)); } @Test - public void testGetBooleanValue_ValidDefault_True() throws PropertyException { + void testGetBooleanValue_ValidDefault_True() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = "true") @@ -561,7 +578,7 @@ public class BeanConfiguratorTest { } @Test - public void testGetBooleanValue_ValidDefault_False() throws PropertyException { + void testGetBooleanValue_ValidDefault_False() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = FALSE_STRING) @@ -592,7 +609,7 @@ public class BeanConfiguratorTest { } @Test - public void testGetIntegerValue_NoDefault() throws PropertyException { + void testGetIntegerValue_NoDefault() throws PropertyException { class Config { @Property(name = THE_VALUE) @@ -611,8 +628,8 @@ public class BeanConfiguratorTest { assertEquals(200, cfg.value.intValue()); } - @Test(expected = PropertyInvalidException.class) - public void testGetIntegerValue_InvalidDefault() throws PropertyException { + @Test + void testGetIntegerValue_InvalidDefault() { class Config { @Property(name = THE_VALUE, defaultValue = INVALID_VALUE) @@ -625,11 +642,13 @@ public class BeanConfiguratorTest { } props.setProperty(THE_VALUE, NUMBER_STRING); - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new Config(), props)); + } @Test - public void testGetIntegerValue_ValidDefault() throws PropertyException { + void testGetIntegerValue_ValidDefault() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = "201") @@ -654,7 +673,7 @@ public class BeanConfiguratorTest { } @Test - public void testGetLongValue_NoDefault() throws PropertyException { + void testGetLongValue_NoDefault() throws PropertyException { class Config { @Property(name = THE_VALUE) @@ -673,8 +692,8 @@ public class BeanConfiguratorTest { assertEquals(20000L, cfg.value.longValue()); } - @Test(expected = PropertyInvalidException.class) - public void testGetLongValue_InvalidDefault() throws PropertyException { + @Test + void testGetLongValue_InvalidDefault() { class Config { @Property(name = THE_VALUE, defaultValue = INVALID_VALUE) @@ -687,11 +706,13 @@ public class BeanConfiguratorTest { } props.setProperty(THE_VALUE, NUMBER_STRING_LONG); - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new Config(), props)); + } @Test - public void testGetLongValue_ValidDefault() throws PropertyException { + void testGetLongValue_ValidDefault() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = "20001") @@ -716,12 +737,12 @@ public class BeanConfiguratorTest { } @Test - public void testGetPropValue_Prop_NoDefault() throws PropertyException { + void testGetPropValue_Prop_NoDefault() throws PropertyException { testStringValueNoDefault(); } @Test - public void testGetPropValue_Prop_Default() throws PropertyException { + void testGetPropValue_Prop_Default() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = STRING_VALUE_DEFAULT) @@ -742,7 +763,7 @@ public class BeanConfiguratorTest { } @Test - public void testGetPropValue_EmptyProp_EmptyOk() throws PropertyException { + void testGetPropValue_EmptyProp_EmptyOk() throws PropertyException { class Config { @Property(name = THE_VALUE, accept = "empty") @@ -763,7 +784,7 @@ public class BeanConfiguratorTest { } @Test - public void testGetPropValue_NullProp_EmptyOk() throws PropertyException { + void testGetPropValue_NullProp_EmptyOk() throws PropertyException { class Config { @Property(name = THE_VALUE, accept = "empty") @@ -782,7 +803,7 @@ public class BeanConfiguratorTest { } @Test - public void testGetPropValue_EmptyDefault_EmptyOk() throws PropertyException { + void testGetPropValue_EmptyDefault_EmptyOk() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = "", accept = "empty") @@ -801,7 +822,7 @@ public class BeanConfiguratorTest { } @Test - public void testGetPropValue_Default_EmptyOk() throws PropertyException { + void testGetPropValue_Default_EmptyOk() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = STRING_VALUE, accept = "empty") @@ -819,8 +840,8 @@ public class BeanConfiguratorTest { assertEquals(STRING_VALUE, cfg.value); } - @Test(expected = PropertyMissingException.class) - public void testGetPropValue_EmptyDefault_EmptyNotOk() throws PropertyException { + @Test + void testGetPropValue_EmptyDefault_EmptyNotOk() { class Config { @Property(name = THE_VALUE, defaultValue = "") @@ -832,11 +853,12 @@ public class BeanConfiguratorTest { } } - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyMissingException.class, () -> beancfg.configureFromProperties( + new Config(), props)); } @Test - public void testGetPropValue_Default_EmptyNotOk() throws PropertyException { + void testGetPropValue_Default_EmptyNotOk() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = STRING_VALUE, accept = "") @@ -855,16 +877,7 @@ public class BeanConfiguratorTest { } @Test - public void testMakeBoolean_True() throws PropertyException { - props.setProperty(THE_VALUE, "true"); - PlainBooleanConfig cfg = new PlainBooleanConfig(); - beancfg.configureFromProperties(cfg, props); - - assertEquals(true, cfg.value); - } - - @Test - public void testMakeBoolean_False() throws PropertyException { + void testMakeBoolean_False() throws PropertyException { props.setProperty(THE_VALUE, FALSE_STRING); PlainBooleanConfig cfg = new PlainBooleanConfig(); beancfg.configureFromProperties(cfg, props); @@ -872,14 +885,16 @@ public class BeanConfiguratorTest { assertEquals(false, cfg.value); } - @Test(expected = PropertyInvalidException.class) - public void testMakeBoolean_Invalid() throws PropertyException { + @Test + void testMakeBoolean_Invalid() { props.setProperty(THE_VALUE, INVALID_VALUE); - beancfg.configureFromProperties(new PlainBooleanConfig(), props); + + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new PlainBooleanConfig(), props)); } @Test - public void testMakeInteger_Valid() throws PropertyException { + void testMakeInteger_Valid() throws PropertyException { props.setProperty(THE_VALUE, "300"); PlainPrimIntConfig cfg = new PlainPrimIntConfig(); beancfg.configureFromProperties(cfg, props); @@ -887,20 +902,23 @@ public class BeanConfiguratorTest { assertEquals(300, cfg.value); } - @Test(expected = PropertyInvalidException.class) - public void testMakeInteger_Invalid() throws PropertyException { + @Test + void testMakeInteger_Invalid() { props.setProperty(THE_VALUE, INVALID_VALUE); - beancfg.configureFromProperties(new PlainPrimIntConfig(), props); + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new PlainPrimIntConfig(), props)); + } - @Test(expected = PropertyInvalidException.class) - public void testMakeInteger_TooBig() throws PropertyException { + @Test + void testMakeInteger_TooBig() { props.setProperty(THE_VALUE, String.valueOf(Integer.MAX_VALUE + 10L)); - beancfg.configureFromProperties(new PlainPrimIntConfig(), props); + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new PlainPrimIntConfig(), props)); } @Test - public void testMakeLong_Valid() throws PropertyException { + void testMakeLong_Valid() throws PropertyException { props.setProperty(THE_VALUE, "30000"); PlainPrimLongConfig cfg = new PlainPrimLongConfig(); beancfg.configureFromProperties(cfg, props); @@ -908,14 +926,15 @@ public class BeanConfiguratorTest { assertEquals(30000L, cfg.value); } - @Test(expected = PropertyInvalidException.class) - public void testMakeLong_Invalid() throws PropertyException { + @Test + void testMakeLong_Invalid() { props.setProperty(THE_VALUE, INVALID_VALUE); - beancfg.configureFromProperties(new PlainPrimLongConfig(), props); + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new PlainPrimLongConfig(), props)); } @Test - public void testCheckDefaultValue_NotEmpty_Valid() throws PropertyException { + void testCheckDefaultValue_NotEmpty_Valid() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = "700") @@ -933,8 +952,8 @@ public class BeanConfiguratorTest { assertEquals(700L, cfg.value); } - @Test(expected = PropertyInvalidException.class) - public void testCheckDefaultValue_NotEmpty_Invalid() throws PropertyException { + @Test + void testCheckDefaultValue_NotEmpty_Invalid() { class Config { @Property(name = THE_VALUE, defaultValue = INVALID_VALUE) @@ -946,17 +965,18 @@ public class BeanConfiguratorTest { } } - beancfg.configureFromProperties(new Config(), props); + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new Config(), props)); } - @Test(expected = PropertyInvalidException.class) - public void testCheckDefaultValue_Empty_EmptyOk_Invalid() throws PropertyException { - - beancfg.configureFromProperties(new PrimLongDefaultBlankAcceptEmptyConfig(), props); + @Test + void testCheckDefaultValue_Empty_EmptyOk_Invalid() { + assertThrows(PropertyInvalidException.class, () -> beancfg.configureFromProperties( + new PrimLongDefaultBlankAcceptEmptyConfig(), props)); } @Test - public void testIsEmptyOkPropertyString_True() throws PropertyException { + void testIsEmptyOkPropertyString_True() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = "", accept = "empty") @@ -984,14 +1004,15 @@ public class BeanConfiguratorTest { assertEquals(STRING_VALUE, cfg.value); } - @Test(expected = PropertyMissingException.class) - public void testIsEmptyOkPropertyString_False() throws PropertyException { + @Test + void testIsEmptyOkPropertyString_False() { - beancfg.configureFromProperties(new PrimLongDefaultBlankAcceptBlankConfig(), props); + assertThrows(PropertyException.class, () -> beancfg + .configureFromProperties(new PrimLongDefaultBlankAcceptBlankConfig(), props)); } @Test - public void testIsEmptyOkProperty_True() throws PropertyException { + void testIsEmptyOkProperty_True() throws PropertyException { class Config { @Property(name = THE_VALUE, defaultValue = "", accept = "empty") @@ -1009,14 +1030,14 @@ public class BeanConfiguratorTest { assertEquals("", cfg.value); } - @Test(expected = PropertyMissingException.class) - public void testIsEmptyOkProperty_False() throws PropertyException { - - beancfg.configureFromProperties(new PrimLongDefaultBlankAcceptBlankConfig(), props); + @Test + void testIsEmptyOkProperty_False() { + assertThrows(PropertyMissingException.class, () -> beancfg + .configureFromProperties(new PrimLongDefaultBlankAcceptBlankConfig(), props)); } @Test - public void testPutToProperties() throws Exception { + void testPutToProperties() throws Exception { /* * Implements the extra interface, too. @@ -1082,7 +1103,7 @@ public class BeanConfiguratorTest { } @Test - public void testPutProperty() throws Exception { + void testPutProperty() throws Exception { class Config { // no annotation - should not be copied @@ -1134,7 +1155,7 @@ public class BeanConfiguratorTest { } @Test - public void testGetGetter() throws Exception { + void testGetGetter() throws Exception { class Config { // getter method starts with "is" for these @@ -1205,8 +1226,8 @@ public class BeanConfiguratorTest { assertEquals(STRING_VALUE, props.getProperty("string")); } - @Test(expected = PropertyAccessException.class) - public void testGetGetter_NoGetter() throws Exception { + @Test + void testGetGetter_NoGetter() { class Config { @Property(name = THE_VALUE) @@ -1215,11 +1236,12 @@ public class BeanConfiguratorTest { Config cfg = new Config(); cfg.value = STRING_VALUE; - beancfg.addToProperties(cfg, props, "", ""); + assertThrows(PropertyAccessException.class, () -> beancfg + .addToProperties(cfg, props, "", "")); } - @Test(expected = PropertyAccessException.class) - public void testGetGetter_NoGetterForBoolean() throws Exception { + @Test + void testGetGetter_NoGetterForBoolean() { class Config { @Property(name = THE_VALUE) @@ -1228,11 +1250,12 @@ public class BeanConfiguratorTest { Config cfg = new Config(); cfg.value = true; - beancfg.addToProperties(cfg, props, "", ""); + assertThrows(PropertyAccessException.class, () -> beancfg + .addToProperties(cfg, props, "", "")); } - @Test(expected = PropertyAccessException.class) - public void testGetGetter_PrivateGetter() throws Exception { + @Test + void testGetGetter_PrivateGetter() { class Config { @Property(name = THE_VALUE) @@ -1246,11 +1269,12 @@ public class BeanConfiguratorTest { Config cfg = new Config(); cfg.value = STRING_VALUE; - beancfg.addToProperties(cfg, props, "", ""); + assertThrows(PropertyAccessException.class, () -> beancfg + .addToProperties(cfg, props, "", "")); } - @Test(expected = PropertyAccessException.class) - public void testGetGetter_SecurityEx() throws Exception { + @Test + void testGetGetter_SecurityEx() { class Config { @Property(name = THE_VALUE) @@ -1272,11 +1296,12 @@ public class BeanConfiguratorTest { } }; - beancfg.addToProperties(cfg, props, "", ""); + assertThrows(PropertyAccessException.class, () -> beancfg + .addToProperties(cfg, props, "", "")); } - @Test(expected = PropertyAccessException.class) - public void testGetBeanValue_Ex() throws Exception { + @Test + void testGetBeanValue_Ex() { class Config { @@ -1293,8 +1318,8 @@ public class BeanConfiguratorTest { final Config cfg = new Config(); cfg.value = STRING_VALUE; - beancfg.addToProperties(cfg, props, "the", "a"); - + assertThrows(PropertyAccessException.class, () -> beancfg + .addToProperties(cfg, props, "the", "a")); } /** diff --git a/utils/src/test/java/org/onap/policy/common/utils/properties/PropertyObjectUtilsTest.java b/utils/src/test/java/org/onap/policy/common/utils/properties/PropertyObjectUtilsTest.java index 93b30643..6c5918cf 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/properties/PropertyObjectUtilsTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/properties/PropertyObjectUtilsTest.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * 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. @@ -18,7 +19,7 @@ package org.onap.policy.common.utils.properties; -import static org.junit.Assert.assertEquals; +import static org.junit.jupiter.api.Assertions.assertEquals; import java.io.IOException; import java.util.AbstractSet; @@ -29,14 +30,14 @@ import java.util.List; import java.util.Map; import java.util.Properties; import java.util.Set; -import org.junit.Test; +import org.junit.jupiter.api.Test; import org.onap.policy.common.utils.coder.CoderException; import org.onap.policy.common.utils.coder.StandardCoder; -public class PropertyObjectUtilsTest { +class PropertyObjectUtilsTest { @Test - public void testToObject() { + void testToObject() { Map<String, String> map = Map.of("abc", "def", "ghi", "jkl"); Properties props = new Properties(); props.putAll(map); @@ -48,7 +49,7 @@ public class PropertyObjectUtilsTest { // with dotted prefix - other items skipped map = Map.of("pfx.abc", "def", "ghi", "jkl", "pfx.mno", "pqr", "differentpfx.stu", "vwx"); props.clear(); - props.putAll(Map.of("pfx.abc", "def", "ghi", "jkl", "pfx.mno", "pqr", "differentpfx.stu", "vwx")); + props.putAll(map); result = PropertyObjectUtils.toObject(props, "pfx."); map = Map.of("abc", "def", "mno", "pqr"); assertEquals(map, result); @@ -59,7 +60,7 @@ public class PropertyObjectUtilsTest { } @Test - public void testSetProperty() { + void testSetProperty() { // one, two, and three components in the name, the last two with subscripts Map<String, Object> map = Map.of("one", "one.abc", "two.def", "two.ghi", "three.jkl.mno[0]", "three.pqr", "three.jkl.mno[1]", "three.stu"); @@ -79,7 +80,7 @@ public class PropertyObjectUtilsTest { } @Test - public void testGetNode() { + void testGetNode() { Map<String, Object> map = Map.of("abc[0].def", "node.ghi", "abc[0].jkl", "node.mno", "abc[1].def", "node.pqr"); Properties props = new Properties(); props.putAll(map); @@ -98,7 +99,7 @@ public class PropertyObjectUtilsTest { } @Test - public void testExpand() { + void testExpand() { // add subscripts out of order Properties props = makeProperties("abc[2]", "expand.def", "abc[1]", "expand.ghi"); @@ -113,7 +114,7 @@ public class PropertyObjectUtilsTest { } @Test - public void testGetObject() { + void testGetObject() { // first value is primitive, while second is a map Properties props = makeProperties("object.abc", "object.def", "object.abc.ghi", "object.jkl"); @@ -128,7 +129,7 @@ public class PropertyObjectUtilsTest { } @Test - public void testGetArray() { + void testGetArray() { // first value is primitive, while second is an array Properties props = makeProperties("array.abc", "array.def", "array.abc[0].ghi", "array.jkl"); @@ -145,7 +146,7 @@ public class PropertyObjectUtilsTest { @Test @SuppressWarnings("unchecked") - public void testCompressLists() throws IOException, CoderException { + void testCompressLists() throws IOException, CoderException { assertEquals("plain-string", PropertyObjectUtils.compressLists("plain-string").toString()); // @formatter:off diff --git a/utils/src/test/java/org/onap/policy/common/utils/properties/SpecPropertiesTest.java b/utils/src/test/java/org/onap/policy/common/utils/properties/SpecPropertiesTest.java index 0e4216c6..627b79c1 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/properties/SpecPropertiesTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/properties/SpecPropertiesTest.java @@ -3,6 +3,7 @@ * ONAP * ================================================================================ * 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. @@ -20,14 +21,15 @@ package org.onap.policy.common.utils.properties; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNull; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNull; +import static org.junit.jupiter.api.Assertions.assertThrows; import java.util.Properties; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class SpecPropertiesTest { +class SpecPropertiesTest { /** * Property prefix of interest. @@ -82,7 +84,7 @@ public class SpecPropertiesTest { /** * Set up the tests. */ - @Before + @BeforeEach public void setUp() { supportingProps = new Properties(); @@ -96,7 +98,7 @@ public class SpecPropertiesTest { } @Test - public void testSpecPropertiesStringString() { + void testSpecPropertiesStringString() { // no supporting properties props = new SpecProperties(MY_PREFIX, MY_SPEC); @@ -112,7 +114,7 @@ public class SpecPropertiesTest { } @Test - public void testSpecPropertiesStringStringProperties() { + void testSpecPropertiesStringStringProperties() { // use supportingProps as default properties props = new SpecProperties(MY_PREFIX, MY_SPEC, supportingProps); @@ -127,7 +129,7 @@ public class SpecPropertiesTest { } @Test - public void testSpecPropertiesStringStringProperties_EmptyPrefix() { + void testSpecPropertiesStringStringProperties_EmptyPrefix() { supportingProps = new Properties(); supportingProps.setProperty(PROP_NO_PREFIX, VAL_NO_PREFIX); @@ -145,7 +147,7 @@ public class SpecPropertiesTest { } @Test - public void testWithTrailingDot() { + void testWithTrailingDot() { // neither has trailing dot assertEquals(PREFIX_GEN, props.getPrefix()); assertEquals(PREFIX_SPEC, props.getSpecPrefix()); @@ -167,7 +169,7 @@ public class SpecPropertiesTest { } @Test - public void testGetPropertyString() { + void testGetPropertyString() { // the key does contain the prefix assertEquals(VAL_NO_PREFIX, props.getProperty(gen(PROP_NO_PREFIX))); assertNull(props.getProperty(gen(PROP_NO_PREFIX + SUFFIX))); @@ -186,7 +188,7 @@ public class SpecPropertiesTest { } @Test - public void testGetPropertyStringString() { + void testGetPropertyStringString() { // the key does contain the prefix assertEquals(VAL_NO_PREFIX, props.getProperty(gen(PROP_NO_PREFIX), VAL_DEFAULT)); assertEquals(VAL_DEFAULT, props.getProperty(gen(PROP_NO_PREFIX + SUFFIX), VAL_DEFAULT)); @@ -207,14 +209,14 @@ public class SpecPropertiesTest { assertNull(props.getProperty(gen(PROP_UNKNOWN), null)); } - @Test(expected = UnsupportedOperationException.class) - public void testHashCode() { - props.hashCode(); + @Test + void testHashCode() { + assertThrows(UnsupportedOperationException.class, () -> props.hashCode()); } - @Test(expected = UnsupportedOperationException.class) - public void testEquals() { - props.equals(props); + @Test + void testEquals() { + assertThrows(UnsupportedOperationException.class, () -> props.equals(props)); } private String gen(String propnm) { diff --git a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyAccessExceptionTest.java b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyAccessExceptionTest.java index ef2051d8..b6bf0c54 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyAccessExceptionTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyAccessExceptionTest.java @@ -3,6 +3,7 @@ * ONAP Policy Engine - Common Modules * ================================================================================ * 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,12 +21,12 @@ package org.onap.policy.common.utils.properties.exception; -import org.junit.Test; +import org.junit.jupiter.api.Test; /** * Test class for PropertyAccessException. */ -public class PropertyAccessExceptionTest extends SupportBasicPropertyExceptionTester { +class PropertyAccessExceptionTest extends SupportBasicPropertyExceptionTester { /** * Test method for @@ -33,7 +34,7 @@ public class PropertyAccessExceptionTest extends SupportBasicPropertyExceptionTe * (java.lang.String, java.lang.String)}. */ @Test - public void testPropertyAccessExceptionStringField() { + void testPropertyAccessExceptionStringField() { verifyPropertyExceptionStringField_AllPopulated(new PropertyAccessException(PROPERTY, FIELD)); verifyPropertyExceptionStringField_NullProperty(new PropertyAccessException(null, FIELD)); verifyPropertyExceptionStringField_NullField(new PropertyAccessException(PROPERTY, null)); @@ -46,7 +47,7 @@ public class PropertyAccessExceptionTest extends SupportBasicPropertyExceptionTe * (java.lang.String, java.lang.String, java.lang.String)}. */ @Test - public void testPropertyAccessExceptionStringFieldString() { + void testPropertyAccessExceptionStringFieldString() { verifyPropertyExceptionStringFieldString(new PropertyAccessException(PROPERTY, FIELD, MESSAGE)); } @@ -56,7 +57,7 @@ public class PropertyAccessExceptionTest extends SupportBasicPropertyExceptionTe * (java.lang.String, java.lang.String, java.lang.Throwable)}. */ @Test - public void testPropertyAccessExceptionStringFieldThrowable() { + void testPropertyAccessExceptionStringFieldThrowable() { verifyPropertyExceptionStringFieldThrowable(new PropertyAccessException(PROPERTY, FIELD, THROWABLE)); } @@ -66,7 +67,7 @@ public class PropertyAccessExceptionTest extends SupportBasicPropertyExceptionTe * (java.lang.String, java.lang.String, java.lang.String, java.lang.Throwable)}. */ @Test - public void testPropertyAccessExceptionStringFieldStringThrowable() { + void testPropertyAccessExceptionStringFieldStringThrowable() { verifyPropertyExceptionStringFieldStringThrowable( new PropertyAccessException(PROPERTY, FIELD, MESSAGE, THROWABLE)); } diff --git a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyAnnotationExceptionTest.java b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyAnnotationExceptionTest.java index 91879763..aad4491c 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyAnnotationExceptionTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyAnnotationExceptionTest.java @@ -3,6 +3,7 @@ * ONAP Policy Engine - Common Modules * ================================================================================ * 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,12 +21,12 @@ package org.onap.policy.common.utils.properties.exception; -import org.junit.Test; +import org.junit.jupiter.api.Test; /** * Test class for PropertyAnnotationException. */ -public class PropertyAnnotationExceptionTest extends SupportBasicPropertyExceptionTester { +class PropertyAnnotationExceptionTest extends SupportBasicPropertyExceptionTester { /** * Test method for @@ -33,7 +34,7 @@ public class PropertyAnnotationExceptionTest extends SupportBasicPropertyExcepti * (java.lang.String, java.lang.String)}. */ @Test - public void testPropertyExceptionStringField() { + void testPropertyExceptionStringField() { verifyPropertyExceptionStringField_AllPopulated(new PropertyAnnotationException(PROPERTY, FIELD)); verifyPropertyExceptionStringField_NullProperty(new PropertyAnnotationException(null, FIELD)); verifyPropertyExceptionStringField_NullField(new PropertyAnnotationException(PROPERTY, null)); @@ -46,7 +47,7 @@ public class PropertyAnnotationExceptionTest extends SupportBasicPropertyExcepti * (java.lang.String, java.lang.String, java.lang.String)}. */ @Test - public void testPropertyExceptionStringFieldString() { + void testPropertyExceptionStringFieldString() { verifyPropertyExceptionStringFieldString(new PropertyAnnotationException(PROPERTY, FIELD, MESSAGE)); } @@ -56,7 +57,7 @@ public class PropertyAnnotationExceptionTest extends SupportBasicPropertyExcepti * (java.lang.String, java.lang.String, java.lang.Throwable)}. */ @Test - public void testPropertyExceptionStringFieldThrowable() { + void testPropertyExceptionStringFieldThrowable() { verifyPropertyExceptionStringFieldThrowable(new PropertyAnnotationException(PROPERTY, FIELD, THROWABLE)); } @@ -66,7 +67,7 @@ public class PropertyAnnotationExceptionTest extends SupportBasicPropertyExcepti * (java.lang.String, java.lang.String, java.lang.String, java.lang.Throwable)}. */ @Test - public void testPropertyExceptionStringFieldStringThrowable() { + void testPropertyExceptionStringFieldStringThrowable() { verifyPropertyExceptionStringFieldStringThrowable( new PropertyAnnotationException(PROPERTY, FIELD, MESSAGE, THROWABLE)); } diff --git a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyExceptionTest.java b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyExceptionTest.java index 9166749b..6075b4cd 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyExceptionTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyExceptionTest.java @@ -3,6 +3,7 @@ * ONAP Policy Engine - Common Modules * ================================================================================ * 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,12 +21,12 @@ package org.onap.policy.common.utils.properties.exception; -import org.junit.Test; +import org.junit.jupiter.api.Test; /** * Test class for PropertyException. */ -public class PropertyExceptionTest extends SupportBasicPropertyExceptionTester { +class PropertyExceptionTest extends SupportBasicPropertyExceptionTester { /** * Test method for @@ -33,7 +34,7 @@ public class PropertyExceptionTest extends SupportBasicPropertyExceptionTester { * (java.lang.String, java.lang.String)}. */ @Test - public void testPropertyExceptionStringField() { + void testPropertyExceptionStringField() { verifyPropertyExceptionStringField_AllPopulated(new PropertyException(PROPERTY, FIELD)); verifyPropertyExceptionStringField_NullProperty(new PropertyException(null, FIELD)); verifyPropertyExceptionStringField_NullField(new PropertyException(PROPERTY, null)); @@ -46,7 +47,7 @@ public class PropertyExceptionTest extends SupportBasicPropertyExceptionTester { * (java.lang.String, java.lang.String, java.lang.String)}. */ @Test - public void testPropertyExceptionStringFieldString() { + void testPropertyExceptionStringFieldString() { verifyPropertyExceptionStringFieldString(new PropertyException(PROPERTY, FIELD, MESSAGE)); } @@ -56,7 +57,7 @@ public class PropertyExceptionTest extends SupportBasicPropertyExceptionTester { * (java.lang.String, java.lang.String, java.lang.Throwable)}. */ @Test - public void testPropertyExceptionStringFieldThrowable() { + void testPropertyExceptionStringFieldThrowable() { verifyPropertyExceptionStringFieldThrowable(new PropertyException(PROPERTY, FIELD, THROWABLE)); } @@ -66,7 +67,7 @@ public class PropertyExceptionTest extends SupportBasicPropertyExceptionTester { * (java.lang.String, java.lang.String, java.lang.String, java.lang.Throwable)}. */ @Test - public void testPropertyExceptionStringFieldStringThrowable() { + void testPropertyExceptionStringFieldStringThrowable() { verifyPropertyExceptionStringFieldStringThrowable(new PropertyException(PROPERTY, FIELD, MESSAGE, THROWABLE)); } diff --git a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyInvalidExceptionTest.java b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyInvalidExceptionTest.java index 3edd7ff4..3222bbbd 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyInvalidExceptionTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyInvalidExceptionTest.java @@ -3,6 +3,7 @@ * ONAP Policy Engine - Common Modules * ================================================================================ * 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,12 +21,12 @@ package org.onap.policy.common.utils.properties.exception; -import org.junit.Test; +import org.junit.jupiter.api.Test; /** * Test class for PropertyInvalidException. */ -public class PropertyInvalidExceptionTest extends SupportBasicPropertyExceptionTester { +class PropertyInvalidExceptionTest extends SupportBasicPropertyExceptionTester { /** * Test method for @@ -33,7 +34,7 @@ public class PropertyInvalidExceptionTest extends SupportBasicPropertyExceptionT * (java.lang.String, java.lang.String)}. */ @Test - public void testPropertyExceptionStringField() { + void testPropertyExceptionStringField() { verifyPropertyExceptionStringField_AllPopulated(new PropertyInvalidException(PROPERTY, FIELD)); verifyPropertyExceptionStringField_NullProperty(new PropertyInvalidException(null, FIELD)); verifyPropertyExceptionStringField_NullField(new PropertyInvalidException(PROPERTY, null)); @@ -46,7 +47,7 @@ public class PropertyInvalidExceptionTest extends SupportBasicPropertyExceptionT * (java.lang.String, java.lang.String, java.lang.String)}. */ @Test - public void testPropertyExceptionStringFieldString() { + void testPropertyExceptionStringFieldString() { verifyPropertyExceptionStringFieldString(new PropertyInvalidException(PROPERTY, FIELD, MESSAGE)); } @@ -56,7 +57,7 @@ public class PropertyInvalidExceptionTest extends SupportBasicPropertyExceptionT * (java.lang.String, java.lang.String, java.lang.Throwable)}. */ @Test - public void testPropertyExceptionStringFieldThrowable() { + void testPropertyExceptionStringFieldThrowable() { verifyPropertyExceptionStringFieldThrowable(new PropertyInvalidException(PROPERTY, FIELD, THROWABLE)); } @@ -66,7 +67,7 @@ public class PropertyInvalidExceptionTest extends SupportBasicPropertyExceptionT * (java.lang.String, java.lang.String, java.lang.String, java.lang.Throwable)}. */ @Test - public void testPropertyExceptionStringFieldStringThrowable() { + void testPropertyExceptionStringFieldStringThrowable() { verifyPropertyExceptionStringFieldStringThrowable( new PropertyInvalidException(PROPERTY, FIELD, MESSAGE, THROWABLE)); } diff --git a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyMissingExceptionTest.java b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyMissingExceptionTest.java index 948bc18e..3a2a7a91 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyMissingExceptionTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/PropertyMissingExceptionTest.java @@ -3,6 +3,7 @@ * ONAP Policy Engine - Common Modules * ================================================================================ * 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,12 +21,12 @@ package org.onap.policy.common.utils.properties.exception; -import org.junit.Test; +import org.junit.jupiter.api.Test; /** * Test class for PropertyMissingException. */ -public class PropertyMissingExceptionTest extends SupportBasicPropertyExceptionTester { +class PropertyMissingExceptionTest extends SupportBasicPropertyExceptionTester { /** * Test method for @@ -33,7 +34,7 @@ public class PropertyMissingExceptionTest extends SupportBasicPropertyExceptionT * (java.lang.String, java.lang.String)}. */ @Test - public void testPropertyExceptionStringField() { + void testPropertyExceptionStringField() { verifyPropertyExceptionStringField_AllPopulated(new PropertyMissingException(PROPERTY, FIELD)); verifyPropertyExceptionStringField_NullProperty(new PropertyMissingException(null, FIELD)); verifyPropertyExceptionStringField_NullField(new PropertyMissingException(PROPERTY, null)); diff --git a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/SupportBasicPropertyExceptionTester.java b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/SupportBasicPropertyExceptionTester.java index be3f8183..653a4efd 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/properties/exception/SupportBasicPropertyExceptionTester.java +++ b/utils/src/test/java/org/onap/policy/common/utils/properties/exception/SupportBasicPropertyExceptionTester.java @@ -3,6 +3,7 @@ * ONAP Policy Engine - Common Modules * ================================================================================ * 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. @@ -21,8 +22,8 @@ package org.onap.policy.common.utils.properties.exception; import static org.assertj.core.api.Assertions.assertThat; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotNull; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNotNull; /** * Superclass used to test subclasses of {@link PropertyException}. diff --git a/utils/src/test/java/org/onap/policy/common/utils/resources/DirectoryUtilsTest.java b/utils/src/test/java/org/onap/policy/common/utils/resources/DirectoryUtilsTest.java index c12ef9f8..40776f52 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/resources/DirectoryUtilsTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/resources/DirectoryUtilsTest.java @@ -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. @@ -25,12 +26,12 @@ import static org.assertj.core.api.Assertions.assertThat; import java.io.IOException; import java.nio.file.Path; import org.apache.commons.io.FileUtils; -import org.junit.Test; +import org.junit.jupiter.api.Test; -public class DirectoryUtilsTest { +class DirectoryUtilsTest { @Test - public void testCreateTempDirectory() throws IOException { + void testCreateTempDirectory() throws IOException { Path path = DirectoryUtils.createTempDirectory("directoryUtilsTest"); var file = path.toFile(); diff --git a/utils/src/test/java/org/onap/policy/common/utils/resources/ResourceUtilsTest.java b/utils/src/test/java/org/onap/policy/common/utils/resources/ResourceUtilsTest.java index c56409ee..d1e891ca 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/resources/ResourceUtilsTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/resources/ResourceUtilsTest.java @@ -22,11 +22,11 @@ package org.onap.policy.common.utils.resources; -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.io.File; import java.io.FileWriter; @@ -34,16 +34,16 @@ import java.io.IOException; import java.net.MalformedURLException; import java.net.URL; import java.util.Set; -import org.junit.After; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.AfterEach; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; /** * The Class ResourceUtilsTest. * * @author Liam Fallon (liam.fallon@ericsson.com) */ -public class ResourceUtilsTest { +class ResourceUtilsTest { private File tmpDir = null; private File tmpEmptyFile = null; private File tmpUsedFile = null; @@ -63,7 +63,7 @@ public class ResourceUtilsTest { * * @throws IOException Signals that an I/O exception has occurred. */ - @Before + @BeforeEach public void setupResourceUtilsTest() throws IOException { tmpDir = new File(System.getProperty("java.io.tmpdir")); tmpEmptyFile = File.createTempFile(this.getClass().getName(), ".tmp"); @@ -80,7 +80,7 @@ public class ResourceUtilsTest { /** * Clean resource utils test. */ - @After + @AfterEach public void cleanDownResourceUtilsTest() { assertTrue(tmpEmptyFile.delete()); assertTrue(tmpUsedFile.delete()); @@ -90,7 +90,7 @@ public class ResourceUtilsTest { * Test get url resource. */ @Test - public void testgetUrlResource() { + void testgetUrlResource() { URL theUrl = ResourceUtils.getUrlResource(tmpDir.getAbsolutePath()); assertNull(theUrl); @@ -135,7 +135,7 @@ public class ResourceUtilsTest { * Test get local file. */ @Test - public void testGetLocalFile() { + void testGetLocalFile() { URL theUrl = ResourceUtils.getLocalFile(tmpDir.getAbsolutePath()); assertNotNull(theUrl); @@ -183,7 +183,7 @@ public class ResourceUtilsTest { * Test get resource as stream. */ @Test - public void testGetResourceAsStream() throws IOException { + void testGetResourceAsStream() throws IOException { verifyStream(tmpDir.getAbsolutePath()); verifyStream(tmpEmptyFile.getAbsolutePath()); verifyStream(tmpUsedFile.getAbsolutePath()); @@ -209,7 +209,7 @@ public class ResourceUtilsTest { * Test get resource as string. */ @Test - public void testGetResourceAsString() { + void testGetResourceAsString() { String theString = ResourceUtils.getResourceAsString(tmpDir.getAbsolutePath()); assertNotNull(theString); @@ -250,7 +250,7 @@ public class ResourceUtilsTest { } @Test - public void testgetUrl4Resource() { + void testgetUrl4Resource() { URL theUrl = ResourceUtils.getUrl4Resource(tmpDir.getAbsolutePath()); assertNotNull(theUrl); @@ -286,7 +286,7 @@ public class ResourceUtilsTest { } @Test - public void testGetFilePath4Resource() { + void testGetFilePath4Resource() { assertNull(ResourceUtils.getFilePath4Resource(null)); assertEquals("/something/else", ResourceUtils.getFilePath4Resource("/something/else")); assertTrue(ResourceUtils.getFilePath4Resource("xml/example.xml").endsWith("xml/example.xml")); @@ -294,7 +294,7 @@ public class ResourceUtilsTest { } @Test - public void testGetDirectoryContents() throws MalformedURLException { + void testGetDirectoryContents() throws MalformedURLException { assertTrue(ResourceUtils.getDirectoryContents(null).isEmpty()); assertTrue(ResourceUtils.getDirectoryContents("idontexist").isEmpty()); assertTrue(ResourceUtils.getDirectoryContents("logback-test.xml").isEmpty()); diff --git a/utils/src/test/java/org/onap/policy/common/utils/resources/TextFileUtilsTest.java b/utils/src/test/java/org/onap/policy/common/utils/resources/TextFileUtilsTest.java index 91268979..ed3be6a3 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/resources/TextFileUtilsTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/resources/TextFileUtilsTest.java @@ -1,6 +1,6 @@ /*- * ============LICENSE_START======================================================= - * Copyright (C) 2019 Nordix Foundation. + * Copyright (C) 2019-2024 Nordix Foundation. * Modifications Copyright (C) 2020-2021 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); @@ -22,25 +22,25 @@ package org.onap.policy.common.utils.resources; import static org.assertj.core.api.Assertions.assertThat; -import static org.junit.Assert.assertEquals; +import static org.junit.jupiter.api.Assertions.assertEquals; import java.io.File; import java.io.FileInputStream; import java.io.IOException; import org.apache.commons.io.FileUtils; -import org.junit.Test; +import org.junit.jupiter.api.Test; /** * Test text file utilities. * * @author Liam Fallon (liam.fallon@est.tech) */ -public class TextFileUtilsTest { +class TextFileUtilsTest { private static final String FILE_CONTENT = "This is the contents of a text file"; @Test - public void testPutToFile() throws IOException { + void testPutToFile() throws IOException { final File tempTextFile = File.createTempFile("Test", ".txt"); tempTextFile.deleteOnExit(); @@ -56,7 +56,7 @@ public class TextFileUtilsTest { } @Test - public void testPutToFileWithNewPath() throws IOException { + void testPutToFileWithNewPath() throws IOException { String tempDirAndFileName = System.getProperty("java.io.tmpdir") + "/non/existant/path/Test.txt"; FileUtils.forceDeleteOnExit(new File(tempDirAndFileName)); @@ -72,7 +72,7 @@ public class TextFileUtilsTest { } @Test - public void testCreateTempFile() throws IOException { + void testCreateTempFile() throws IOException { var file = TextFileUtils.createTempFile("textFileUtilsTest", ".txt"); file.deleteOnExit(); @@ -80,7 +80,7 @@ public class TextFileUtilsTest { } @Test - public void testSetDefaultPermissions() throws IOException { + void testSetDefaultPermissions() throws IOException { var file = new File("target/tempfile.txt"); file.deleteOnExit(); diff --git a/utils/src/test/java/org/onap/policy/common/utils/security/CryptoUtilsTest.java b/utils/src/test/java/org/onap/policy/common/utils/security/CryptoUtilsTest.java index 625fd1f5..ee338376 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/security/CryptoUtilsTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/security/CryptoUtilsTest.java @@ -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. @@ -20,12 +21,12 @@ package org.onap.policy.common.utils.security; -import static org.junit.Assert.assertEquals; -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.assertNull; +import static org.junit.jupiter.api.Assertions.assertTrue; import java.security.GeneralSecurityException; -import org.junit.Test; +import org.junit.jupiter.api.Test; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -33,7 +34,7 @@ import org.slf4j.LoggerFactory; * Unit test for simple App. */ -public class CryptoUtilsTest { +class CryptoUtilsTest { private static Logger logger = LoggerFactory.getLogger(CryptoUtilsTest.class); private static final String PASS = "HelloWorld"; private static final String SECRET_KEY = "MTIzNDU2Nzg5MDEyMzQ1Ng=="; @@ -42,7 +43,7 @@ public class CryptoUtilsTest { private static final String ENCRYPTED_MSG = "original value : {} encrypted value: {}"; @Test - public void testEncrypt() throws GeneralSecurityException { + void testEncrypt() throws GeneralSecurityException { logger.info("testEncrypt:"); CryptoCoder cryptoUtils = new CryptoUtils(SECRET_KEY); String encryptedValue = cryptoUtils.encrypt(PASS); @@ -55,7 +56,7 @@ public class CryptoUtilsTest { } @Test - public void testDecrypt() throws GeneralSecurityException { + void testDecrypt() throws GeneralSecurityException { logger.info("testDecrypt:"); CryptoCoder cryptoUtils = new CryptoUtils(SECRET_KEY); String decryptedValue = cryptoUtils.decrypt(ENCRYPTED_PASS); @@ -64,7 +65,7 @@ public class CryptoUtilsTest { } @Test - public void testStaticEncrypt() { + void testStaticEncrypt() { logger.info("testStaticEncrypt:"); String encryptedValue = CryptoUtils.encrypt(PASS, SECRET_KEY); logger.info(ENCRYPTED_MSG, PASS, encryptedValue); @@ -75,7 +76,7 @@ public class CryptoUtilsTest { } @Test - public void testStaticDecrypt() { + void testStaticDecrypt() { logger.info("testStaticDecrypt:"); String decryptedValue = CryptoUtils.decrypt(ENCRYPTED_PASS, SECRET_KEY); logger.info(DECRYPTED_MSG, ENCRYPTED_PASS, decryptedValue); @@ -83,7 +84,7 @@ public class CryptoUtilsTest { } @Test - public void testBadInputs() { + void testBadInputs() { String badKey = CryptoUtils.encrypt(PASS, "test"); assertEquals(PASS, badKey); @@ -104,7 +105,7 @@ public class CryptoUtilsTest { } @Test - public void testAll() { + void testAll() { logger.info("testAll:"); String encryptedValue = CryptoUtils.encrypt(PASS, SECRET_KEY); logger.info(ENCRYPTED_MSG, PASS, encryptedValue); diff --git a/utils/src/test/java/org/onap/policy/common/utils/services/FeatureApiUtilsTest.java b/utils/src/test/java/org/onap/policy/common/utils/services/FeatureApiUtilsTest.java index d111962e..3de93779 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/services/FeatureApiUtilsTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/services/FeatureApiUtilsTest.java @@ -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,18 +21,18 @@ package org.onap.policy.common.utils.services; -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.Arrays; import java.util.LinkedList; import java.util.List; import java.util.function.Predicate; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class FeatureApiUtilsTest { +class FeatureApiUtilsTest { private static final String HANDLED = "handled"; private MyPred pred; @@ -41,7 +42,7 @@ public class FeatureApiUtilsTest { /** * Initializes fields. */ - @Before + @BeforeEach public void setUp() { tried = new LinkedList<>(); errors = new LinkedList<>(); @@ -49,7 +50,7 @@ public class FeatureApiUtilsTest { } @Test - public void testApplyFeatureTrue() { + void testApplyFeatureTrue() { assertTrue(FeatureApiUtils.apply(Arrays.asList("exceptT0", "falseT1", HANDLED, "falseT2", HANDLED), pred, (str, ex) -> errors.add(str))); @@ -58,7 +59,7 @@ public class FeatureApiUtilsTest { } @Test - public void testApplyFeatureFalse() { + void testApplyFeatureFalse() { List<String> lst = Arrays.asList("falseF1", "exceptF2", "falseF3"); assertFalse(FeatureApiUtils.apply(lst, pred, (str, ex) -> errors.add(str))); diff --git a/utils/src/test/java/org/onap/policy/common/utils/services/OrderedServiceImplTest.java b/utils/src/test/java/org/onap/policy/common/utils/services/OrderedServiceImplTest.java index 39c8a2be..1508009f 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/services/OrderedServiceImplTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/services/OrderedServiceImplTest.java @@ -3,6 +3,7 @@ * utils * ================================================================================ * 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,14 +21,14 @@ package org.onap.policy.common.utils.services; -import static org.junit.Assert.assertEquals; +import static org.junit.jupiter.api.Assertions.assertEquals; import java.util.List; -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 OrderedServiceImplTest { +class OrderedServiceImplTest { private static final int HIGH_PRIORITY_NUM = -1000; private static final int LOW_PRIORITY_NUM = 1000; @@ -38,7 +39,7 @@ public class OrderedServiceImplTest { /** * Saves the original state of the ordered service list to restore after each test. */ - @BeforeClass + @BeforeAll public static void setup() { List<GenericService> implementers = GenericService.providers.getList(); highPrioService = implementers.get(0); @@ -48,7 +49,7 @@ public class OrderedServiceImplTest { /** * Restores original state after each test. */ - @Before + @BeforeEach public void resetOrder() { highPrioService.setSequenceNumber(HIGH_PRIORITY_NUM); lowPrioService.setSequenceNumber(LOW_PRIORITY_NUM); @@ -58,7 +59,7 @@ public class OrderedServiceImplTest { * Tests obtaining a list of service implementers. */ @Test - public void getListTest() { + void getListTest() { List<GenericService> implementers = GenericService.providers.getList(); assertEquals(2, implementers.size()); @@ -74,7 +75,7 @@ public class OrderedServiceImplTest { * with the new order. */ @Test - public void rebuildListInvertedPriorityTest() { + void rebuildListInvertedPriorityTest() { List<GenericService> implementers = GenericService.providers.getList(); assertEquals(2, implementers.size()); @@ -104,7 +105,7 @@ public class OrderedServiceImplTest { * if the priorities are equivalent. */ @Test - public void rebuildListEqualPriorityTest() { + void rebuildListEqualPriorityTest() { List<GenericService> implementers = GenericService.providers.getList(); assertEquals(2, implementers.size()); diff --git a/utils/src/test/java/org/onap/policy/common/utils/services/RegistryTest.java b/utils/src/test/java/org/onap/policy/common/utils/services/RegistryTest.java index f4b95089..08799027 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/services/RegistryTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/services/RegistryTest.java @@ -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. @@ -23,16 +24,17 @@ package org.onap.policy.common.utils.services; import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException; import static org.assertj.core.api.Assertions.assertThatIllegalStateException; 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.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.assertNull; +import static org.junit.jupiter.api.Assertions.assertSame; +import static org.junit.jupiter.api.Assertions.assertTrue; -import org.junit.AfterClass; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.AfterAll; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -public class RegistryTest { +class RegistryTest { private static final String UNKNOWN = "unknown"; private static final String NAME_STR = "name-string"; private static final String NAME_OBJ = "name-object"; @@ -45,7 +47,7 @@ public class RegistryTest { /** * Set up. */ - @Before + @BeforeEach public void setUp() { Registry.newRegistry(); @@ -57,7 +59,7 @@ public class RegistryTest { /** * Ensure the registry is left empty when done. */ - @AfterClass + @AfterAll public static void tearDownAfterClass() { Registry.newRegistry(); } @@ -66,7 +68,7 @@ public class RegistryTest { * Sunny day scenario is tested by other tests, so we focus on exceptions here. */ @Test - public void testRegister_Ex() { + void testRegister_Ex() { assertThatIllegalStateException().isThrownBy(() -> Registry.register(NAME_STR, DATA_STR)); assertThatIllegalArgumentException().isThrownBy(() -> Registry.register(null, DATA_STR)); @@ -74,7 +76,7 @@ public class RegistryTest { } @Test - public void testRegisterOrReplace() { + void testRegisterOrReplace() { // should be able to replace Registry.registerOrReplace(NAME_STR, DATA_STR); Registry.registerOrReplace(NAME_STR, DATA_STR); @@ -90,16 +92,16 @@ public class RegistryTest { } @Test - public void testUnregister() { + void testUnregister() { assertTrue(Registry.unregister(NAME_STR)); - assertEquals(null, Registry.getOrDefault(NAME_STR, String.class, null)); + assertNull(Registry.getOrDefault(NAME_STR, String.class, null)); assertFalse(Registry.unregister(NAME_STR)); } @Test - public void testGet() { + void testGet() { assertSame(DATA_STR, Registry.get(NAME_STR, String.class)); assertSame(DATA_OBJ, Registry.get(NAME_OBJ, Object.class)); assertSame(DATA_INT, Registry.get(NAME_INT, Integer.class)); @@ -112,24 +114,24 @@ public class RegistryTest { } @Test - public void testGetOrDefault() { + void testGetOrDefault() { assertSame(DATA_STR, Registry.getOrDefault(NAME_STR, String.class, null)); assertSame(DATA_OBJ, Registry.getOrDefault(NAME_OBJ, Object.class, "xyz")); assertSame(DATA_INT, Registry.getOrDefault(NAME_INT, Integer.class, 10)); - assertEquals(null, Registry.getOrDefault(UNKNOWN, String.class, null)); + assertNull(Registry.getOrDefault(UNKNOWN, String.class, null)); assertEquals("abc", Registry.getOrDefault(UNKNOWN, String.class, "abc")); assertEquals(Integer.valueOf(11), Registry.getOrDefault(UNKNOWN, Integer.class, 11)); } @Test - public void testNewRegistry() { + void testNewRegistry() { assertSame(DATA_STR, Registry.get(NAME_STR, String.class)); Registry.newRegistry(); // should not exist - assertEquals(null, Registry.getOrDefault(NAME_STR, String.class, null)); + assertNull(Registry.getOrDefault(NAME_STR, String.class, null)); // should be able to register it again now Registry.register(NAME_STR, DATA_STR); diff --git a/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerContainerTest.java b/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerContainerTest.java index e57901ac..1bae46cb 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerContainerTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerContainerTest.java @@ -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,19 +21,19 @@ package org.onap.policy.common.utils.services; -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 static org.mockito.Mockito.mock; import static org.mockito.Mockito.never; import static org.mockito.Mockito.verify; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; import org.onap.policy.common.capabilities.Startable; import org.onap.policy.common.utils.services.ServiceManager.RunnableWithEx; -public class ServiceManagerContainerTest { +class ServiceManagerContainerTest { private static final String MY_NAME = "my-name"; private static final String MY_ACTION = "my-action"; private static final String MY_OBJECT = "my-object"; @@ -44,7 +45,7 @@ public class ServiceManagerContainerTest { /** * Set up. */ - @Before + @BeforeEach public void setUp() { starter = mock(RunnableWithEx.class); stopper = mock(RunnableWithEx.class); @@ -54,7 +55,7 @@ public class ServiceManagerContainerTest { } @Test - public void testServiceManagerContainer() throws Exception { + void testServiceManagerContainer() throws Exception { // use no-arg constructor cont = new MyCont(); assertEquals("service manager", cont.getName()); @@ -64,7 +65,7 @@ public class ServiceManagerContainerTest { } @Test - public void test() throws Exception { + void test() throws Exception { assertEquals(MY_NAME, cont.getName()); assertFalse(cont.isAlive()); @@ -87,7 +88,7 @@ public class ServiceManagerContainerTest { } @Test - public void testShutdown() throws Exception { + void testShutdown() throws Exception { cont.start(); cont.shutdown(); assertFalse(cont.isAlive()); diff --git a/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerExceptionTest.java b/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerExceptionTest.java index 5fe321e8..8d6c8a28 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerExceptionTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerExceptionTest.java @@ -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,32 +21,32 @@ package org.onap.policy.common.utils.services; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotNull; -import static org.junit.Assert.assertNull; -import static org.junit.Assert.assertSame; +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.assertSame; -import org.junit.Test; +import org.junit.jupiter.api.Test; -public class ServiceManagerExceptionTest { +class ServiceManagerExceptionTest { private ServiceManagerException sme; @Test - public void testServiceManagerException() { + void testServiceManagerException() { sme = new ServiceManagerException(); assertNull(sme.getMessage()); assertNull(sme.getCause()); } @Test - public void testServiceManagerExceptionString() { + void testServiceManagerExceptionString() { sme = new ServiceManagerException("hello"); assertEquals("hello", sme.getMessage()); assertNull(sme.getCause()); } @Test - public void testServiceManagerExceptionThrowable() { + void testServiceManagerExceptionThrowable() { Throwable thrown = new Throwable("expected exception"); sme = new ServiceManagerException(thrown); assertNotNull(sme.getMessage()); @@ -53,7 +54,7 @@ public class ServiceManagerExceptionTest { } @Test - public void testServiceManagerExceptionStringThrowable() { + void testServiceManagerExceptionStringThrowable() { Throwable thrown = new Throwable("another expected exception"); sme = new ServiceManagerException("world", thrown); assertEquals("world", sme.getMessage()); diff --git a/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerTest.java b/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerTest.java index 83b2629c..dc555584 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/services/ServiceManagerTest.java @@ -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. @@ -22,9 +23,9 @@ package org.onap.policy.common.utils.services; import static org.assertj.core.api.Assertions.assertThatIllegalStateException; 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 static org.mockito.Mockito.mock; import static org.mockito.Mockito.never; import static org.mockito.Mockito.verify; @@ -32,12 +33,12 @@ import static org.mockito.Mockito.when; import java.util.Arrays; import java.util.LinkedList; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; import org.onap.policy.common.capabilities.Startable; import org.onap.policy.common.utils.services.ServiceManager.RunnableWithEx; -public class ServiceManagerTest { +class ServiceManagerTest { private static final String MY_NAME = "my-name"; private static final String ALREADY_RUNNING = MY_NAME + " is already running"; private static final String EXPECTED_EXCEPTION = "expected exception"; @@ -47,23 +48,23 @@ public class ServiceManagerTest { /** * Initializes {@link #svcmgr}. */ - @Before + @BeforeEach public void setUp() { svcmgr = new ServiceManager(MY_NAME); } @Test - public void testServiceName() { + void testServiceName() { assertEquals("service manager", new ServiceManager().getName()); } @Test - public void testGetName() { + void testGetName() { assertEquals(MY_NAME, svcmgr.getName()); } @Test - public void testAddAction() throws Exception { + void testAddAction() throws Exception { RunnableWithEx start1 = mock(RunnableWithEx.class); RunnableWithEx stop1 = mock(RunnableWithEx.class); svcmgr.addAction("first action", start1, stop1); @@ -90,7 +91,7 @@ public class ServiceManagerTest { } @Test - public void testAddStartable() { + void testAddStartable() { Startable start1 = mock(Startable.class); svcmgr.addService("first startable", start1); @@ -115,7 +116,7 @@ public class ServiceManagerTest { } @Test - public void testStart() { + void testStart() { Startable start1 = mock(Startable.class); svcmgr.addService("test start", start1); @@ -136,7 +137,7 @@ public class ServiceManagerTest { } @Test - public void testStart_Ex() { + void testStart_Ex() { Startable start1 = mock(Startable.class); svcmgr.addService("test start ex", start1); @@ -173,7 +174,7 @@ public class ServiceManagerTest { } @Test - public void testStart_RewindEx() { + void testStart_RewindEx() { Startable start1 = mock(Startable.class); svcmgr.addService("test start rewind", start1); @@ -201,7 +202,7 @@ public class ServiceManagerTest { } @Test - public void testStop() { + void testStop() { Startable start1 = mock(Startable.class); svcmgr.addService("first stop", start1); @@ -221,7 +222,7 @@ public class ServiceManagerTest { } @Test - public void testStop_Ex() throws Exception { + void testStop_Ex() throws Exception { RunnableWithEx start1 = mock(RunnableWithEx.class); RunnableWithEx stop1 = mock(RunnableWithEx.class); svcmgr.addAction("first stop ex", start1, stop1); @@ -245,7 +246,7 @@ public class ServiceManagerTest { } @Test - public void testShutdown() { + void testShutdown() { Startable start1 = mock(Startable.class); svcmgr.addService("first stop", start1); @@ -265,7 +266,7 @@ public class ServiceManagerTest { } @Test - public void testRewind() { + void testRewind() { RunnableWithEx starter = mock(RunnableWithEx.class); LinkedList<String> lst = new LinkedList<>(); diff --git a/utils/src/test/java/org/onap/policy/common/utils/time/CurrentTimeTest.java b/utils/src/test/java/org/onap/policy/common/utils/time/CurrentTimeTest.java index 499372ab..7b267581 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/time/CurrentTimeTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/time/CurrentTimeTest.java @@ -3,6 +3,7 @@ * Common Utils * ================================================================================ * 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. @@ -20,14 +21,14 @@ package org.onap.policy.common.utils.time; -import static org.junit.Assert.assertTrue; +import static org.junit.jupiter.api.Assertions.assertTrue; -import org.junit.Test; +import org.junit.jupiter.api.Test; -public class CurrentTimeTest { +class CurrentTimeTest { @Test - public void testGetMillis() { + void testGetMillis() { long tcur = System.currentTimeMillis(); long tval = new CurrentTime().getMillis(); long tval2 = new CurrentTime().getMillis(); @@ -38,7 +39,7 @@ public class CurrentTimeTest { } @Test - public void testGetDate() { + void testGetDate() { long tcur = System.currentTimeMillis(); long tval = new CurrentTime().getDate().getTime(); long tval2 = new CurrentTime().getDate().getTime(); @@ -49,7 +50,7 @@ public class CurrentTimeTest { } @Test - public void testSleep() throws Exception { + void testSleep() throws Exception { long tcur = System.currentTimeMillis(); new CurrentTime().sleep(10); long tend = System.currentTimeMillis(); diff --git a/utils/src/test/java/org/onap/policy/common/utils/validation/AssertionsTest.java b/utils/src/test/java/org/onap/policy/common/utils/validation/AssertionsTest.java index 05d4c5c3..b633425f 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/validation/AssertionsTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/validation/AssertionsTest.java @@ -1,7 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2016-2018 Ericsson. All rights reserved. - * Modifications Copyright (C) 2019 Nordix Foundation. + * Modifications Copyright (C) 2019-2024 Nordix Foundation. * Modifications Copyright (C) 2019 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); @@ -23,17 +23,17 @@ package org.onap.policy.common.utils.validation; import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNull; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNull; -import org.junit.Test; +import org.junit.jupiter.api.Test; /** * The Class ResourceUtilsTest. * * @author Liam Fallon (liam.fallon@ericsson.com) */ -public class AssertionsTest { +class AssertionsTest { private static final String HELLO = "Hello"; private static final String IT_IS_OK = "it is OK"; private static final String IT_IS_NULL = "it is null"; @@ -41,7 +41,7 @@ public class AssertionsTest { private static final String IT_IS_FALSE = "it is false"; @Test - public void testAssertions() { + void testAssertions() { Assertions.argumentNotFalse(true, IT_IS_TRUE); assertThatIllegalArgumentException().isThrownBy(() -> Assertions.argumentNotFalse(false, IT_IS_FALSE)) diff --git a/utils/src/test/java/org/onap/policy/common/utils/validation/TestParameterValidationUtils.java b/utils/src/test/java/org/onap/policy/common/utils/validation/TestParameterValidationUtils.java index 7d46553b..a85e8ec8 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/validation/TestParameterValidationUtils.java +++ b/utils/src/test/java/org/onap/policy/common/utils/validation/TestParameterValidationUtils.java @@ -2,6 +2,7 @@ * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. * Modifications 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. @@ -21,27 +22,27 @@ package org.onap.policy.common.utils.validation; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertTrue; +import static org.junit.jupiter.api.Assertions.assertFalse; +import static org.junit.jupiter.api.Assertions.assertTrue; -import org.junit.Test; +import org.junit.jupiter.api.Test; /** * Class to perform unit test of ParameterValidationUtils. * * @author Ram Krishna Verma (ram.krishna.verma@ericsson.com) */ -public class TestParameterValidationUtils { +class TestParameterValidationUtils { @Test - public void testValidateStringParameter() { + void testValidateStringParameter() { assertTrue(ParameterValidationUtils.validateStringParameter("Policy")); assertFalse(ParameterValidationUtils.validateStringParameter(null)); assertFalse(ParameterValidationUtils.validateStringParameter("")); } @Test - public void testValidateIntParameter() { + void testValidateIntParameter() { assertTrue(ParameterValidationUtils.validateIntParameter(5555)); assertTrue(ParameterValidationUtils.validateIntParameter(Integer.valueOf(7777))); assertFalse(ParameterValidationUtils.validateIntParameter(0)); @@ -49,7 +50,7 @@ public class TestParameterValidationUtils { } @Test - public void testValidateLongParameter() { + void testValidateLongParameter() { assertTrue(ParameterValidationUtils.validateLongParameter(5555L)); assertTrue(ParameterValidationUtils.validateLongParameter(Long.valueOf(7777L))); assertFalse(ParameterValidationUtils.validateLongParameter(0L)); diff --git a/utils/src/test/java/org/onap/policy/common/utils/validation/VersionTest.java b/utils/src/test/java/org/onap/policy/common/utils/validation/VersionTest.java index 4673233e..d7dc44fb 100644 --- a/utils/src/test/java/org/onap/policy/common/utils/validation/VersionTest.java +++ b/utils/src/test/java/org/onap/policy/common/utils/validation/VersionTest.java @@ -3,7 +3,7 @@ * ONAP PAP * ================================================================================ * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved. - * Modifications Copyright (C) 2019 Nordix Foundation. + * Modifications Copyright (C) 2019-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. @@ -21,15 +21,15 @@ package org.onap.policy.common.utils.validation; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotEquals; -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.assertNotEquals; +import static org.junit.jupiter.api.Assertions.assertNull; +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 VersionTest { +class VersionTest { private static final String TYPE = "my-type"; private static final String NAME = "my-name"; @@ -39,20 +39,20 @@ public class VersionTest { private Version vers; - @Before + @BeforeEach public void setUp() { vers = new Version(MAJOR, MINOR, PATCH); } @Test - public void testHashCode() { + void testHashCode() { int hash = vers.hashCode(); int hash2 = new Version(MAJOR, MINOR, PATCH + 1).hashCode(); assertNotEquals(hash, hash2); } @Test - public void testMakeVersion() { + void testMakeVersion() { assertEquals("9.8.7", Version.makeVersion(TYPE, NAME, "9.8.7").toString()); assertEquals("9.0.0", Version.makeVersion(TYPE, NAME, "9").toString()); @@ -65,14 +65,14 @@ public class VersionTest { } @Test - public void testNewVersion() { + void testNewVersion() { vers = vers.newVersion(); assertEquals("11.0.0", vers.toString()); } @Test - public void testEquals() { - assertNotEquals(vers, null); + void testEquals() { + assertNotEquals(null, vers); assertNotEquals(vers, new Object()); assertEquals(vers, vers); @@ -85,7 +85,7 @@ public class VersionTest { } @Test - public void testCompareTo() { + void testCompareTo() { vers = new Version(101, 201, 301); // equals case @@ -111,32 +111,32 @@ public class VersionTest { } @Test - public void testToString() { + void testToString() { assertEquals("10.2.3", vers.toString()); } @Test - public void testGetMajor() { + void testGetMajor() { assertEquals(MAJOR, vers.getMajor()); } @Test - public void testGetMinor() { + void testGetMinor() { assertEquals(MINOR, vers.getMinor()); } @Test - public void testGetPatch() { + void testGetPatch() { assertEquals(PATCH, vers.getPatch()); } @Test - public void testVersionIntIntInt() { + void testVersionIntIntInt() { assertEquals("5.6.7", new Version(5, 6, 7).toString()); } @Test - public void testVersion() { + void testVersion() { assertEquals("0.0.0", new Version().toString()); } } |