fix: add missing schemas so rule config is properly validated (#291)

Author: bradzacher

src/rules/lowercase-name.js

@@ -55,6 +55,21 @@ module.exports = {
     messages: {
       unexpectedLowercase: '`{{ method }}`s should begin with lowercase',
     },
+    schema: [
+      {
+        type: 'object',
+        properties: {
+          ignore: {
+            type: 'array',
+            items: {
+              enum: ['describe', 'test', 'it'],
+            },
+            additionalItems: false,
+          },
+        },
+        additionalProperties: false,
+      },
+    ],
     fixable: 'code',
   },
   create(context) {

src/rules/no-alias-methods.js

@@ -11,6 +11,7 @@ module.exports = {
       replaceAlias: `Replace {{ replace }}() with its canonical name of {{ canonical }}()`,
     },
     fixable: 'code',
+    schema: [],
   },
   create(context) {
     // The Jest methods which have aliases. The canonical name is the first

src/rules/no-commented-out-tests.js

@@ -16,6 +16,7 @@ module.exports = {
     messages: {
       commentedTests: 'Some tests seem to be commented',
     },
+    schema: [],
   },
   create(context) {
     const sourceCode = context.getSourceCode();

src/rules/no-disabled-tests.js

@@ -17,6 +17,7 @@ module.exports = {
       disabledSuite: 'Disabled test suite',
       disabledTest: 'Disabled test',
     },
+    schema: [],
   },
   create(context) {
     let suiteDepth = 0;

src/rules/no-empty-title.js

@@ -19,6 +19,7 @@ module.exports = {
       describe: 'describe should not have an empty title',
       test: 'test should not have an empty title',
     },
+    schema: [],
   },
   create(context) {
     return {

src/rules/no-focused-tests.js

@@ -27,6 +27,7 @@ module.exports = {
     messages: {
       focusedTest: 'Unexpected focused test.',
     },
+    schema: [],
   },
   create: context => ({
     CallExpression(node) {

src/rules/no-identical-title.js

@@ -54,6 +54,7 @@ module.exports = {
       multipleDescribeTitle:
         'Describe block title is used multiple times in the same describe block.',
     },
+    schema: [],
   },
   create(context) {
     const contexts = [newDescribeContext()];

src/rules/no-jasmine-globals.js

@@ -19,6 +19,7 @@ module.exports = {
         'Illegal usage of `pending`, prefer explicitly skipping a test using `test.skip`',
       illegalJasmine: 'Illegal usage of jasmine global',
     },
+    schema: [],
   },
   create(context) {
     return {

src/rules/no-jest-import.js

@@ -10,6 +10,7 @@ module.exports = {
     messages: {
       unexpectedImport: `Jest is automatically in scope. Do not import "jest", as Jest doesn't export anything.`,
     },
+    schema: [],
   },
   create(context) {
     return {

src/rules/no-large-snapshots.js

@@ -30,6 +30,17 @@ module.exports = {
       tooLongSnapshots:
         'Expected Jest snapshot to be smaller than {{ lineLimit }} lines but was {{ lineCount }} lines long',
     },
+    schema: [
+      {
+        type: 'object',
+        properties: {
+          maxSize: {
+            type: 'number',
+          },
+        },
+        additionalProperties: false,
+      },
+    ],
   },
   create(context) {
     if (context.getFilename().endsWith('.snap')) {