wip

Author: bmish

lib/rules/no-arrow-tests.js

@@ -168,7 +168,7 @@ module.exports = {
          * @returns {boolean}
          */
         function isPropertyInModule(propertyNode) {
-            return (
+            return !!(
                 propertyNode &&
                 propertyNode.parent &&
                 propertyNode.parent.type === "ObjectExpression" &&

lib/rules/no-assert-equal.js

@@ -164,7 +164,7 @@ module.exports = {
          * @returns {boolean}
          */
         function isModuleHookCallback(propertyNode) {
-            return (
+            return !!(
                 propertyNode.parent &&
                 propertyNode.parent.type === "Property" &&
                 utils.isModuleHookPropertyKey(propertyNode.parent.key) &&

lib/rules/no-conditional-assertions.js

@@ -76,6 +76,7 @@ module.exports = {
          * @param {import('eslint').Rule.Node} assertNode
          */
         function checkAndReport(assertNode) {
+            /** @type {import('eslint').Rule.Node | null} */
             let currentNode = assertNode;
 
             while (
@@ -86,7 +87,7 @@ module.exports = {
                 currentNode = currentNode.parent;
             }
 
-            if (CONDITIONAL_NODE_TYPES.has(currentNode.type)) {
+            if (currentNode && CONDITIONAL_NODE_TYPES.has(currentNode.type)) {
                 context.report({
                     node: assertNode,
                     messageId: "noAssertionInsideConditional",

lib/rules/no-hooks-from-ancestor-modules.js

@@ -47,7 +47,7 @@ module.exports = {
          * @returns {boolean}
          */
         function isInModuleCallbackBody(callExpressionNode) {
-            return (
+            return !!(
                 callExpressionNode &&
                 callExpressionNode.parent &&
                 callExpressionNode.parent.type === "ExpressionStatement" &&

lib/utils.js

@@ -221,7 +221,7 @@ exports.isModule = function (calleeNode) {
  * @returns {boolean}
  */
 exports.isInModule = function (propertyNode) {
-    return (
+    return !!(
         propertyNode &&
         propertyNode.parent && // ObjectExpression
         propertyNode.parent.parent && // CallExpression?