refactor: convert all query keys into arrays

resolves #65

Author: Katarina Anton

src/__tests__/subscription-storage.spec.ts

@@ -38,14 +38,14 @@ describe('subscription storage', () => {
     const observerFn = jest.fn();
     const { subscription, next } = subscriptionFactory(observerFn);
 
-    storeSubscription(queryClient, 'test', subscription);
+    storeSubscription(queryClient, ['test'], subscription);
 
     next('value-1');
     expect(observerFn).toHaveBeenCalledTimes(1);
     expect(observerFn).toHaveBeenCalledWith('value-1');
     observerFn.mockClear();
 
-    cleanupSubscription(queryClient, 'test');
+    cleanupSubscription(queryClient, ['test']);
 
     next('value-2');
     expect(observerFn).not.toHaveBeenCalled();
@@ -59,8 +59,8 @@ describe('subscription storage', () => {
     const { subscription: subscriptionB, next: nextB } =
       subscriptionFactory(observerBFn);
 
-    storeSubscription(queryClient, 'testA', subscriptionA);
-    storeSubscription(queryClient, 'testB', subscriptionB);
+    storeSubscription(queryClient, ['testA'], subscriptionA);
+    storeSubscription(queryClient, ['testB'], subscriptionB);
 
     nextA('A1');
     expect(observerAFn).toHaveBeenCalledTimes(1);
@@ -71,7 +71,7 @@ describe('subscription storage', () => {
     expect(observerBFn).toHaveBeenCalledWith('B1');
     observerBFn.mockClear();
 
-    cleanupSubscription(queryClient, 'testA');
+    cleanupSubscription(queryClient, ['testA']);
 
     nextA('A2');
     expect(observerAFn).not.toHaveBeenCalled();
@@ -82,7 +82,7 @@ describe('subscription storage', () => {
 
   it('should not fail when key does not exist', () => {
     expect(() =>
-      cleanupSubscription(queryClient, 'test-non-existing')
+      cleanupSubscription(queryClient, ['test-non-existing'])
     ).not.toThrow();
   });
 });

src/__tests__/use-infinite-subscription.spec.tsx

@@ -49,7 +49,7 @@ describe('useInfiniteSubscription', () => {
     };
   }
 
-  const testSubscriptionKey = 'test-subscription-key';
+  const testSubscriptionKey = ['test-subscription-key'];
 
   function mapToPages<T>(data: T) {
     return { pageParams: [undefined], pages: [data] };

src/__tests__/use-subscription.spec.tsx

@@ -41,7 +41,7 @@ describe('useSubscription', () => {
   const test$ = interval(testInterval).pipe(finalize(finalizeFn));
   const testSubscriptionFn = jest.fn(() => test$);
 
-  const testSubscriptionKey = 'test-subscription-key';
+  const testSubscriptionKey = ['test-subscription-key'];
 
   afterEach(() => {
     testSubscriptionFn.mockClear();
@@ -231,7 +231,7 @@ describe('useSubscription', () => {
       firstHookRender = renderHook(
         (options: UseSubscriptionOptions) =>
           useSubscription(
-            'first-subscription-key',
+            ['first-subscription-key'],
             testSubscriptionFn,
             options
           ),

src/subscription-storage.ts

@@ -1,7 +1,7 @@
 import { QueryKey, hashQueryKey, QueryClient } from 'react-query';
 import { Subscription } from 'rxjs';
 
-const clientCacheSubscriptionsKey = '__activeSubscriptions__';
+const clientCacheSubscriptionsKey = ['__activeSubscriptions__'];
 const defaultKey = Symbol('__default__');
 
 type SubscriptionStorageItem = Map<string | typeof defaultKey, Subscription>;