From: aravind.est <aravindhan.a@est.tech>
Date: Wed, 12 Mar 2025 15:17:54 +0000 (+0000)
Subject: Fix sonar security and maintainability issues
X-Git-Tag: 0.3.0~7
X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=commitdiff_plain;h=0e1dbe1101f1cb324e5ad1b8d607b04d6ec900c6;p=nonrtric%2Fplt%2Frappmanager.git

Fix sonar security and maintainability issues

One sonar security and 4 maintainability issues are fixed.

Issue-ID: NONRTRIC-1057
Change-Id: Ifbd7b4463a5e005fd5e481e24d7c6bee5c9cd740
Signed-off-by: aravind.est <aravindhan.a@est.tech>
---

diff --git a/rapp-manager-acm/src/test/java/org/oransc/rappmanager/acm/service/AcmDeployerTest.java b/rapp-manager-acm/src/test/java/org/oransc/rappmanager/acm/service/AcmDeployerTest.java
index 406ec31..f176c7b 100644
--- a/rapp-manager-acm/src/test/java/org/oransc/rappmanager/acm/service/AcmDeployerTest.java
+++ b/rapp-manager-acm/src/test/java/org/oransc/rappmanager/acm/service/AcmDeployerTest.java
@@ -1,7 +1,7 @@
 /*-
  * ============LICENSE_START======================================================================
  * Copyright (C) 2023 Nordix Foundation. All rights reserved.
- * Copyright (C) 2023-2024 OpenInfra Foundation Europe. All rights reserved.
+ * Copyright (C) 2023-2025 OpenInfra Foundation Europe. All rights reserved.
  * ===============================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -114,7 +114,7 @@ class AcmDeployerTest {
     }
 
     @BeforeEach
-    public void init() {
+    void init() {
         mockServer = MockRestServiceServer.createServer(restTemplate);
     }
 
diff --git a/rapp-manager-application/src/main/java/org/oransc/rappmanager/rest/RappController.java b/rapp-manager-application/src/main/java/org/oransc/rappmanager/rest/RappController.java
index 9638d5f..a5e698d 100644
--- a/rapp-manager-application/src/main/java/org/oransc/rappmanager/rest/RappController.java
+++ b/rapp-manager-application/src/main/java/org/oransc/rappmanager/rest/RappController.java
@@ -1,7 +1,7 @@
 /*-
  * ============LICENSE_START======================================================================
  * Copyright (C) 2023 Nordix Foundation. All rights reserved.
- * Copyright (C) 2023-2024 OpenInfra Foundation Europe. All rights reserved.
+ * Copyright (C) 2023-2025 OpenInfra Foundation Europe. All rights reserved.
  * ===============================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -92,7 +92,11 @@ public class RappController {
             rappCacheService.putRapp(rapp);
             return ResponseEntity.accepted().build();
         } else {
-            logger.info("Invalid Rapp package for {}", rappId);
+            if (rappId == null) {
+                logger.info("Invalid Rapp package with null rAppId");
+            } else {
+                logger.info("Invalid Rapp package for {}", rappId);
+            }
             throw new RappHandlerException(HttpStatus.BAD_REQUEST, "Invalid rApp package.");
         }
     }
diff --git a/rapp-manager-application/src/test/java/org/oransc/rappmanager/service/DeploymentArtifactsServiceTest.java b/rapp-manager-application/src/test/java/org/oransc/rappmanager/service/DeploymentArtifactsServiceTest.java
index 854e205..d1978b2 100644
--- a/rapp-manager-application/src/test/java/org/oransc/rappmanager/service/DeploymentArtifactsServiceTest.java
+++ b/rapp-manager-application/src/test/java/org/oransc/rappmanager/service/DeploymentArtifactsServiceTest.java
@@ -1,6 +1,6 @@
 /*
  * ============LICENSE_START======================================================================
- * Copyright (C) 2024 OpenInfra Foundation Europe. All rights reserved.
+ * Copyright (C) 2024-2025 OpenInfra Foundation Europe. All rights reserved.
  * ===============================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -72,7 +72,7 @@ class DeploymentArtifactsServiceTest {
     private final String validRappFile = "valid-rapp-package.csar";
 
     @BeforeEach
-    public void init() {
+    void init() {
         mockServer = MockRestServiceServer.createServer(restTemplate);
     }
 
diff --git a/rapp-manager-dme/src/test/java/org/oransc/rappmanager/dme/service/DmeDeployerTest.java b/rapp-manager-dme/src/test/java/org/oransc/rappmanager/dme/service/DmeDeployerTest.java
index dac4345..01bb422 100644
--- a/rapp-manager-dme/src/test/java/org/oransc/rappmanager/dme/service/DmeDeployerTest.java
+++ b/rapp-manager-dme/src/test/java/org/oransc/rappmanager/dme/service/DmeDeployerTest.java
@@ -1,7 +1,7 @@
 /*-
  * ============LICENSE_START======================================================================
  * Copyright (C) 2023 Nordix Foundation. All rights reserved.
- * Copyright (C) 2024 OpenInfra Foundation Europe. All rights reserved.
+ * Copyright (C) 2024-2025 OpenInfra Foundation Europe. All rights reserved.
  * ===============================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -98,7 +98,7 @@ class DmeDeployerTest {
     }
 
     @BeforeEach
-    public void init() {
+    void init() {
         mockServer = MockRestServiceServer.createServer(restTemplate);
     }
 
diff --git a/rapp-manager-sme/src/test/java/org/oransc/rappmanager/sme/service/SmeDeployerTest.java b/rapp-manager-sme/src/test/java/org/oransc/rappmanager/sme/service/SmeDeployerTest.java
index f5cbe8a..2c58653 100644
--- a/rapp-manager-sme/src/test/java/org/oransc/rappmanager/sme/service/SmeDeployerTest.java
+++ b/rapp-manager-sme/src/test/java/org/oransc/rappmanager/sme/service/SmeDeployerTest.java
@@ -1,7 +1,7 @@
 /*-
  * ============LICENSE_START======================================================================
  * Copyright (C) 2023 Nordix Foundation. All rights reserved.
- * Copyright (C) 2024 OpenInfra Foundation Europe. All rights reserved.
+ * Copyright (C) 2024-2025 OpenInfra Foundation Europe. All rights reserved.
  * ===============================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -99,7 +99,7 @@ class SmeDeployerTest {
     }
 
     @BeforeEach
-    public void init() {
+    void init() {
         mockServer = MockRestServiceServer.createServer(restTemplate);
     }