Handling empty objects

lo1tuma · Sep 30, 2016 · 4a8a686 · 4a8a686
1 parent 0efd731
commit 4a8a686
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/lib/rules/max-top-level-suites.js b/lib/rules/max-top-level-suites.js
@@ -12,12 +12,13 @@ var R = require('ramda'),
 module.exports = function (context) {
     var stack = [],
         topLevelDescribes = [],
-        suiteLimit = context.options[0];
+        options = context.options[0] || {},
+        suiteLimit;
 
-    if (R.isNil(suiteLimit)) {
+    if (R.isNil(options.limit)) {
         suiteLimit = defaultSuiteLimit;
     } else {
-        suiteLimit = suiteLimit.limit;
+        suiteLimit = options.limit;
     }
 
     return {

diff --git a/test/rules/max-top-level-suites.js b/test/rules/max-top-level-suites.js
@@ -36,6 +36,10 @@ ruleTester.run('max-top-level-suites', rules['max-top-level-suites'], {
             options: [ { limit: 0 } ],
             code: 'someOtherFunction();'
         },
+        {
+            options: [ { } ],
+            code: 'someOtherFunction();'
+        },
         'someOtherFunction();'
     ],
 
@@ -139,6 +143,14 @@ ruleTester.run('max-top-level-suites', rules['max-top-level-suites'], {
             errors: [
                 { message: 'The number of top-level suites is more than 0.' }
             ]
+        },
+        {
+            options: [ { } ],
+            code: 'describe("this is a test", function () { });' +
+                  'describe.only("this is a different test", function () { });',
+            errors: [
+                { message: 'The number of top-level suites is more than 1.' }
+            ]
         }
     ]
 });