@putout/plugin-tape 21.9.1 → 21.11.0

package/README.md

@@ -42,6 +42,7 @@ npm i @putout/plugin-tape -D
 - ✅ [convert-throws-to-try-catch](#convert-throws-to-try-catch);
 - ✅ [declare](#declare);
 - ✅ [extract-args-from-called-with](#extract-args-from-called-with);
+- ✅ [extract-result-from-assertion](#extract-result-from-assertion);
 - ✅ [jest](#jest);
 - ✅ [remove-default-messages](#remove-default-messages);
 - ✅ [remove-only](#remove-only);
@@ -49,9 +50,9 @@ npm i @putout/plugin-tape -D
 - ✅ [remove-useless-not-called-args](#remove-useless-not-called-args);
 - ✅ [remove-useless-t-end](#remove-useless-t-end);
 - ✅ [remove-useless-undefined](#remove-useless-undefined);
+- ✅ [remove-t-from-async](#remove-t-from-async);
 - ✅ [switch-expected-with-result](#switch-expected-with-result);
 - ✅ [sync-with-name](#sync-with-name);
-- ✅ [move-out-result-from-assertion](#move-out-result-from-assertion);
 
 ## Config
 
@@ -88,13 +89,14 @@ npm i @putout/plugin-tape -D
         "tape/remove-default-messages": "on",
         "tape/remove-useless-not-called-args": "on",
         "tape/remove-useless-undefined": "on",
+        "tape/remove-t-from-async": "on",
         "tape/remove-only": ["on", {
             "allowed": ["test"]
         }],
         "tape/remove-skip": ["on", {
             "allowed": ["test"]
         }],
-        "tape/move-out-result-from-assertion": "on"
+        "tape/extract-result-from-assertion": "on"
     }
 }
 ```
@@ -640,6 +642,38 @@ test('test: remove me', () => {
 });
 ```
 
+## remove-t-from-async
+
+### ❌ Example of incorrect code
+
+```js
+test('sqlite: all returns empty array', async (t) => {
+    const db = await createDb();
+    await db.exec('CREATE TABLE t (id INTEGER PRIMARY KEY, x TEXT)');
+    const rows = await db.all('SELECT x FROM t WHERE x = :x', {
+        x: 'missing',
+    });
+    
+    await t.dbAll(rows, []);
+    
+    t.end();
+});
+```
+
+### ✅ Example of correct code
+
+```js
+test('sqlite: all returns empty array', async ({dbAll}) => {
+    const db = await createDb();
+    await db.exec('CREATE TABLE t (id INTEGER PRIMARY KEY, x TEXT)');
+    const rows = await db.all('SELECT x FROM t WHERE x = :x', {
+        x: 'missing',
+    });
+    
+    await dbAll(rows, []);
+});
+```
+
 ## remove-useless-undefined
 
 ### ❌ Example of incorrect code
@@ -841,7 +875,7 @@ test('some test', (t) => {
 });
 ```
 
-## move-out-result-from-assertion
+## extract-result-from-assertion
 
 Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/264bc87f56eae9aa2b2884ff64fe7a06/94836a115a072a18fd717d5454792c1895ab0476).
 

package/lib/{move-out-result-from-assertion → extract-result-from-assertion}/index.js

@@ -7,9 +7,10 @@ const {
     isMemberExpression,
     isOptionalMemberExpression,
     isObjectExpression,
+    isArrayExpression,
 } = types;
 
-export const report = () => `Move out result from assertion`;
+export const report = () => `Extract result from assertion`;
 export const match = () => ({
     't.__(__a(__b))': (vars, path) => !getBinding(path, 'result'),
     't.__a(__b, __c)': ({__a, __b, __c}, path) => {
@@ -37,6 +38,9 @@ export const match = () => ({
         if (expectedBinding)
             return false;
         
+        if (isArrayExpression(__c))
+            return true;
+        
         return isObjectExpression(__c);
     },
 });

package/lib/index.js

@@ -1,8 +1,9 @@
+import * as removeTFromAsync from './remove-t-from-async/index.js';
 import * as applyStringify from './apply-stringify/index.js';
-import * as movOutResultFromAssertion from './move-out-result-from-assertion/index.js';
 import * as applyAssertionsOrder from './apply-assertions-order/index.js';
-import * as removeUselessUndefined from './remove-useless-undefined/index.js';
 import * as extractArgsFromCalledWith from './extract-args-from-called-with/index.js';
+import * as extractResultFromAssertion from './extract-result-from-assertion/index.js';
+import * as removeUselessUndefined from './remove-useless-undefined/index.js';
 import * as applyStub from './apply-stub/index.js';
 import * as applyDestructuring from './apply-destructuring/index.js';
 import * as applyWithName from './apply-with-name/index.js';
@@ -68,8 +69,9 @@ export const rules = {
     'remove-only': removeOnly,
     'remove-skip': removeSkip,
     'extract-args-from-called-with': extractArgsFromCalledWith,
+    'extract-result-from-assertion': extractResultFromAssertion,
     'remove-useless-undefined': removeUselessUndefined,
     'apply-assertions-order': applyAssertionsOrder,
-    'move-out-result-from-assertion': movOutResultFromAssertion,
     'apply-stringify': applyStringify,
+    'remove-t-from-async': removeTFromAsync,
 };

package/lib/remove-t-from-async/index.js

@@ -0,0 +1,22 @@
+import {types, operator} from 'putout';
+
+const {remove} = operator;
+const {objectProperty, objectPattern} = types;
+
+export const report = () => `Use 'if condition' instead of 'ternary expression'`;
+
+export const replace = () => ({
+    'await t.__a(__args)': ({__a}, path) => {
+        const next = path.parentPath.getNextSibling();
+        
+        path.scope.block.params = [
+            objectPattern([
+                objectProperty(__a, __a),
+            ]),
+        ];
+        
+        remove(next);
+        
+        return 'await __a(__args)';
+    },
+});

package/package.json

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