@putout/plugin-tape 9.2.0 → 9.4.0

package/README.md

@@ -20,6 +20,7 @@ npm i @putout/plugin-tape -D
 ```json
 {
     "rules": {
+        "tape/apply-stub": "on",
         "tape/apply-stub-operator": "on",
         "tape/apply-with-name": "on",
         "tape/add-t-end": "on",
@@ -333,6 +334,32 @@ test('copymitter', async (t) => {
 });
 ```
 
+## 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).
+
+### ❌ Example of incorrect code
+
+```js
+const a = async () => true;
+const b = async () => {};
+
+const c = async () => throw Error('hello');
+const d = async () => {
+    throw Error('hello');
+};
+```
+
+### ✅ Example of correct code
+
+```js
+const a = stub().resolves(true);
+const b =  stub().resolves();
+
+const c = stub().rejects(Error('hello'));
+const d = stub().rejects(Error('hello'));
+```
+
 ## apply-stub-operator
 
 ### ❌ Example of incorrect code

package/lib/apply-stub/index.js

@@ -0,0 +1,54 @@
+'use strict';
+
+const {
+    template,
+    types,
+} = require('putout');
+
+const {
+    isUnaryExpression,
+    isThrowStatement,
+    isBlockStatement,
+} = types;
+
+const REJECTS = template('stub().rejects(A)');
+
+module.exports.report = () => `Use 'stub()' instead of creating a function`;
+
+module.exports.match = () => ({
+    'async () => __a': ({__a}, path) => {
+        if (!path.parentPath.isVariableDeclarator())
+            return false;
+        
+        if (!isBlockStatement(__a))
+            return true;
+        
+        if (!__a.body.length)
+            return true;
+        
+        return isThrowStatement(__a.body[0]);
+    },
+});
+
+module.exports.replace = () => ({
+    'async () => __a': ({__a}) => {
+        if (isUnaryExpression(__a, {operator: 'throw'})) {
+            return REJECTS({
+                A: __a.argument,
+            });
+        }
+        
+        if (isBlockStatement(__a) && !__a.body.length) {
+            return 'stub().resolves()';
+        }
+        
+        if (isBlockStatement(__a) && isThrowStatement(__a.body[0])) {
+            return REJECTS({
+                A: __a.body[0].argument,
+            });
+        }
+        
+        return 'stub().resolves(__a)';
+    },
+});
+

package/lib/apply-with-name/index.js

@@ -57,7 +57,17 @@ function checkStub(node, path) {
     const {name} = node;
     const binding = bindings[name];
     
-    return binding;
+    if (!binding)
+        return false;
+    
+    const initPath = binding.path.get('init');
+    const calleePath = initPath.get('callee');
+    const str = initPath.toString();
+    
+    if (calleePath.isMemberExpression() && str.includes('withName'))
+        return false;
+    
+    return true;
 }
 
 function applyWithNameToNode(node, path) {

package/lib/convert-emitter-to-promise/index.js

@@ -60,9 +60,10 @@ function replace({__a}, path) {
     
     const arg = path.get('arguments.1').node;
     arg.async = true;
-    const {body} = arg.body;
     
+    const {body} = arg.body;
     const n = body.length;
+    
     for (let i = 0; i < n; i++) {
         let nodes = [];
         

package/lib/index.js

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

package/lib/remove-stop-all/index.js

@@ -11,12 +11,15 @@ const {
     TEST_ASYNC_SKIP,
 } = require('../test-signatures');
 
-const {traverse} = operator;
+const {
+    traverse,
+    remove,
+} = operator;
 
 module.exports.report = () => `Remove 'stopAll()' when you not calling 'reImport()'`;
 
 module.exports.fix = (path) => {
-    path.remove();
+    remove(path);
 };
 
 module.exports.traverse = ({push}) => ({

package/lib/remove-useless-t-end/index.js

@@ -2,12 +2,15 @@
 
 const {operator} = require('putout');
 
-const {traverse} = operator;
+const {
+    traverse,
+    remove,
+} = operator;
 
 module.exports.report = () => `Avoid useless 't.end()'`;
 
 module.exports.fix = (path) => {
-    path.remove();
+    remove(path);
 };
 
 const TEST = 'test("__a", (t) => __body)';

package/package.json

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