notifications: fix test race

Author: hieblmi

notifications/manager_test.go

@@ -140,9 +140,11 @@ func TestManager_ReservationNotification(t *testing.T) {
 		return len(mgr.subscribers[NotificationTypeReservation]) > 0
 	}, time.Second*5, 10*time.Millisecond)
 
-	mockClient.Lock()
-	require.Equal(t, 1, mockClient.timesCalled)
-	mockClient.Unlock()
+	require.Eventually(t, func() bool {
+		mockClient.Lock()
+		defer mockClient.Unlock()
+		return mockClient.timesCalled == 1
+	}, time.Second*5, 10*time.Millisecond)
 
 	// Send a test notification
 	testNotif := getTestNotification(testReservationId)