From 18c97de214a69a61e742f48f678f6a8fb842adbc Mon Sep 17 00:00:00 2001
From: Brian Macdonald <brian.lmacdonald@gmail.com>
Date: Sat, 10 Feb 2018 16:47:31 -0500
Subject: [PATCH] fix: add warning to describe (#65)

---
 rules/__tests__/valid-describe.test.js | 26 +++++++++++++++++++++++++-
 rules/valid-describe.js                | 13 +++++++++++++
 2 files changed, 38 insertions(+), 1 deletion(-)

diff --git a/rules/__tests__/valid-describe.test.js b/rules/__tests__/valid-describe.test.js
index b79921189..a44da5b64 100644
--- a/rules/__tests__/valid-describe.test.js
+++ b/rules/__tests__/valid-describe.test.js
@@ -11,7 +11,6 @@ const ruleTester = new RuleTester({
 
 ruleTester.run('valid-describe', rules['valid-describe'], {
   valid: [
-    'describe("foo")',
     'describe("foo", function() {})',
     'describe("foo", () => {})',
     'xdescribe("foo", () => {})',
@@ -36,6 +35,31 @@ ruleTester.run('valid-describe', rules['valid-describe'], {
     `,
   ],
   invalid: [
+    {
+      code: 'describe(() => {})',
+      errors: [
+        {
+          message: 'First argument must be name',
+          line: 1,
+          column: 10,
+        },
+        {
+          message: 'Describe requires name and callback arguments',
+          line: 1,
+          column: 10,
+        },
+      ],
+    },
+    {
+      code: 'describe("foo")',
+      errors: [
+        {
+          message: 'Describe requires name and callback arguments',
+          line: 1,
+          column: 10,
+        },
+      ],
+    },
     {
       code: 'describe("foo", async () => {})',
       errors: [{ message: 'No async describe callback', line: 1, column: 17 }],
diff --git a/rules/valid-describe.js b/rules/valid-describe.js
index 7d844441a..0d92abd28 100644
--- a/rules/valid-describe.js
+++ b/rules/valid-describe.js
@@ -33,7 +33,20 @@ module.exports = {
     return {
       CallExpression(node) {
         if (isDescribe(node)) {
+          const name = node.arguments[0];
           const callbackFunction = node.arguments[1];
+          if (name.type !== 'Literal') {
+            context.report({
+              message: 'First argument must be name',
+              loc: paramsLocation(node.arguments),
+            });
+          }
+          if (callbackFunction === undefined) {
+            context.report({
+              message: 'Describe requires name and callback arguments',
+              loc: paramsLocation(node.arguments),
+            });
+          }
           if (callbackFunction && isFunction(callbackFunction)) {
             if (isAsync(callbackFunction)) {
               context.report({