@putout/plugin-tape 3.3.0 → 3.6.0

package/README.md

@@ -30,7 +30,11 @@ 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",
+        "tape/remove-default-messages": "on",
+        "tape/remove-useless-not-called-args": "on"
     }
 }
 ```
@@ -46,7 +50,7 @@ test('plugin-apply-destructuring: transform: array: destructuring', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 test('plugin-apply-destructuring: transform: array: destructuring', (t) => {
@@ -63,7 +67,7 @@ test('plugin-apply-destructuring: transform: array: destructuring', (t) => {
 const test = require('tape');
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const test = require('supertape');
@@ -82,7 +86,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const tryCatch = require('try-catch');
@@ -109,6 +113,20 @@ test('some message', (t) => {
 });
 ```
 
+### ✅ 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
@@ -127,7 +145,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const test = require('supertape');
@@ -143,7 +161,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const tryCatch = require('try-catch');
@@ -175,7 +193,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const test = require('supertape');
@@ -209,7 +227,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const test = require('supertape');
@@ -239,7 +257,7 @@ test('copymitter', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const {once} = require('events');
@@ -266,7 +284,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 import tryCatch from 'try-catch';
@@ -289,7 +307,7 @@ test('some message', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 test('some message', (t) => {
@@ -312,7 +330,7 @@ test('xxx', (t) => {
 });
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 const {test, stub} = require('supertape');
@@ -324,6 +342,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
@@ -332,7 +368,7 @@ test('xxx', (t) => {
 t.ok(result.includes('hello'));
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 t.match(result, /hello/);
@@ -346,7 +382,7 @@ t.match(result, /hello/);
 t.ok(set.calledWith(1, 2));
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 t.calledWith(set, [1, 2]);
@@ -360,7 +396,7 @@ t.calledWith(set, [1, 2]);
 t.equal(error, null);
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 t.notOk(error);
@@ -374,7 +410,7 @@ t.notOk(error);
 t.equal(result, true);
 ```
 
-### ✅ Correct code Example
+### ✅ Correct code example
 
 ```js
 t.ok(result);
@@ -388,12 +424,43 @@ 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);
+```
+
+## remove-useless-not-called-args
+
+### ❌ Incorrect code example
+
+```js
+t.notCalled(fn, []);
+```
+
+### ✅ Correct code example
+
+```js
+t.notCalled(fn);
+```
+
 ## License
 
 MIT

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

@@ -21,5 +21,8 @@ module.exports.rules = {
     ...getRule('expand-try-catch-arguments'),
     ...getRule('apply-stub-operator'),
     ...getRule('declare-stub'),
+    ...getRule('declare-t'),
+    ...getRule('remove-default-messages'),
+    ...getRule('remove-useless-not-called-args'),
 };
 

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/lib/remove-useless-not-called-args/index.js

@@ -0,0 +1,8 @@
+'use strict';
+
+module.exports.report = () => 'Remove useless "notCalled" args';
+
+module.exports.replace = () => ({
+    't.notCalled(__a, __array)': 't.notCalled(__a)',
+});
+

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-tape",
-  "version": "3.3.0",
+  "version": "3.6.0",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "putout plugin helps with tests",
   "homepage": "http://github.com/coderaiser/putout",