diff options
author | 2019-08-22 14:21:14 -0400 | |
---|---|---|
committer | 2019-08-22 17:07:37 -0400 | |
commit | 962dd9714c6d35cacdec0ee8e601431aa349001a (patch) | |
tree | 03ce05c9c64680f0c44fa22734294292a1a12064 /POLICY-SDK-APP/src/main/java | |
parent | e78f07ad32ffb52ed5d3e02295ed5282841870f3 (diff) |
Remove powermock to increase coverage
Removed powermock from the junit tests of these classes to increase
reported sonar coverage:
RESTfulPAPEngine
BrmsGateway
Typically added override methods to the classes under test, to replace
the overrides originally provided by powermock.
Also needed to add code to RESTfulPAPEngineTest to clear the XACML
properties before the test ran. Modified to use the existing
XACMLProperties.reloadProperties() method.
Also modified PolicyNotificationMailTest to use a host/port that
has no listener so that the test runs faster.
Fixed some newly introduced sonar issues.
Change-Id: I65e36b01e9506987032eb21baac808ed3dfd4f47
Issue-ID: POLICY-1937
Signed-off-by: Jim Hahn <jrh3@att.com>
Diffstat (limited to 'POLICY-SDK-APP/src/main/java')
-rw-r--r-- | POLICY-SDK-APP/src/main/java/org/onap/policy/admin/RESTfulPAPEngine.java | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/RESTfulPAPEngine.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/RESTfulPAPEngine.java index a500c1dbe..fc47179c1 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/RESTfulPAPEngine.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/RESTfulPAPEngine.java @@ -36,7 +36,9 @@ import java.io.InputStream; import java.io.OutputStream; import java.io.UnsupportedEncodingException; import java.net.HttpURLConnection; +import java.net.MalformedURLException; import java.net.URL; +import java.net.URLConnection; import java.net.URLEncoder; import java.nio.charset.StandardCharsets; import java.util.Arrays; @@ -379,12 +381,10 @@ public class RESTfulPAPEngine extends StdPDPItemSetChangeNotifier implements PAP } } - URL url = new URL(fullURL); - // // Open up the connection // - connection = (HttpURLConnection) url.openConnection(); + connection = (HttpURLConnection) makeConnection(fullURL); // // Setup our method and headers // @@ -527,4 +527,10 @@ public class RESTfulPAPEngine extends StdPDPItemSetChangeNotifier implements PAP LOGGER.info("JSON response from PAP: " + json); return json; } + + // these may be overridden by junit tests + + protected URLConnection makeConnection(String fullURL) throws IOException { + return new URL(fullURL).openConnection(); + } } |