ESLint: enable 'arrow-body-style' rule (#1840)

graphql · May 1, 2019 · 1023264 · 1023264
1 parent cd80f94
commit 1023264
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/.eslintrc.yml b/.eslintrc.yml
@@ -278,7 +278,7 @@ rules:
   # ECMAScript 6
   # https://eslint.org/docs/rules/#ecmascript-6
 
-  arrow-body-style: off
+  arrow-body-style: error
   constructor-super: error
   no-class-assign: error
   no-const-assign: error

diff --git a/src/subscription/__tests__/subscribe-test.js b/src/subscription/__tests__/subscribe-test.js
@@ -381,9 +381,9 @@ describe('Subscription Initialization Phase', () => {
 
   it('resolves to an error for subscription resolver errors', async () => {
     // Returning an error
-    const subscriptionReturningErrorSchema = emailSchemaWithResolvers(() => {
-      return new Error('test error');
-    });
+    const subscriptionReturningErrorSchema = emailSchemaWithResolvers(
+      () => new Error('test error'),
+    );
     await testReportsError(subscriptionReturningErrorSchema);
 
     // Throwing an error
@@ -394,9 +394,7 @@ describe('Subscription Initialization Phase', () => {
 
     // Resolving to an error
     const subscriptionResolvingErrorSchema = emailSchemaWithResolvers(
-      async () => {
-        return new Error('test error');
-      },
+      async () => new Error('test error'),
     );
     await testReportsError(subscriptionResolvingErrorSchema);