Fixes

Author: kamilkisiela

packages/plugins/graphql-jit/src/index.ts

@@ -72,13 +72,13 @@ export const useGraphQlJit = (
     return compiledQuery;
   }
 
-  function execute(args: ExecutionArgs) {
+  function jitExecute(args: ExecutionArgs) {
     const cacheEntry = getCacheEntry(args);
 
     return cacheEntry.query(args.rootValue, args.contextValue, args.variableValues);
   }
 
-  function subscribe(args: ExecutionArgs) {
+  function jitSubscribe(args: ExecutionArgs) {
     const cacheEntry = getCacheEntry(args);
 
     return cacheEntry.subscribe
@@ -98,12 +98,12 @@ export const useGraphQlJit = (
     },
     async onExecute({ args, setExecuteFn }) {
       if (!pluginOptions.enableIf || (pluginOptions.enableIf && (await pluginOptions.enableIf(args)))) {
-        setExecuteFn(execute);
+        setExecuteFn(jitExecute);
       }
     },
     async onSubscribe({ args, setSubscribeFn }) {
       if (!pluginOptions.enableIf || (pluginOptions.enableIf && (await pluginOptions.enableIf(args)))) {
-        setSubscribeFn(subscribe);
+        setSubscribeFn(jitSubscribe);
       }
     },
   };

packages/plugins/graphql-jit/test/graphql-jit.spec.ts

@@ -52,7 +52,7 @@ describe('useGraphQlJit', () => {
 
     expect(onExecuteSpy).toHaveBeenCalledTimes(1);
     expect(onExecuteSpy.mock.calls[0][0].executeFn).not.toBe(execute);
-    expect(onExecuteSpy.mock.calls[0][0].executeFn.name).toBe('jitExecutor');
+    expect(onExecuteSpy.mock.calls[0][0].executeFn.name).toBe('jitExecute');
   });
 
   it('Should override subscribe function', async () => {
@@ -72,7 +72,7 @@ describe('useGraphQlJit', () => {
 
     expect(onSubscribeSpy).toHaveBeenCalledTimes(1);
     expect(onSubscribeSpy.mock.calls[0][0].subscribeFn).not.toBe(subscribe);
-    expect(onSubscribeSpy.mock.calls[0][0].subscribeFn.name).toBe('jitSubscriber');
+    expect(onSubscribeSpy.mock.calls[0][0].subscribeFn.name).toBe('jitSubscribe');
   });
 
   it('Should not override execute function when enableIf returns false', async () => {
@@ -97,7 +97,7 @@ describe('useGraphQlJit', () => {
 
     expect(onExecuteSpy).toHaveBeenCalledTimes(1);
     expect(onExecuteSpy.mock.calls[0][0].executeFn).toBe(execute);
-    expect(onExecuteSpy.mock.calls[0][0].executeFn.name).not.toBe('jitExecutor');
+    expect(onExecuteSpy.mock.calls[0][0].executeFn.name).not.toBe('jitExecute');
   });
 
   it('Should not override subscribe function when enableIf returns false', async () => {
@@ -122,7 +122,7 @@ describe('useGraphQlJit', () => {
 
     expect(onSubscribeSpy).toHaveBeenCalledTimes(1);
     expect(onSubscribeSpy.mock.calls[0][0].subscribeFn).toBe(subscribe);
-    expect(onSubscribeSpy.mock.calls[0][0].subscribeFn.name).not.toBe('jitSubscriber');
+    expect(onSubscribeSpy.mock.calls[0][0].subscribeFn.name).not.toBe('jitSubscribe');
   });
 
   it('Should execute correctly', async () => {