Chore: remove unused null check.

Author: aladdin-add

lib/rules/consistent-output.js

@@ -30,9 +30,6 @@ module.exports = {
     return {
       Program (ast) {
         utils.getTestInfo(context, ast).forEach(testRun => {
-          if (!testRun.invalid) {
-            return;
-          }
           const readableCases = testRun.invalid.filter(testCase => testCase.type === 'ObjectExpression');
           const casesWithoutOutput = readableCases
             .filter(testCase => testCase.properties.map(utils.getKeyName).indexOf('output') === -1);

lib/rules/no-identical-tests.js

@@ -37,11 +37,6 @@ module.exports = {
      *@returns {boolean} if eq, return true, else return false.
     */
     function eq (testA, testB) {
-      // https://github.com/not-an-aardvark/eslint-plugin-eslint-plugin/issues/35
-      if (!testA || !testB) {
-        return testA === testB;
-      }
-
       if (testA.type !== testB.type) {
         return false;
       }
@@ -78,8 +73,7 @@ module.exports = {
         utils.getTestInfo(context, ast).forEach(testRun => {
           [testRun.valid, testRun.invalid].forEach(tests => {
             const cache = [];
-            // to avoid tests being null
-            (tests || []).forEach(test => {
+            tests.forEach(test => {
               if (cache.some(item => eq(item, test))) {
                 context.report({
                   node: test,

lib/rules/prefer-output-null.js

@@ -33,7 +33,7 @@ module.exports = {
     return {
       Program (ast) {
         utils.getTestInfo(context, ast).forEach(testRun => {
-          (testRun.invalid || []).forEach(test => {
+          testRun.invalid.forEach(test => {
             /**
             * Get a test case's giving keyname node.
             * @param {string} the keyname to find.

lib/rules/test-case-property-ordering.js

@@ -37,7 +37,7 @@ module.exports = {
       Program (ast) {
         utils.getTestInfo(context, ast).forEach(testRun => {
           [testRun.valid, testRun.invalid].forEach(tests => {
-            (tests || []).forEach(test => {
+            tests.forEach(test => {
               const properties = (test && test.properties) || [];
               const keyNames = properties.map(utils.getKeyName);
 

lib/rules/test-case-shorthand-strings.js

@@ -37,17 +37,15 @@ module.exports = {
     */
     function reportTestCases (cases) {
       const caseInfoList = cases.map(testCase => {
-        if (testCase) {
-          if (testCase.type === 'Literal' || testCase.type === 'TemplateLiteral') {
-            return { node: testCase, shorthand: true, needsLongform: false };
-          }
-          if (testCase.type === 'ObjectExpression') {
-            return {
-              node: testCase,
-              shorthand: false,
-              needsLongform: !(testCase.properties.length === 1 && utils.getKeyName(testCase.properties[0]) === 'code'),
-            };
-          }
+        if (testCase.type === 'Literal' || testCase.type === 'TemplateLiteral') {
+          return { node: testCase, shorthand: true, needsLongform: false };
+        }
+        if (testCase.type === 'ObjectExpression') {
+          return {
+            node: testCase,
+            shorthand: false,
+            needsLongform: !(testCase.properties.length === 1 && utils.getKeyName(testCase.properties[0]) === 'code'),
+          };
         }
         return null;
       }).filter(Boolean);