@putout/plugin-tape 14.1.0 → 15.0.0

package/README.md

@@ -17,6 +17,44 @@ npm i @putout/plugin-tape -D
 
 ## Rules
 
+- ✅ [add-args](#add-args);
+- ✅ [add-await-to-re-import](#add-await-to-re-import);
+- ✅ [add-node-prefix-to-mock-require](#add-node-prefix-to-mock-require);
+- ✅ [add-stop-all](#add-stop-all);
+- ✅ [add-t-end](#add-t-end);
+- ✅ [apply-destructuring](#apply-destructuring);
+- ✅ [apply-stub](#apply-stub);
+- ✅ [apply-with-name](#apply-with-name);
+- ✅ [convert-called-with-args](#convert-called-with-args);
+- ✅ [convert-called-with-no-args-to-called-with](#convert-called-with-no-args-to-called-with);
+- ✅ [convert-called-with-to-called-with-no-args](#convert-called-with-to-called-with-no-args);
+- ✅ [convert-deep-equal-to-equal](#convert-deep-equal-to-equal);
+- ✅ [convert-does-not-throw-to-try-catch](#convert-does-not-throw-to-try-catch);
+- ✅ [convert-emitter-to-promise](#convert-emitter-to-promise);
+- ✅ [convert-equal-to-called-once](#convert-equal-to-called-once);
+- ✅ [convert-equal-to-deep-equal](#convert-equal-to-deep-equal);
+- ✅ [convert-equal-to-not-ok](#convert-equal-to-not-ok);
+- ✅ [convert-equal-to-ok](#convert-equal-to-ok);
+- ✅ [convert-equals-to-equal](#convert-equals-to-equal);
+- ✅ [convert-match-regexp-to-string](#convert-match-regexp-to-string);
+- ✅ [convert-mock-require-to-mock-import](#convert-mock-require-to-mock-import);
+- ✅ [convert-ok-to-called-with](#convert-ok-to-called-with);
+- ✅ [convert-ok-to-match](#convert-ok-to-match);
+- ✅ [convert-tape-to-supertape](#convert-tape-to-supertape);
+- ✅ [convert-throws-to-try-catch](#convert-throws-to-try-catch);
+- ✅ [declare](#declare);
+- ✅ [jest](#jest);
+- ✅ [remove-default-messages](#remove-default-messages);
+- ✅ [remove-only](#remove-only);
+- ✅ [remove-skip](#remove-skip);
+- ✅ [remove-stop-all](# [remove-stop-all);
+- ✅ [remove-useless-not-called-args](#remove-useless-not-called-args);
+- ✅ [remove-useless-t-end](#remove-useless-t-end);
+- ✅ [switch-expected-with-result](#switch-expected-with-result);
+- ✅ [sync-with-name](#sync-with-name);
+
+## Config
+
 ```json
 {
     "rules": {
@@ -24,11 +62,11 @@ npm i @putout/plugin-tape -D
         "tape/jest": "on",
         "tape/apply-stub": "on",
         "tape/apply-destructuring": "on",
-        "tape/apply-stub-operator": "on",
         "tape/apply-with-name": "on",
         "tape/add-t-end": "on",
         "tape/add-stop-all": "on",
         "tape/add-await-to-re-import": "on",
+        "tape/add-node-prefix-to-mock-require": "on",
         "tape/remove-useless-t-end": "on",
         "tape/sync-with-name": "on",
         "tape/switch-expected-with-result": "on",
@@ -350,7 +388,7 @@ test('copymitter', (t) => {
 ### ✅ Example of correct code
 
 ```js
-const {once} = require('events');
+const {once} = require('node:events');
 
 test('copymitter', async (t) => {
     const cp = copymitter(from, to, ['1']);
@@ -403,26 +441,6 @@ const c = stub().rejects(Error('hello'));
 const d = stub().rejects(Error('hello'));
 ```
 
-## apply-stub-operator
-
-### ❌ Example of incorrect code
-
-```js
-test('some message', (t) => {
-    t.ok(fn.calledWith(a));
-    t.end();
-});
-```
-
-### ✅ Example of correct code
-
-```js
-test('some message', (t) => {
-    t.calledWith(fn, [a]);
-    t.end();
-});
-```
-
 ## apply-with-name
 
 ### ❌ Example of incorrect code
@@ -528,6 +546,26 @@ test('xxx', (t) => {
 });
 ```
 
+## add-node-prefix-to-mock-require
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/38f43d1972b3ebe37597022b42024cd5/ccd8cf8a3ff4a30c6dccc528358f9439f1fd366e).
+
+### ❌ Example of incorrect code
+
+```js
+mockRequire('fs/promises', {
+    readdir,
+});
+```
+
+### ✅ Example of correct code
+
+```js
+mockRequire('node:fs/promises', {
+    readdir,
+});
+```
+
 ## add-args
 
 ### ❌ Example of incorrect code

package/lib/add-node-prefix-to-mock-require/index.js

@@ -0,0 +1,36 @@
+'use strict';
+
+const {isBuiltin} = require('node:module');
+const {operator, types} = require('putout');
+
+const {isStringLiteral} = types;
+const {
+    getTemplateValues,
+    setLiteralValue,
+} = operator;
+
+module.exports.report = ({__a}) => `Add 'node:' prefix: '${__a.value}' -> 'node:${__a.value}'`;
+
+module.exports.fix = ({__a}) => {
+    setLiteralValue(__a, `node:${__a.value}`);
+};
+
+const MOCK_REQUIRE = 'mockRequire(__a, __b)';
+
+module.exports.traverse = ({push}) => ({
+    [MOCK_REQUIRE](path) {
+        const {__a} = getTemplateValues(path, MOCK_REQUIRE);
+        
+        if (!isStringLiteral(__a))
+            return;
+        
+        if (__a.value.startsWith('node:'))
+            return;
+        
+        if (isBuiltin(__a.value))
+            push({
+                path,
+                __a,
+            });
+    },
+});

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

@@ -36,7 +36,9 @@ module.exports.replace = () => ({
 
 function match({__body}) {
     for (const codeLine of __body.body) {
-        const {expression = codeLine} = codeLine;
+        const {
+            expression = codeLine,
+        } = codeLine;
         
         if (compare(expression, ON_END))
             return true;
@@ -64,7 +66,9 @@ function replace({__a}, path) {
     for (let i = 0; i < n; i++) {
         let nodes = [];
         const current = body[i];
-        const {expression = current} = current;
+        const {
+            expression = current,
+        } = current;
         
         if (compare(expression, ON_ERROR))
             nodes = getErrorNodes(expression);

package/lib/convert-ok-to-called-with/index.js

@@ -1,10 +1,22 @@
 'use strict';
 
-module.exports.report = () => `Use 't.calledWith()' instead of 't.ok()'`;
+const {operator} = require('putout');
+const {compare} = operator;
+
+module.exports.report = (path) => {
+    if (compare(path, 't.ok(__args)'))
+        return `Use 't.calledWith()' instead of 't.ok()'`;
+    
+    return `Use 't.notCalledWith()' instead of 't.notOk()'`;
+};
 
 module.exports.replace = () => ({
     't.ok(__a.calledWith(__args))': 't.calledWith(__a, [__args])',
     't.ok(__a.calledWith(__args), __b)': 't.calledWith(__a, [__args], __b)',
+    
     't.ok(__a.calledWith(...__b))': 't.calledWith(__a, __b)',
     't.ok(__a.calledWith(...__b), __c)': 't.calledWith(__a, __b, __c)',
+    
+    't.notOk(__a.called, __b)': 't.notCalled(__a, __b)',
+    't.ok(__a.calledWith(), __b)': 't.calledWithNoArgs(__a, __b)',
 });

package/lib/index.js

@@ -27,7 +27,6 @@ const declare = require('./declare');
 const removeDefaultMessages = require('./remove-default-messages');
 const removeUselessNotCalledArgs = require('./remove-useless-not-called-args');
 const switchExpectedWithResult = require('./switch-expected-with-result');
-const applyStubOperator = require('./apply-stub-operator');
 const addTEnd = require('./add-t-end');
 const addStopAll = require('./add-stop-all');
 const removeStopAll = require('./remove-stop-all');
@@ -35,6 +34,7 @@ const removeUselessTEnd = require('./remove-useless-t-end');
 const removeOnly = require('./remove-only');
 const removeSkip = require('./remove-skip');
 const convertEqualsToEqual = require('./convert-equals-to-equal');
+const addNodePrefixToMockRequire = require('./add-node-prefix-to-mock-require');
 
 module.exports.rules = {
     'convert-mock-require-to-mock-import': ['off', convertMockRequireToMockImport],
@@ -66,11 +66,11 @@ module.exports.rules = {
     'remove-default-messages': removeDefaultMessages,
     'remove-useless-not-called-args': removeUselessNotCalledArgs,
     'switch-expected-with-result': switchExpectedWithResult,
-    'apply-stub-operator': applyStubOperator,
     'add-t-end': addTEnd,
     'add-stop-all': addStopAll,
     'remove-stop-all': removeStopAll,
     'remove-useless-t-end': removeUselessTEnd,
     'remove-only': removeOnly,
     'remove-skip': removeSkip,
+    'add-node-prefix-to-mock-require': addNodePrefixToMockRequire,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-tape",
-  "version": "14.1.0",
+  "version": "15.0.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with tests",
@@ -32,10 +32,10 @@
   ],
   "devDependencies": {
     "@putout/plugin-putout": "*",
-    "@putout/test": "^9.0.0",
-    "c8": "^9.0.0",
-    "eslint": "^9.0.0-alpha.0",
-    "eslint-plugin-n": "^17.0.0-0",
+    "@putout/test": "^11.0.0",
+    "c8": "^10.0.0",
+    "eslint": "^9.0.0",
+    "eslint-plugin-n": "^17.0.0",
     "eslint-plugin-putout": "^22.0.0",
     "lerna": "^6.0.1",
     "madrun": "^10.0.0",
@@ -43,11 +43,11 @@
     "nodemon": "^3.0.1"
   },
   "peerDependencies": {
-    "putout": ">=35"
+    "putout": ">=36"
   },
   "license": "MIT",
   "engines": {
-    "node": ">=18"
+    "node": ">=18.6"
   },
   "publishConfig": {
     "access": "public"

package/lib/apply-stub-operator/index.js

@@ -1,14 +0,0 @@
-'use strict';
-
-module.exports.report = () => 'Use Stub operator';
-
-module.exports.replace = () => ({
-    't.ok(__a.calledWith(__b))': 't.calledWith(__a, [__b])',
-    't.ok(__a.calledWith(__b, __c))': 't.calledWith(__a, [__b, __c])',
-    't.ok(__a.calledWith(...__b))': 't.calledWith(__a, __b)',
-    't.ok(__a.calledWith(__b), __c)': 't.calledWith(__a, [__b], __c)',
-    't.ok(__a.calledWith(__b, __c), __d)': 't.calledWith(__a, [__b, __c], __d)',
-    't.ok(__a.calledWith(...__b), __c)': 't.calledWith(__a, __b, __c)',
-    't.notOk(__a.called, __b)': 't.notCalled(__a, __b)',
-    't.ok(__a.calledWith(), __b)': 't.calledWithNoArgs(__a, __b)',
-});