@putout/plugin-tape 3.2.0 → 3.5.0

package/README.md

@@ -22,6 +22,7 @@ npm i @putout/plugin-tape -D
         "tape/convert-tape-to-supertape": "on",
         "tape/convert-throws-to-try-catch": "on",
         "tape/convert-does-not-throw-to-try-catch": "on",
+        "tape/convert-called-with-args": "on",
         "tape/convert-called-with-to-called-with-no-args": "on",
         "tape/convert-called-with-no-args-to-called-with": "on",
         "tape/expand-try-catch-arguments": "on",
@@ -29,7 +30,9 @@ npm i @putout/plugin-tape -D
         "tape/convert-emitter-to-promise": "on",
         "tape/convert-ok-to-match": "on",
         "tape/convert-ok-to-called-with": "on",
-        "tape/convert-match-regexp-to-string": "on"
+        "tape/convert-match-regexp-to-string": "on",
+        "tape/declare-stub": "on",
+        "tape/declare-t": "on"
     }
 }
 ```
@@ -45,7 +48,7 @@ test('plugin-apply-destructuring: transform: array: destructuring', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 test('plugin-apply-destructuring: transform: array: destructuring', (t) => {
@@ -62,7 +65,7 @@ test('plugin-apply-destructuring: transform: array: destructuring', (t) => {
 const test = require('tape');
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const test = require('supertape');
@@ -81,7 +84,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const tryCatch = require('try-catch');
@@ -108,7 +111,55 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
+
+```js
+const test = require('supertape');
+const tryCatch = require('try-catch');
+
+test('some test', (t) => {
+    const [error] = tryCatch(copymitter);
+    
+    t.notOk(error, 'should not throw when no args');
+    t.end();
+});
+```
+
+## convert-called-with-args
+
+### ❌ Incorrect code example
+
+```js
+const test = require('supertape');
+const {stub} = test;
+
+test('some message', (t) => {
+    const fn = stub();
+    
+    fn();
+    
+    t.calledWith(fn, 'hello');
+    t.end();
+});
+```
+
+### ✅ Correct code example
+
+```js
+const test = require('supertape');
+const {stub} = test;
+
+test('some message', (t) => {
+    const fn = stub();
+    
+    fn();
+    
+    t.calledWith(fn, ['hello']);
+    t.end();
+});
+```
+
+### ✅ Correct code example
 
 ```js
 const tryCatch = require('try-catch');
@@ -117,7 +168,7 @@ const test = require('supertape');
 test('some message', (t) => {
     const [error] = tryCatch(copymitter);
     
-    t.nottOk(error, 'should throw when no args');
+    t.notOk(error, 'should throw when no args');
     t.end();
 });
 ```
@@ -140,7 +191,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const test = require('supertape');
@@ -174,7 +225,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const test = require('supertape');
@@ -204,7 +255,7 @@ test('copymitter', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const {once} = require('events');
@@ -231,7 +282,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 import tryCatch from 'try-catch';
@@ -254,7 +305,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 test('some message', (t) => {
@@ -277,7 +328,7 @@ test('xxx', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const {test, stub} = require('supertape');
@@ -289,6 +340,24 @@ test('xxx', (t) => {
 });
 ```
 
+## declare-t
+
+### ❌ Incorrect code example
+
+```js
+test('xxx', () => {
+    t.end();
+});
+```
+
+### ✅ Correct code example
+
+```js
+test('xxx', (t) => {
+    t.end();
+});
+```
+
 ## convert-ok-to-match
 
 ### ❌ Incorrect code example
@@ -297,7 +366,7 @@ test('xxx', (t) => {
 t.ok(result.includes('hello'));
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 t.match(result, /hello/);
@@ -311,7 +380,7 @@ t.match(result, /hello/);
 t.ok(set.calledWith(1, 2));
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 t.calledWith(set, [1, 2]);
@@ -325,7 +394,7 @@ t.calledWith(set, [1, 2]);
 t.equal(error, null);
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 t.notOk(error);
@@ -339,7 +408,7 @@ t.notOk(error);
 t.equal(result, true);
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 t.ok(result);
@@ -353,12 +422,29 @@ t.ok(result);
 t.match(result, RegExp('hello'));
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 t.match(result, 'hello');
 ```
 
+## remove-default-messages
+
+`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.
+
+### ❌ Incorrect code example
+
+```js
+t.equal(result, expected, 'should equal');
+```
+
+### ✅ Correct code example
+
+```js
+t.equal(result, expected);
+```
+
 ## License
 
 MIT

package/lib/convert-called-with-args/index.js

@@ -0,0 +1,25 @@
+'use strict';
+
+const {types} = require('putout');
+const {
+    isIdentifier,
+    isArrayExpression,
+} = types;
+
+module.exports.report = () => `Use 'array' as calledWith 'args'`;
+
+module.exports.match = () => ({
+    't.calledWith(__a, __b)': ({__b}) => {
+        if (isIdentifier(__b))
+            return false;
+        
+        if (isArrayExpression(__b))
+            return false;
+        
+        return true;
+    },
+});
+
+module.exports.replace = () => ({
+    't.calledWith(__a, __b)': 't.calledWith(__a, [__b])',
+});

package/lib/convert-called-with-no-args-to-called-with/index.js

@@ -5,5 +5,8 @@ module.exports.report = () => '"calledWithArgs" should be used when arguments ar
 module.exports.replace = () => ({
     't.calledWithNoArgs(__a, __array)': 't.calledWith(__a, __array)',
     't.calledWithNoArgs(__a, __array, __b)': 't.calledWith(__a, __array, __b)',
+    
+    't.calledWithNoArgs(__a, __identifier)': 't.calledWith(__a, __identifier)',
+    't.calledWithNoArgs(__a, __identifier, __b)': 't.calledWith(__a, __identifier, __b)',
 });
 

package/lib/convert-match-regexp-to-string/index.js

@@ -11,16 +11,22 @@ module.exports.match = () => ({
     't.match(__a, RegExp(__b), __c)': checkStr,
     't.match(__a, /__b/)': checkRegExp,
     't.match(__a, /__b/, __c)': checkRegExp,
+    
+    't.notMatch(__a, RegExp(__b))': checkStr,
+    't.notMatch(__a, RegExp(__b), __c)': checkStr,
+    't.notMatch(__a, /__b/)': checkRegExp,
+    't.notMatch(__a, /__b/, __c)': checkRegExp,
 });
 
 module.exports.replace = () => ({
     't.match(__a, RegExp(__b))': 't.match(__a, __b)',
     't.match(__a, RegExp(__b), __c)': 't.match(__a, __b, __c)',
-    't.match(__a, /__b/)': ({__b}) => {
-        return `t.match(__a, '${__b.pattern}')`;
-    },
-    't.match(__a, /__b/, __c)': ({__b}) => {
-        return `t.match(__a, '${__b.pattern}', __c)`;
-    },
+    't.match(__a, /__b/)': ({__b}) => `t.match(__a, '${__b.pattern}')`,
+    't.match(__a, /__b/, __c)': ({__b}) => `t.match(__a, '${__b.pattern}', __c)`,
+    
+    't.notMatch(__a, RegExp(__b))': 't.notMatch(__a, __b)',
+    't.notMatch(__a, RegExp(__b), __c)': 't.notMatch(__a, __b, __c)',
+    't.notMatch(__a, /__b/)': ({__b}) => `t.notMatch(__a, '${__b.pattern}')`,
+    't.notMatch(__a, /__b/, __c)': ({__b}) => `t.notMatch(__a, '${__b.pattern}', __c)`,
 });
 

package/lib/declare-t/index.js

@@ -0,0 +1,19 @@
+'use strict';
+
+const {types} = require('putout');
+const {Identifier} = types;
+
+module.exports.report = () => 'Argument "t" is missing';
+
+module.exports.fix = (path) => {
+    path.scope.block.params = [Identifier('t')];
+};
+
+module.exports.traverse = ({push}) => ({
+    't.end()': (path) => {
+        if (path.scope.bindings.t)
+            return;
+        
+        push(path);
+    },
+});

package/lib/index.js

@@ -10,6 +10,7 @@ module.exports.rules = {
     ...getRule('convert-throws-to-try-catch'),
     ...getRule('convert-does-not-throw-to-try-catch'),
     ...getRule('convert-called-with-to-called-with-no-args'),
+    ...getRule('convert-called-with-args'),
     ...getRule('convert-called-with-no-args-to-called-with'),
     ...getRule('convert-emitter-to-promise'),
     ...getRule('convert-ok-to-match'),
@@ -20,5 +21,7 @@ module.exports.rules = {
     ...getRule('expand-try-catch-arguments'),
     ...getRule('apply-stub-operator'),
     ...getRule('declare-stub'),
+    ...getRule('declare-t'),
+    ...getRule('remove-default-messages'),
 };
 

package/lib/remove-default-messages/index.js

@@ -0,0 +1,15 @@
+'use strict';
+
+module.exports.report = () => 'Avoid passing default messages to operators';
+
+module.exports.replace = () => ({
+    't.ok(__a, "should be truthy")': 't.ok(__a)',
+    't.notOk(__a, "should be falsy")': 't.notOk(__a)',
+    't.match(__a, __b, "should match")': 't.match(__a, __b)',
+    't.notMatch(__a, __b, "should not match")': 't.notMatch(__a, __b)',
+    't.equal(__a, __b, "should equal")': 't.equal(__a, __b)',
+    't.notEqual(__a, __b, "should not equal")': 't.notEqual(__a, __b)',
+    't.deepEqual(__a, __b, "should deep equal")': 't.deepEqual(__a, __b)',
+    't.notDeepEqual(__a, __b, "should not deep equal")': 't.notDeepEqual(__a, __b)',
+});
+

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-tape",
-  "version": "3.2.0",
+  "version": "3.5.0",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "putout plugin helps with tests",
   "homepage": "http://github.com/coderaiser/putout",
@@ -36,7 +36,7 @@
     "c8": "^7.5.0",
     "eslint": "^8.0.0-beta.0",
     "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-putout": "^9.0.0",
+    "eslint-plugin-putout": "^10.0.0",
     "lerna": "^4.0.0",
     "madrun": "^8.0.1",
     "nodemon": "^2.0.1"