@putout/plugin-tape 9.5.0 → 9.8.0

package/README.md

@@ -21,6 +21,7 @@ npm i @putout/plugin-tape -D
 {
     "rules": {
         "tape/apply-stub": "on",
+        "tape/apply-destructuring": "on",
         "tape/apply-stub-operator": "on",
         "tape/apply-with-name": "on",
         "tape/add-t-end": "on",
@@ -55,11 +56,18 @@ npm i @putout/plugin-tape -D
 
 ## switch-expected-with-result
 
+📼**Supertape** uses more natural way of
+comparing: first you pass `result` and then `expected`.
+
+It gives you ability to use value instead of `expected` and
+understand code faster: no need to search for a second argument.
+While `result` is always a variable, so it most likely much shorter.
+
 ### ❌ Example of incorrect code
 
 ```js
 test('plugin-apply-destructuring: transform: array: destructuring', (t) => {
-    t.eqaul(expected, result);
+    t.equal(expected, result);
     t.end();
 });
 ```
@@ -68,7 +76,7 @@ test('plugin-apply-destructuring: transform: array: destructuring', (t) => {
 
 ```js
 test('plugin-apply-destructuring: transform: array: destructuring', (t) => {
-    t.eqaul(result, expected);
+    t.equal(result, expected);
     t.end();
 });
 ```
@@ -334,6 +342,26 @@ test('copymitter', async (t) => {
 });
 ```
 
+## apply-destructuring
+
+Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/adf238850271c0a4ad917e71bab8757c/4ff416c3d9f3de0930339c7a4929eb4b62461d1a).
+
+### ❌ Example of incorrect code
+
+```js
+const test = require('supertape');
+const {stub} = test;
+```
+
+### ✅ Example of correct code
+
+```js
+const {
+    test,
+    stub,
+} = require('supertape');
+```
+
 ## apply-stub
 
 Apply [stub](https://github.com/cloudcmd/stub) functions created. Look how it works in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/d098a8c8d03a072a7f3981e3cfb1d7af/f1542bfa39513ac625b2a9cd9243cc06dee286c0).
@@ -463,7 +491,7 @@ mockImport('fs/promises', {
 });
 ```
 
-### `tape`
+### `test`
 
 #### ❌ Example of incorrect code
 
@@ -604,7 +632,7 @@ t.ok(result.includes('hello'));
 t.match(result, /hello/);
 ```
 
-## convert-ok-to-match
+## convert-ok-to-called-with
 
 ### ❌ Example of incorrect code
 
@@ -686,7 +714,7 @@ t.match(result, 'hello');
 
 ## remove-default-messages
 
-`supertape` will put this information for you, and it is always the same.
+📼**Supertape** will put this information for you, and it is always the same.
 No need to repeat the same information twice on one line, better to avoid it.
 
 ### ❌ Example of incorrect code
@@ -742,7 +770,14 @@ test('some test', (t) => {
 test.skip('some test', (t) => {
     t.end();
 });
+```
 
+### ✅ Example of correct code
+
+```js
+test('some test', (t) => {
+    t.end();
+});
 ```
 
 ## remove-stop-all

package/lib/apply-destructuring/index.js

@@ -0,0 +1,15 @@
+'use strict';
+
+module.exports.report = () => `Use destructuring when using 'stub()' in 'test()'`;
+
+module.exports.match = () => ({
+    'const test = require("supertape")': (vars, path) => path.scope.bindings.stub,
+});
+
+module.exports.replace = () => ({
+    'const test = require("supertape")': (vars, path) => {
+        path.scope.bindings.stub.path.remove();
+        return 'const {test, stub} = require("supertape")';
+    },
+});
+

package/lib/convert-ok-to-match/index.js

@@ -1,6 +1,6 @@
 'use strict';
 
-module.exports.report = () => 't.match should be used instead of t.ok with includes';
+module.exports.report = () => `Use 't.match()' instead of 't.ok()'`;
 
 module.exports.exclude = () => [
     't.ok(keys(__a).includes(__b))',
@@ -9,5 +9,7 @@ module.exports.exclude = () => [
 module.exports.replace = () => ({
     't.ok(__a.includes(__b))': 't.match(__a, __b)',
     't.ok(__a.includes(__b), __c)': 't.match(__a, __b, __c)',
+    't.ok(__a.test(__b))': 't.match(__b, __a)',
+    't.ok(__a.test(__b), __c)': 't.match(__b, __a, __c)',
 });
 

package/lib/index.js

@@ -6,6 +6,7 @@ const getRule = (a) => ({
 
 module.exports.rules = {
     ...getRule('apply-stub'),
+    ...getRule('apply-destructuring'),
     ...getRule('apply-with-name'),
     ...getRule('add-await-to-re-import'),
     ...getRule('sync-with-name'),

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@putout/plugin-tape",
-  "version": "9.5.0",
+  "version": "9.8.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
-  "description": "putout plugin helps with tests",
+  "description": "🐊Putout plugin helps with tests",
   "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-tape#readme",
   "main": "lib/index.js",
   "release": false,
@@ -35,7 +35,7 @@
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
     "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-putout": "^14.0.0",
+    "eslint-plugin-putout": "^15.0.0",
     "lerna": "^4.0.0",
     "madrun": "^9.0.0",
     "montag": "^1.2.1",