X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=policy-agent%2Fsrc%2Ftest%2Fjava%2Forg%2Foransc%2Fpolicyagent%2Ftasks%2FRepositorySupervisionTest.java;h=d837f78dcf9a4f9fea5a80bd57f58282151f275e;hb=72e4ad7ac672c8837fdbd8c394dccc9b2549253e;hp=fd3dbc01c77f0e305e762331df3b495c35ded792;hpb=444d11bb9de7be056d7609c395f3ef191c3c1702;p=nonrtric.git

diff --git a/policy-agent/src/test/java/org/oransc/policyagent/tasks/RepositorySupervisionTest.java b/policy-agent/src/test/java/org/oransc/policyagent/tasks/RepositorySupervisionTest.java
index fd3dbc01..d837f78d 100644
--- a/policy-agent/src/test/java/org/oransc/policyagent/tasks/RepositorySupervisionTest.java
+++ b/policy-agent/src/test/java/org/oransc/policyagent/tasks/RepositorySupervisionTest.java
@@ -20,6 +20,7 @@
 
 package org.oransc.policyagent.tasks;
 
+import static org.assertj.core.api.Assertions.assertThat;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.spy;
@@ -32,6 +33,8 @@ import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
 import java.util.Vector;
+
+import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -42,6 +45,7 @@ import org.oransc.policyagent.clients.A1ClientFactory;
 import org.oransc.policyagent.configuration.ImmutableRicConfig;
 import org.oransc.policyagent.repository.ImmutablePolicy;
 import org.oransc.policyagent.repository.ImmutablePolicyType;
+import org.oransc.policyagent.repository.Lock.LockType;
 import org.oransc.policyagent.repository.Policies;
 import org.oransc.policyagent.repository.Policy;
 import org.oransc.policyagent.repository.PolicyType;
@@ -60,7 +64,7 @@ public class RepositorySupervisionTest {
         .build();
 
     private static final Ric RIC_1 = new Ric(ImmutableRicConfig.builder() //
-        .name("ric1") //
+        .name("RIC_1") //
         .baseUrl("baseUrl1") //
         .managedElementIds(new Vector<String>(Arrays.asList("kista_1", "kista_2"))) //
         .build());
@@ -93,20 +97,29 @@ public class RepositorySupervisionTest {
     @Mock
     private RicSynchronizationTask recoveryTaskMock;
 
-    private PolicyTypes types;
-    private Policies policies;
-    private Rics rics;
+    private final PolicyTypes types = new PolicyTypes();
+    private Policies policies = new Policies();
+    private Rics rics = new Rics();
 
     @BeforeEach
     public void init() {
         doReturn(Mono.just(a1ClientMock)).when(a1ClientFactory).createA1Client(any(Ric.class));
-        types = new PolicyTypes();
-        policies = new Policies();
-        rics = new Rics();
+        types.clear();
+        policies.clear();
+        rics.clear();
         RIC_1.setState(RicState.UNDEFINED);
         RIC_1.clearSupportedPolicyTypes();
     }
 
+    @AfterEach
+    public void verifyNoRicLocks() {
+        for (Ric ric : this.rics.getRics()) {
+            ric.getLock().lockBlocking(LockType.EXCLUSIVE);
+            ric.getLock().unlockBlocking();
+            assertThat(ric.getLock().getLockCounter()).isEqualTo(0);
+        }
+    }
+
     @Test
     public void whenRicIdleAndNoChangedPoliciesOrPolicyTypes_thenNoRecovery() {
         RIC_1.setState(RicState.IDLE);
@@ -309,4 +322,4 @@ public class RepositorySupervisionTest {
             when(a1ClientMock.getPolicyTypeIdentities()).thenReturn(Mono.error((Exception) returnValue));
         }
     }
-}
\ No newline at end of file
+}