nodejs
diff --git a/Collapse file
‎.eslintrc‎
Copy file name to clipboardExpand all lines: .eslintrc
+1Lines changed: 1 addition & 0 deletions b/Collapse file
‎.eslintrc‎
Copy file name to clipboardExpand all lines: .eslintrc
+1Lines changed: 1 addition & 0 deletions
diff --git a/Collapse file
‎tools/eslint-rules/assert-fail-single-argument.js‎
Copy file name to clipboard
+30Lines changed: 30 additions & 0 deletions b/Collapse file
‎tools/eslint-rules/assert-fail-single-argument.js‎
Copy file name to clipboard
+30Lines changed: 30 additions & 0 deletions
@@ -85,6 +85,7 @@ rules:
   prefer-const: 2
 
   # Custom rules in tools/eslint-rules
+  assert-fail-single-argument: 2
   new-with-error: [2, "Error", "RangeError", "TypeError", "SyntaxError", "ReferenceError"]
 
 
 
@@ -0,0 +1,30 @@
+/**
+ * @fileoverview Prohibit use of a single argument only in `assert.fail()`. It
+ * is almost always an error.
+ * @author Rich Trott
+ */
+'use strict';
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+const msg = 'assert.fail() message should be third argument';
+
+function isAssert(node) {
+  return node.callee.object && node.callee.object.name === 'assert';
+}
+
+function isFail(node) {
+  return node.callee.property && node.callee.property.name === 'fail';
+}
+
+module.exports = function(context) {
+  return {
+    'CallExpression': function(node) {
+      if (isAssert(node) && isFail(node) && node.arguments.length === 1) {
+        context.report(node, msg);
+      }
+    }
+  };
+};