diff options
author | Jim Hahn <jrh3@att.com> | 2019-04-15 14:37:46 -0400 |
---|---|---|
committer | Jim Hahn <jrh3@att.com> | 2019-04-15 17:29:44 -0400 |
commit | 8714a151ff87751b54c362c8a9248995e665e896 (patch) | |
tree | c7becba4a7d3d77aae47ae2ff41d62314efb4186 /main/src/test/java | |
parent | 7d8509acbf1a6ce1c0dc8bbd87e0a793420cafd4 (diff) |
Fix sonar issues in PAP
Moved methods into nested class of PdpModifyRequestMap. Note: they
will have to be moved back out when the broadcast capability is
added.
Removed extra "continue" from Publisher.
Rather than remove unused parameters and cause significant clutter
to the code, chose to use the parameters in a logger call for RequestImpl.
Removed unnecessary cast from UpdateReq.
Use version constant in PapRestControllerV1.
Log exception in PdpGroupQueryControllerV1.
Log exception in PdpGroupStateChangeControllerV1.
Shortened functional in PdpGroupDeleteProvider.
Removed unnecessary type parameters from PdpGroupDeployProvider.
Removed unneeded parameter and exception from ProviderBase.
Extracted common string from Main.
Shorted functionals in PapActivator.
Changed uses of getCanonicalName() to getName(); the former only has
limited use, while the latter is typically what is wanted and is
actually required whenever forClass(name) is used.
Fix junit tests broken by rename of ToscaPolicy version from
"typeVersion" to "type_version".
Change-Id: Ia00e1b3541c9e25b196a951e45681f67aa7f1bfe
Issue-ID: POLICY-1542
Signed-off-by: Jim Hahn <jrh3@att.com>
Diffstat (limited to 'main/src/test/java')
-rw-r--r-- | main/src/test/java/org/onap/policy/pap/main/parameters/CommonTestData.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/main/src/test/java/org/onap/policy/pap/main/parameters/CommonTestData.java b/main/src/test/java/org/onap/policy/pap/main/parameters/CommonTestData.java index e7ffce07..10e500fb 100644 --- a/main/src/test/java/org/onap/policy/pap/main/parameters/CommonTestData.java +++ b/main/src/test/java/org/onap/policy/pap/main/parameters/CommonTestData.java @@ -170,7 +170,7 @@ public class CommonTestData { public Map<String, Object> getPolicyModelsProviderParametersMap() { final Map<String, Object> map = new TreeMap<>(); map.put("name", PolicyModelsProviderParameters.class.getSimpleName()); - map.put("implementation", DatabasePolicyModelsProviderImpl.class.getCanonicalName()); + map.put("implementation", DatabasePolicyModelsProviderImpl.class.getName()); map.put("databaseDriver", "org.h2.Driver"); map.put("databaseUrl", "jdbc:h2:mem:testdb"); map.put("databaseUser", "policy"); |