aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authork.kedron <k.kedron@partner.samsung.com>2019-07-23 15:10:54 +0200
committerTomasz Golabek <tomasz.golabek@nokia.com>2019-07-24 06:28:16 +0000
commit8b6d4ee2ffbfb11a1f9d494b2de403aab112cf93 (patch)
treeae3f73aa34afbfe2f68c2b11d1cc4daff3829ad4
parent6692cde32bc3e588cfb6bf30fd99e21a903e0444 (diff)
Refactor ConfigurationRepositoryTest to remove the powermock.
Refactor the ConfigurationRepositoryTest to doesn't affect to other tests. Removed powermock. Issue-ID: SDC-2473 Signed-off-by: Krystian Kedron <k.kedron@partner.samsung.com> Change-Id: Iad67cdce365fef89da8b56499d86139d6927da07
-rwxr-xr-xcommon/onap-common-configuration-management/onap-configuration-management-core/pom.xml7
-rw-r--r--common/onap-common-configuration-management/onap-configuration-management-core/src/test/java/org/onap/config/impl/ConfigurationRepositoryTest.java19
2 files changed, 8 insertions, 18 deletions
diff --git a/common/onap-common-configuration-management/onap-configuration-management-core/pom.xml b/common/onap-common-configuration-management/onap-configuration-management-core/pom.xml
index 5de500163c..78c8df3ce7 100755
--- a/common/onap-common-configuration-management/onap-configuration-management-core/pom.xml
+++ b/common/onap-common-configuration-management/onap-configuration-management-core/pom.xml
@@ -105,14 +105,7 @@
<artifactId>logback-classic</artifactId>
<scope>test</scope>
</dependency>
- <dependency>
- <groupId>org.powermock</groupId>
- <artifactId>powermock-module-junit4</artifactId>
- <version>2.0.2</version>
- <scope>test</scope>
- </dependency>
</dependencies>
-
<build>
<plugins>
<plugin>
diff --git a/common/onap-common-configuration-management/onap-configuration-management-core/src/test/java/org/onap/config/impl/ConfigurationRepositoryTest.java b/common/onap-common-configuration-management/onap-configuration-management-core/src/test/java/org/onap/config/impl/ConfigurationRepositoryTest.java
index c7cc96cfbe..d604d1e757 100644
--- a/common/onap-common-configuration-management/onap-configuration-management-core/src/test/java/org/onap/config/impl/ConfigurationRepositoryTest.java
+++ b/common/onap-common-configuration-management/onap-configuration-management-core/src/test/java/org/onap/config/impl/ConfigurationRepositoryTest.java
@@ -22,6 +22,7 @@
package org.onap.config.impl;
import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue;
import java.util.Set;
@@ -30,16 +31,12 @@ import org.apache.commons.configuration2.BaseConfiguration;
import org.apache.commons.configuration2.Configuration;
import org.junit.Before;
import org.junit.Test;
-import org.junit.runner.RunWith;
import org.onap.config.Constants;
-import org.powermock.core.classloader.annotations.PrepareForTest;
-import org.powermock.modules.junit4.PowerMockRunner;
-@RunWith(PowerMockRunner.class)
-@PrepareForTest(ConfigurationRepository.class)
public class ConfigurationRepositoryTest {
private static final String[] EMPTY_ARRAY_OF_STRING = new String[0];
+ private static final String TEST_NAME_SPACE = "testNameSpace";
private ConfigurationRepository repository;
@@ -62,11 +59,11 @@ public class ConfigurationRepositoryTest {
final Set<String> namespaces = repository.getNamespaces();
// then
- assertEquals(1, tenants.size());
- assertEquals(1, namespaces.size());
+ assertTrue(tenants.size() > 0);
+ assertTrue(namespaces.size() > 0);
- assertEquals(Constants.DEFAULT_TENANT, tenants.toArray(EMPTY_ARRAY_OF_STRING)[0]);
- assertEquals(Constants.DEFAULT_NAMESPACE, namespaces.toArray(EMPTY_ARRAY_OF_STRING)[0]);
+ assertNotNull(tenants.toArray(EMPTY_ARRAY_OF_STRING)[0]);
+ assertNotNull(namespaces.toArray(EMPTY_ARRAY_OF_STRING)[0]);
}
@Test
@@ -76,9 +73,9 @@ public class ConfigurationRepositoryTest {
// when
repository.populateConfiguration(Constants.DEFAULT_TENANT + Constants.KEY_ELEMENTS_DELIMITER
- + Constants.DEFAULT_NAMESPACE, inputConfig);
+ + TEST_NAME_SPACE, inputConfig);
final Configuration outputConfig =
- repository.getConfigurationFor(Constants.DEFAULT_TENANT, Constants.DEFAULT_NAMESPACE);
+ repository.getConfigurationFor(Constants.DEFAULT_TENANT, TEST_NAME_SPACE);
// then
assertEquals(inputConfig, outputConfig);