Merge pull request #791 from bmish/require-return-from-computed-test-cleanup

Cleanup tests for `require-return-from-computed` rule

Author: bmish

lib/rules/require-return-from-computed.js

@@ -10,6 +10,8 @@ function isReachable(segment) {
   return segment.reachable;
 }
 
+const ERROR_MESSAGE = 'Always return a value from computed properties';
+
 module.exports = {
   meta: {
     type: 'problem',
@@ -24,11 +26,11 @@ module.exports = {
     schema: [],
   },
 
-  create(context) {
-    const message = 'Always return a value from computed properties';
+  ERROR_MESSAGE,
 
+  create(context) {
     const report = function (node) {
-      context.report(node, message);
+      context.report(node, ERROR_MESSAGE);
     };
 
     let funcInfo = {

tests/lib/rules/require-return-from-computed.js

@@ -5,6 +5,8 @@
 const rule = require('../../../lib/rules/require-return-from-computed');
 const RuleTester = require('eslint').RuleTester;
 
+const { ERROR_MESSAGE } = rule;
+
 // ------------------------------------------------------------------------------
 // Tests
 // ------------------------------------------------------------------------------
@@ -40,7 +42,8 @@ eslintTester.run('require-return-from-computed', rule, {
       output: null,
       errors: [
         {
-          message: 'Always return a value from computed properties',
+          message: ERROR_MESSAGE,
+          type: 'FunctionExpression',
         },
       ],
     },
@@ -49,7 +52,8 @@ eslintTester.run('require-return-from-computed', rule, {
       output: null,
       errors: [
         {
-          message: 'Always return a value from computed properties',
+          message: ERROR_MESSAGE,
+          type: 'FunctionExpression',
         },
       ],
     },
@@ -58,10 +62,12 @@ eslintTester.run('require-return-from-computed', rule, {
       output: null,
       errors: [
         {
-          message: 'Always return a value from computed properties',
+          message: ERROR_MESSAGE,
+          type: 'FunctionExpression',
         },
         {
-          message: 'Always return a value from computed properties',
+          message: ERROR_MESSAGE,
+          type: 'FunctionExpression',
         },
       ],
     },
@@ -70,7 +76,8 @@ eslintTester.run('require-return-from-computed', rule, {
       output: null,
       errors: [
         {
-          message: 'Always return a value from computed properties',
+          message: ERROR_MESSAGE,
+          type: 'FunctionExpression',
         },
       ],
     },
@@ -79,7 +86,8 @@ eslintTester.run('require-return-from-computed', rule, {
       output: null,
       errors: [
         {
-          message: 'Always return a value from computed properties',
+          message: ERROR_MESSAGE,
+          type: 'FunctionExpression',
         },
       ],
     },