From bdf514f934e76c1ae506bd95d0166547b6e0a385 Mon Sep 17 00:00:00 2001
From: wbars <wbs@kereval.com>
Date: Tue, 1 Sep 2020 16:44:50 +0200
Subject: [PATCH] IUAINFRA-52 | Fix sonar

---
 .../AudienceSecretRetrieverForSoapui.java     | 27 ++++++++++++++++---
 1 file changed, 24 insertions(+), 3 deletions(-)

diff --git a/audience-retriever/src/main/java/net/ihe/gazelle/app/audienceretriever/adapter/AudienceSecretRetrieverForSoapui.java b/audience-retriever/src/main/java/net/ihe/gazelle/app/audienceretriever/adapter/AudienceSecretRetrieverForSoapui.java
index 3570a14..3252c5b 100644
--- a/audience-retriever/src/main/java/net/ihe/gazelle/app/audienceretriever/adapter/AudienceSecretRetrieverForSoapui.java
+++ b/audience-retriever/src/main/java/net/ihe/gazelle/app/audienceretriever/adapter/AudienceSecretRetrieverForSoapui.java
@@ -8,22 +8,40 @@ import java.io.FileInputStream;
 import java.io.IOException;
 import java.util.Properties;
 
+/**
+ * {@link AudienceSecretRetriever} used by SoapUI project to retrieve secrets.
+ */
 public class AudienceSecretRetrieverForSoapui implements AudienceSecretRetriever {
 
     private static final GazelleLogger LOGGER = GazelleLoggerFactory.getInstance().getLogger(AudienceSecretRetrieverForSoapui.class);
 
     private String propertiesFile = "/opt/simulators/audience.properties";
 
+    /**
+     * Default constructor for the class.
+     */
     public AudienceSecretRetrieverForSoapui() {
+        //Empty Constructor
     }
 
+    /**
+     * Constructor allowing to configure the properties file path.
+     *
+     * @param propertiesFile path to the properties file.
+     */
     public AudienceSecretRetrieverForSoapui(String propertiesFile) {
         this.propertiesFile = propertiesFile;
     }
 
-    public static Properties readPropertiesFile(String fileName) {
+    /**
+     * Read property file as {@link Properties}.
+     *
+     * @param filePath path to hte properties file.
+     * @return the {@link Properties} defined by the file.
+     */
+    private static Properties readPropertiesFile(String filePath) {
         Properties prop = null;
-        try (FileInputStream fis = new FileInputStream(fileName)) {
+        try (FileInputStream fis = new FileInputStream(filePath)) {
             prop = new Properties();
             prop.load(fis);
         } catch (IOException e) {
@@ -32,9 +50,12 @@ public class AudienceSecretRetrieverForSoapui implements AudienceSecretRetriever
         return prop;
     }
 
+    /**
+     * {@inheritDoc}
+     */
     @Override
     public String retrieveSecretForAudience(String audience) {
         Properties prop = readPropertiesFile(propertiesFile);
-        return prop.getProperty(audience);
+        return prop != null ? prop.getProperty(audience) : null;
     }
 }
-- 
GitLab