Add no-incorrect-deep-equal rule

docs/rules/no-incorrect-deep-equal.md

@@ -0,0 +1,28 @@
+# Avoid using `deepEqual` with primitives
+
+The `deepEqual` and `notDeepEqual` assertions are unnecessary when comparing primitives. Use `is` or `not` instead.
+
+This rule is fixable.
+
+## Fail
+
+```js
+t.deepEqual(expression, 'foo');
+t.deepEqual(expression, 1);
+t.deepEqual(expression, `foo${bar}`);
+t.deepEqual(expression, null);
+t.deepEqual(expression, undefined);
+t.notDeepEqual(expression, undefined);
+```
+
+
+## Pass
+
+```js
+t.is(expression, 'foo');
+
+t.deepEqual(expression, otherExpression);
+t.deepEqual(expression, {});
+t.deepEqual(expression, []);
+t.notDeepEqual(expression, []);
+```

index.js

@@ -28,6 +28,7 @@ module.exports = {
 				'ava/no-identical-title': 'error',
 				'ava/no-ignored-test-files': 'error',
 				'ava/no-import-test-files': 'error',
+				'ava/no-incorrect-deep-equal': 'error',
 				'ava/no-invalid-end': 'error',
 				'ava/no-nested-tests': 'error',
 				'ava/no-only-test': 'error',

readme.md

@@ -46,6 +46,7 @@ Configure it in `package.json`.
 			"ava/no-identical-title": "error",
 			"ava/no-ignored-test-files": "error",
 			"ava/no-import-test-files": "error",
+			"ava/no-incorrect-deep-equal": "error",
 			"ava/no-invalid-end": "error",
 			"ava/no-nested-tests": "error",
 			"ava/no-only-test": "error",
@@ -83,6 +84,7 @@ The rules will only activate in test files.
 - [no-identical-title](docs/rules/no-identical-title.md) - Ensure no tests have the same title.
 - [no-ignored-test-files](docs/rules/no-ignored-test-files.md) - Ensure no tests are written in ignored files.
 - [no-import-test-files](docs/rules/no-import-test-files.md) - Ensure no test files are imported anywhere.
+- [no-incorrect-deep-equal](docs/rules/no-incorrect-deep-equal.md) - Avoid using `deepEqual` with literals. *(fixable)*
 - [no-invalid-end](docs/rules/no-invalid-end.md) - Ensure `t.end()` is only called inside `test.cb()`.
 - [no-nested-tests](docs/rules/no-nested-tests.md) - Ensure no tests are nested.
 - [no-only-test](docs/rules/no-only-test.md) - Ensure no `test.only()` are present. *(fixable)*

rules/no-incorrect-deep-equal.js

@@ -0,0 +1,125 @@
+'use strict';
+const {visitIf} = require('enhance-visitors');
+const util = require('../util');
+const createAvaRule = require('../create-ava-rule');
+
+const MESSAGE_ID_LITERAL = 'no-deep-equal-with-literal';
+const MESSAGE_ID_TEMPLATE = 'no-deep-equal-with-template';
+const MESSAGE_ID_UNDEFINED = 'no-deep-equal-with-undefined';
+
+const fixIs = (fixer, node) => {
+	return fixer.replaceText(node.callee.property, 'is');
+};
+
+const fixNot = (fixer, node) => {
+	return fixer.replaceText(node.callee.property, 'not');
+};
+
+const create = context => {
+	const ava = createAvaRule();
+
+	const callExpression = 'CallExpression';
+	const deepEqual = '[callee.property.name="deepEqual"]';
+	const notDeepEqual = '[callee.property.name="notDeepEqual"]';
+
+	const argumentsLiteral = ':matches([arguments.0.type="Literal"],[arguments.1.type="Literal"])';
+	const argumentsUndefined = ':matches([arguments.0.type="Identifier"][arguments.0.name="undefined"],[arguments.1.type="Identifier"][arguments.1.name="undefined"])';
+	const argumentsTemplate = ':matches([arguments.0.type="TemplateLiteral"],[arguments.1.type="TemplateLiteral"])';
+
+	return ava.merge({
+		[`${callExpression}${deepEqual}${argumentsLiteral}`]: visitIf([
+			ava.isInTestFile,
+			ava.isInTestNode
+		])(node => {
+			context.report({
+				node,
+				messageId: MESSAGE_ID_LITERAL,
+				data: {
+					callee: node.callee.property.name
+				},
+				fix: fixer => fixIs(fixer, node)
+			});
+		}),
+		[`${callExpression}${deepEqual}${argumentsUndefined}`]: visitIf([
+			ava.isInTestFile,
+			ava.isInTestNode
+		])(node => {
+			context.report({
+				node,
+				messageId: MESSAGE_ID_UNDEFINED,
+				data: {
+					callee: node.callee.property.name
+				},
+				fix: fixer => fixIs(fixer, node)
+			});
+		}),
+		[`${callExpression}${deepEqual}${argumentsTemplate}`]: visitIf([
+			ava.isInTestFile,
+			ava.isInTestNode
+		])(node => {
+			context.report({
+				node,
+				messageId: MESSAGE_ID_TEMPLATE,
+				data: {
+					callee: node.callee.property.name
+				},
+				fix: fixer => fixIs(fixer, node)
+			});
+		}),
+		[`${callExpression}${notDeepEqual}${argumentsLiteral}`]: visitIf([
+			ava.isInTestFile,
+			ava.isInTestNode
+		])(node => {
+			context.report({
+				node,
+				messageId: MESSAGE_ID_LITERAL,
+				data: {
+					callee: node.callee.property.name
+				},
+				fix: fixer => fixNot(fixer, node)
+			});
+		}),
+		[`${callExpression}${notDeepEqual}${argumentsUndefined}`]: visitIf([
+			ava.isInTestFile,
+			ava.isInTestNode
+		])(node => {
+			context.report({
+				node,
+				messageId: MESSAGE_ID_UNDEFINED,
+				data: {
+					callee: node.callee.property.name
+				},
+				fix: fixer => fixNot(fixer, node)
+			});
+		}),
+		[`${callExpression}${notDeepEqual}${argumentsTemplate}`]: visitIf([
+			ava.isInTestFile,
+			ava.isInTestNode
+		])(node => {
+			context.report({
+				node,
+				messageId: MESSAGE_ID_TEMPLATE,
+				data: {
+					callee: node.callee.property.name
+				},
+				fix: fixer => fixNot(fixer, node)
+			});
+		})
+	});
+};
+
+module.exports = {
+	create,
+	meta: {
+		docs: {
+			url: util.getDocsUrl(__filename)
+		},
+		fixable: true,
+		messages: {
+			[MESSAGE_ID_LITERAL]: 'Avoid using `{{callee}}` with literals',
+			[MESSAGE_ID_TEMPLATE]: 'Avoid using `{{callee}}` with templates',
+			[MESSAGE_ID_UNDEFINED]: 'Avoid using `{{callee}}` with `undefined`'
+		},
+		type: 'suggestion'
+	}
+};