Catch formatError

CHANGELOG.md

@@ -6,6 +6,7 @@
 * Pass `ctx` instead of `ctx.request` to options function in Koa integration ([@HriBB](https://github.com/HriBB)) in [PR #154](https://github.com/apollostack/apollo-server/pull/154)
 * Manage TypeScript declaration files using npm. ([@od1k](https:/github.com/od1k) in [#162](https://github.com/apollostack/apollo-server/pull/162))
 * Fix connect example in readme. ([@conrad-vanl](https://github.com/conrad-vanl) in [#165](https://github.com/apollostack/apollo-server/pull/165))
+* Add try/catch to formatError. ([@nicolaslopezj](https://github.com/nicolaslopezj) in [#174](https://github.com/apollostack/apollo-server/pull/174))
 
 ### v0.3.2
 * Added missing exports for hapi integration ([@nnance](https://github.com/nnance)) in [PR #152](https://github.com/apollostack/apollo-server/pull/152)

src/core/runQuery.ts

@@ -69,11 +69,18 @@ function doRunQuery(options: QueryOptions): Promise<GraphQLResult> {
     logFunction({action: LogAction.request, step: LogStep.start});
 
     function format(errors: Array<Error>): Array<Error> {
-        // TODO: fix types! shouldn't have to cast.
-        // the blocker is that the typings aren't right atm:
-        // GraphQLResult returns Array<GraphQLError>, but the formatError function
-        // returns Array<GraphQLFormattedError>
-        return errors.map(options.formatError || formatError as any) as Array<Error>;
+        return errors.map((error) => {
+          if (options.formatError) {
+            try {
+              return options.formatError(error);
+            } catch (err) {
+              console.error('Error in formatError function:', err);
+              return formatError(error);
+            }
+          } else {
+            return formatError(error);
+          }
+        }) as Array<Error>;
     }
 
     function printStackTrace(error: Error) {

src/integrations/integrations.test.ts

@@ -424,6 +424,31 @@ export default (createApp: CreateAppFunc, destroyApp?: DestroyAppFunc) => {
           });
       });
 
+      it('should use default formatter if provided formatError fails', () => {
+          app = createApp({apolloOptions: {
+              schema: Schema,
+              formatError: (err) => {
+                throw new Error('I should be catched');
+              },
+          }});
+          const logStub = stub(console, 'error');
+          const expected = /at resolveOrError/;
+          app = createApp({apolloOptions: {
+              schema: Schema,
+              debug: true,
+          }});
+          const req = request(app)
+              .post('/graphql')
+              .send({
+                  query: 'query test{ testError }',
+              });
+          return req.then((res) => {
+              logStub.restore();
+              expect(logStub.callCount).to.equal(1);
+              return expect(logStub.getCall(0).args[0]).to.match(expected);
+          });
+      });
+
       it('sends stack trace to error if debug mode is set', () => {
           const expected = /at resolveOrError/;
           const stackTrace = [];