use concurrent data structure for hooks

Signed-off-by: christian.lutnik <[email protected]>

Author: chrfwow

src/test/java/dev/openfeature/sdk/LockingSingeltonTest.java

@@ -36,7 +36,6 @@ void beforeEach() {
         OpenFeatureAPI.lock = apiLock;
 
         clientHooksLock = setupLock(clientHooksLock, mockInnerReadLock(), mockInnerWriteLock());
-        client.hooksLock = clientHooksLock;
     }
 
     @Nested
@@ -134,28 +133,6 @@ void onProviderErrorProviderReadyShouldApiWriteLockAndUnlock() {
         }
     }
 
-    @Test
-    void addHooksShouldWriteLockAndUnlock() {
-        client.addHooks(new Hook() {});
-        verify(clientHooksLock.writeLock()).lock();
-        verify(clientHooksLock.writeLock()).unlock();
-
-        api.addHooks(new Hook() {});
-        verify(apiLock.writeLock()).lock();
-        verify(apiLock.writeLock()).unlock();
-    }
-
-    @Test
-    void getHooksShouldReadLockAndUnlock() {
-        client.getHooks();
-        verify(clientHooksLock.readLock()).lock();
-        verify(clientHooksLock.readLock()).unlock();
-
-        api.getHooks();
-        verify(apiLock.readLock()).lock();
-        verify(apiLock.readLock()).unlock();
-    }
-
     @Test
     void setTransactionalContextPropagatorShouldWriteLockAndUnlock() {
         api.setTransactionContextPropagator(new NoOpTransactionContextPropagator());
@@ -170,13 +147,6 @@ void getTransactionalContextPropagatorShouldReadLockAndUnlock() {
         verify(apiLock.readLock()).unlock();
     }
 
-    @Test
-    void clearHooksShouldWriteLockAndUnlock() {
-        api.clearHooks();
-        verify(apiLock.writeLock()).lock();
-        verify(apiLock.writeLock()).unlock();
-    }
-
     private static ReentrantReadWriteLock.ReadLock mockInnerReadLock() {
         ReentrantReadWriteLock.ReadLock readLockMock = mock(ReentrantReadWriteLock.ReadLock.class);
         doNothing().when(readLockMock).lock();