diff options
author | halil.cakal <halil.cakal@est.tech> | 2024-09-25 11:07:01 +0100 |
---|---|---|
committer | halil.cakal <halil.cakal@est.tech> | 2024-09-25 15:46:38 +0100 |
commit | fe846d2dd21d57889356943762f2006b2122427c (patch) | |
tree | a50cb76c8d340f21a5b663e00971f8da1607c6b2 | |
parent | 2dbf14888235bf6140fe2e041e8681e15a725314 (diff) |
BugFix: 500 Server error when upgrade and service restart with data in DB
- return NONE for get effective trust level api if the trustlevel caches
empty (restart case)
Issue-ID: CPS-2409
Change-Id: I72a755ca6ba5d4a78f5458a235c0d1f43aaa8d53
Signed-off-by: halil.cakal <halil.cakal@est.tech>
2 files changed, 6 insertions, 1 deletions
diff --git a/cps-ncmp-service/src/main/java/org/onap/cps/ncmp/impl/inventory/trustlevel/TrustLevelManager.java b/cps-ncmp-service/src/main/java/org/onap/cps/ncmp/impl/inventory/trustlevel/TrustLevelManager.java index afe6ad5c1c..9b0cc3a0c0 100644 --- a/cps-ncmp-service/src/main/java/org/onap/cps/ncmp/impl/inventory/trustlevel/TrustLevelManager.java +++ b/cps-ncmp-service/src/main/java/org/onap/cps/ncmp/impl/inventory/trustlevel/TrustLevelManager.java @@ -140,7 +140,7 @@ public class TrustLevelManager { */ public TrustLevel getEffectiveTrustLevel(final String cmHandleId) { final TrustLevel dmiTrustLevel = TrustLevel.COMPLETE; // TODO: CPS-2375 - final TrustLevel cmHandleTrustLevel = trustLevelPerCmHandle.get(cmHandleId); + final TrustLevel cmHandleTrustLevel = trustLevelPerCmHandle.getOrDefault(cmHandleId, TrustLevel.NONE); return dmiTrustLevel.getEffectiveTrustLevel(cmHandleTrustLevel); } diff --git a/cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/impl/inventory/trustlevel/TrustLevelManagerSpec.groovy b/cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/impl/inventory/trustlevel/TrustLevelManagerSpec.groovy index fe762f891a..e0f5f9c4f8 100644 --- a/cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/impl/inventory/trustlevel/TrustLevelManagerSpec.groovy +++ b/cps-ncmp-service/src/test/groovy/org/onap/cps/ncmp/impl/inventory/trustlevel/TrustLevelManagerSpec.groovy @@ -149,6 +149,11 @@ class TrustLevelManagerSpec extends Specification { assert effectiveTrustLevel == TrustLevel.NONE } + def 'Select effective trust level when the trust level caches are empty (restart case)'() { + expect: 'effective trust level is NONE when cm-1 does not exist in the cache' + assert objectUnderTest.getEffectiveTrustLevel('ch-1') == TrustLevel.NONE + } + def 'CmHandle trust level removed'() { given: 'a cm handle' trustLevelPerCmHandle.put('ch-1', TrustLevel.COMPLETE) |