added test

altair-graphql · Dec 15, 2024 · 20e0918 · 20e0918
1 parent 628fa46
commit 20e0918
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 4 deletions.
diff --git a/packages/altair-app/src/app/modules/altair/services/query/query.service.ts b/packages/altair-app/src/app/modules/altair/services/query/query.service.ts
@@ -316,10 +316,8 @@ export class QueryService {
   private getRequestHandlerId(window: PerWindowState, isSubscription: boolean) {
     const defaultRequestHandlerId = window.query.requestHandlerId ?? HTTP_HANDLER_ID;
 
-    if (isSubscription) {
-      if (!window.query.subscriptionUseDefaultRequestHandler) {
-        return window.query.subscriptionRequestHandlerId ?? WEBSOCKET_HANDLER_ID;
-      }
+    if (isSubscription && !window.query.subscriptionUseDefaultRequestHandler) {
+      return window.query.subscriptionRequestHandlerId ?? WEBSOCKET_HANDLER_ID;
     }
 
     return defaultRequestHandlerId;

diff --git a/packages/altair-app/src/app/modules/altair/store/query/query.reducer.spec.ts b/packages/altair-app/src/app/modules/altair/store/query/query.reducer.spec.ts
@@ -0,0 +1,16 @@
+import { describe } from '@jest/globals';
+import { getInitialState } from './query.reducer';
+import { AltairConfig, setAltairConfig } from 'altair-graphql-core/build/config';
+
+describe('initialState', () => {
+  it('should return correct initialSubscriptionRequestHandlerId', () => {
+    const altairConfig = new AltairConfig({
+      initialSubscriptionRequestHandlerId: 'graphql-sse',
+    });
+    setAltairConfig(altairConfig);
+
+    const initialState = getInitialState();
+
+    expect(initialState.subscriptionRequestHandlerId).toBe('graphql-sse');
+  });
+});