ci: Run unit tests in CI and guard undefined variables

.github/workflows/unit-tests.yml

@@ -0,0 +1,27 @@
+name: Unit tests
+on: [pull_request]
+
+permissions:
+  contents: read
+
+jobs:
+  unit-tests:
+    runs-on: ubuntu-22.04
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 1
+      - name: Install Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20.x
+      - name: Install dependencies
+        run: npm install
+      - name: Run unit tests
+        run: npm run test:unit:coverage
+      - name: Upload coverage report
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage-report
+          path: coverage/

.gitignore

@@ -3,6 +3,9 @@ dist/
 dist-tsc/
 .env
 
+# vitest
+coverage/
+
 # playwright
 test-results/
 playwright-report/

src/data-consumption/factory/hookCreator.ts

@@ -37,7 +37,9 @@ export const createDataConsumptionHook = <T>(
       sortedStringify(hookArguments?.variables)
       !== sortedStringify(variablesState)
     ) {
-      setVariablesState(() => JSON.parse(sortedStringify(hookArguments?.variables)));
+      setVariablesState(() => (hookArguments?.variables === undefined
+        ? undefined
+        : JSON.parse(sortedStringify(hookArguments.variables))));
     }
   }
 

tests/unit/data-consumption/factory/hookCreator.test.ts

@@ -116,6 +116,23 @@ describe('createDataConsumptionHook', () => {
       expect(variablesOf(subscribeEvents[1])).toEqual(PT_BR_VARIABLES);
     });
 
+    it('resubscribes without variables when they become undefined', () => {
+      const { rerender } = renderHook(
+        ({ variables }: { variables?: object }) => createDataConsumptionHook<CaptionData>(
+          DataConsumptionHooks.CUSTOM_SUBSCRIPTION,
+          { query: QUERY_WITH_ONE_VARIABLE, variables },
+        ),
+        { initialProps: { variables: EN_VARIABLES as object | undefined } },
+      );
+
+      rerender({ variables: undefined });
+
+      expect(unsubscribeEvents).toHaveLength(1);
+      expect(variablesOf(unsubscribeEvents[0])).toEqual(EN_VARIABLES);
+      expect(subscribeEvents).toHaveLength(2);
+      expect(variablesOf(subscribeEvents[1])).toBeUndefined();
+    });
+
     it('does not resubscribe when a rerender passes a new variables object with the same content', () => {
       const { rerender } = renderCustomSubscriptionHook({ locale: 'en' });