@putout/plugin-tape 7.4.0 → 8.0.0

package/README.md

@@ -31,6 +31,7 @@ npm i @putout/plugin-tape -D
         "tape/convert-called-with-to-called-with-no-args": "on",
         "tape/convert-called-with-no-args-to-called-with": "on",
         "tape/convert-equal-to-called-once": "on",
+        "tape/convert-equal-to-deep-equal": "on",
         "tape/convert-deep-equal-to-equal": "on",
         "tape/expand-try-catch-arguments": "on",
         "tape/convert-emitter-to-promise": "on",
@@ -644,6 +645,30 @@ t.equal(result, true);
 t.ok(result);
 ```
 
+## convert-equal-to-deep-equal
+
+### ❌ Incorrect code example
+
+```js
+const expected = {
+    hello: 'world',
+};
+
+t.equal(error, expected);
+t.end();
+```
+
+### ✅ Correct code example
+
+```js
+const expected = {
+    hello: 'world',
+};
+
+t.deepEqual(error, expected);
+t.end();
+```
+
 ## convert-match-regexp-to-string
 
 ### ❌ Incorrect code example

package/lib/convert-equal-to-deep-equal/index.js

@@ -0,0 +1,45 @@
+'use strict';
+
+const {types} = require('putout');
+
+const {
+    isArrayExpression,
+    isIdentifier,
+    isObjectExpression,
+} = types;
+
+module.exports.report = () => `Use 't.deepEqual()' when comparing Objects an Arrays`;
+
+module.exports.match = () => ({
+    't.equal(__a, __b)': check,
+    't.notEqual(__a, __b)': check,
+});
+
+module.exports.replace = () => ({
+    't.equal(__a, __b)': 't.deepEqual(__a, __b)',
+    't.notEqual(__a, __b)': 't.notDeepEqual(__a, __b)',
+});
+
+function check({__b}, path) {
+    if (isObjectExpression(__b))
+        return true;
+    
+    if (isArrayExpression(__b))
+        return true;
+    
+    if (!isIdentifier(__b))
+        return false;
+    
+    const __bDeclaration = path.scope.bindings[__b.name];
+    
+    if (!__bDeclaration)
+        return false;
+    
+    const {id, init} = __bDeclaration.path.node;
+    
+    if (!isIdentifier(id, {name: 'expected'}))
+        return false;
+    
+    return isObjectExpression(init) || isArrayExpression(init);
+}
+

package/lib/convert-equal-to-not-ok/index.js

@@ -19,5 +19,8 @@ module.exports.replace = () => ({
     't.deepEqual(__a, undefined, __b)': 't.notOk(__a, __b)',
     't.deepEqual(__a, false)': 't.notOk(__a)',
     't.deepEqual(__a, false, __b)': 't.notOk(__a, __b)',
+    
+    't.equal(__a)': 't.notOk(__a)',
+    't.notEqual(__a)': 't.ok(__a)',
 });
 

package/lib/index.js

@@ -19,6 +19,7 @@ module.exports.rules = {
     ...getRule('convert-ok-to-called-with'),
     ...getRule('convert-match-regexp-to-string'),
     ...getRule('convert-equal-to-not-ok'),
+    ...getRule('convert-equal-to-deep-equal'),
     ...getRule('convert-equal-to-ok'),
     ...getRule('convert-equal-to-called-once'),
     ...getRule('convert-deep-equal-to-equal'),
@@ -33,7 +34,7 @@ module.exports.rules = {
     ...getRule('add-stop-all'),
     ...getRule('remove-stop-all'),
     ...getRule('remove-useless-t-end'),
-    'remove-only': require('@putout/plugin-remove-only'),
-    'remove-skip': require('@putout/plugin-remove-skip'),
+    ...getRule('remove-only'),
+    ...getRule('remove-skip'),
 };
 

package/lib/remove-only/index.js

@@ -0,0 +1,15 @@
+'use strict';
+
+module.exports.report = () => '"test.only" should not be used';
+
+module.exports.replace = () => ({
+    '__a.only(__b, __c)': '__a(__b, __c)',
+    '__a.only(__b, __c, __d)': '__a(__b, __c, __d)',
+    '__a["only"](__b, __c)': '__a(__b, __c)',
+});
+
+module.exports.filter = (path) => {
+    const {parentPath} = path.parentPath;
+    return parentPath.isProgram();
+};
+

package/lib/remove-skip/index.js

@@ -0,0 +1,15 @@
+'use strict';
+
+module.exports.report = () => '"test.skip" should not be used';
+
+module.exports.replace = () => ({
+    '__a.skip(__b, __c)': '__a(__b, __c)',
+    '__a.skip(__b, __c, __d)': '__a(__b, __c, __d)',
+    '__a["skip"](__b, __c)': '__a(__b, __c)',
+});
+
+module.exports.filter = (path) => {
+    const {parentPath} = path.parentPath;
+    return parentPath.isProgram();
+};
+

package/package.json

@@ -1,9 +1,10 @@
 {
   "name": "@putout/plugin-tape",
-  "version": "7.4.0",
+  "version": "8.0.0",
+  "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "putout plugin helps with tests",
-  "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-tape",
+  "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-tape#readme",
   "main": "lib/index.js",
   "release": false,
   "tag": false,
@@ -22,10 +23,7 @@
     "coverage": "madrun coverage",
     "report": "madrun report"
   },
-  "dependencies": {
-    "@putout/plugin-remove-only": "^5.0.0",
-    "@putout/plugin-remove-skip": "^4.0.0"
-  },
+  "dependencies": {},
   "keywords": [
     "putout",
     "putout-plugin",
@@ -37,14 +35,14 @@
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
     "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-putout": "^12.0.0",
+    "eslint-plugin-putout": "^13.0.0",
     "lerna": "^4.0.0",
     "madrun": "^8.0.1",
     "montag": "^1.2.1",
     "nodemon": "^2.0.1"
   },
   "peerDependencies": {
-    "putout": ">=22.5"
+    "putout": ">=24"
   },
   "license": "MIT",
   "engines": {