Prettier

apollographql · May 18, 2020 · 66559c8 · 66559c8
1 parent 1dde2db
commit 66559c8
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 17 deletions.
diff --git a/packages/apollo-engine-reporting/src/__tests__/plugin.test.ts b/packages/apollo-engine-reporting/src/__tests__/plugin.test.ts
@@ -201,8 +201,7 @@ describe('schema reporting', () => {
   });
 });
 
-
-it("trace construction", async () => {
+it('trace construction', async () => {
   const schema = makeExecutableSchema({ typeDefs });
   addMockFunctionsToSchema({ schema });
 
@@ -477,18 +476,22 @@ function makeTestHTTP(): Trace.HTTP {
 describe('tests for the shouldReportQuery reporting option', () => {
   const schemaReportingFunctions = {
     startSchemaReporting: jest.fn(),
-    executableSchemaIdGenerator: jest.fn()
-  }
+    executableSchemaIdGenerator: jest.fn(),
+  };
   const schema = makeExecutableSchema({ typeDefs });
   addMockFunctionsToSchema({ schema });
 
   const addTrace = jest.fn();
   beforeEach(() => {
     addTrace.mockClear();
-  })
+  });
 
   it('report no traces', async () => {
-    const pluginInstance = plugin({ traceReporting: false }, addTrace, schemaReportingFunctions);
+    const pluginInstance = plugin(
+      { traceReporting: false },
+      addTrace,
+      schemaReportingFunctions,
+    );
 
     const context = await pluginTestHarness({
       pluginInstance,
@@ -517,7 +520,10 @@ describe('tests for the shouldReportQuery reporting option', () => {
         traceReporting: async request => {
           return request.request.operationName === 'report';
         },
-      }, addTrace, schemaReportingFunctions);
+      },
+      addTrace,
+      schemaReportingFunctions,
+    );
 
     const context1 = await pluginTestHarness({
       pluginInstance,
@@ -566,17 +572,17 @@ describe('tests for the shouldReportQuery reporting option', () => {
   });
 
   it('report traces async based on operation name', async () => {
-
     const pluginInstance = plugin(
       {
-        traceReporting: async (request) => {
+        traceReporting: async request => {
           let shouldTrace = await (async () => {
             return request.request.operationName === 'report';
-          })()
+          })();
           return shouldTrace;
         },
       },
       addTrace,
+      schemaReportingFunctions
     );
 
     const context1 = await pluginTestHarness({

diff --git a/packages/apollo-engine-reporting/src/agent.ts b/packages/apollo-engine-reporting/src/agent.ts
@@ -23,7 +23,7 @@ import {
   GraphQLRequestContext,
   GraphQLRequestContextDidResolveOperation,
   Logger,
-} from "apollo-server-types";
+} from 'apollo-server-types';
 import { InMemoryLRUCache } from 'apollo-server-caching';
 import { defaultEngineReportingSignature } from 'apollo-graphql';
 import { ApolloServerPlugin } from 'apollo-server-plugin-base';
@@ -59,7 +59,10 @@ export type VariableValueOptions =
   | SendValuesBaseOptions;
 
 export type TraceReportingOptions<TContext> =
-  ((request: GraphQLRequestContextDidResolveOperation<TContext>) => Promise<boolean>) | boolean;
+  | ((
+      request: GraphQLRequestContextDidResolveOperation<TContext>,
+    ) => Promise<boolean>)
+  | boolean;
 
 export type GenerateClientInfo<TContext> = (
   requestContext: GraphQLRequestContext<TContext>,
@@ -246,7 +249,7 @@ export interface EngineReportingOptions<TContext> {
    * ```
    *
    */
-  traceReporting?: TraceReportingOptions<TContext>
+  traceReporting?: TraceReportingOptions<TContext>;
   /**
    * [DEPRECATED] Use sendVariableValues
    * Passing an array into privateVariables is equivalent to passing { exceptNames: array } into

diff --git a/packages/apollo-engine-reporting/src/plugin.ts b/packages/apollo-engine-reporting/src/plugin.ts
@@ -186,14 +186,17 @@ export const plugin = <TContext>(
 
       return {
         async didResolveOperation(requestContext) {
-          if (typeof options.traceReporting !== "function") return;
-          const shouldReportTrace = await options.traceReporting(requestContext);
+          if (typeof options.traceReporting !== 'function') return;
+          const shouldReportTrace = await options.traceReporting(
+            requestContext,
+          );
 
           // Help the user understand they've returned an unexpected value,
           // which might be a subtle mistake.
-          if (typeof shouldReportTrace !== "boolean") {
+          if (typeof shouldReportTrace !== 'boolean') {
             (requestContext.logger || logger).warn(
-              "The 'traceReporting' predicate function must return a boolean value.");
+              "The 'traceReporting' predicate function must return a boolean value.",
+            );
             return;
           }