@putout/plugin-tape 19.3.0 → 20.1.0

package/README.md

@@ -1,7 +1,7 @@
 # @putout/plugin-tape [![NPM version][NPMIMGURL]][NPMURL]
 
 [NPMIMGURL]: https://img.shields.io/npm/v/@putout/plugin-tape.svg?style=flat&longCache=true
-[NPMURL]: https://npmjs.org/package/@putout/plugin-tape"npm"
+[NPMURL]: https://npmjs.org/package/@putout/plugin-tape "npm"
 
 > Tape-inspired TAP-compatible simplest high speed test runner with superpowers.
 >
@@ -18,9 +18,6 @@ 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);
@@ -37,7 +34,6 @@ npm i @putout/plugin-tape -D
 - ✅ [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);
@@ -47,7 +43,6 @@ npm i @putout/plugin-tape -D
 - ✅ [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);
@@ -58,15 +53,11 @@ npm i @putout/plugin-tape -D
 ```json
 {
     "rules": {
-        "tape/convert-mock-require-to-mock-import": "off",
         "tape/jest": "on",
         "tape/apply-stub": "on",
         "tape/apply-destructuring": "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",
@@ -89,8 +80,7 @@ npm i @putout/plugin-tape -D
         "tape/remove-default-messages": "on",
         "tape/remove-useless-not-called-args": "on",
         "tape/remove-only": "on",
-        "tape/remove-skip": "on",
-        "tape/remove-stop-all": "on"
+        "tape/remove-skip": "on"
     }
 }
 ```
@@ -175,13 +165,13 @@ test('some message', (t) => {
 ### ✅ Example of correct code
 
 ```js
-const tryCatch = require('try-catch');
+const {tryCatch} = require('try-catch');
 const test = require('supertape');
 
 test('some message', (t) => {
     const [error] = tryCatch(copymitter);
     
-    t.equal(error.message, 'from shoulde be a string!', 'should throw when no args');
+    t.equal(error.message, 'from should be a string!');
     t.end();
 });
 ```
@@ -203,7 +193,7 @@ test('some message', (t) => {
 
 ```js
 const test = require('supertape');
-const tryCatch = require('try-catch');
+const {tryCatch} = require('try-catch');
 
 test('some test', (t) => {
     const [error] = tryCatch(copymitter);
@@ -407,13 +397,12 @@ Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/adf238850
 
 ```js
 const test = require('supertape');
-const {stub} = test;
 ```
 
 ### ✅ Example of correct code
 
 ```js
-const {test, stub} = require('supertape');
+const {test} = require('supertape');
 ```
 
 ## apply-stub
@@ -495,38 +484,7 @@ test('should call init before show', (t) => {
 
 ## declare
 
-### `mockImport`
-
-#### ❌ Example of incorrect code
-
-```js
-import {stub} from 'supertape';
-
-mockImport('fs/promises', {
-    readFile: stub().resolves(''),
-});
-```
-
-##### ✅ Example of correct code
-
-```js
-import {stub} from 'supertape';
-import {createMockImport} from 'mock-import';
-
-const {
-    mockImport,
-    stopAll,
-    reImport,
-} = createMockImport(import.meta.url);
-
-mockImport('fs/promises', {
-    readFile: stub().resolves(''),
-});
-```
-
-### `test`
-
-#### ❌ Example of incorrect code
+### ❌ Example of incorrect code
 
 ```js
 test('xxx', (t) => {
@@ -535,7 +493,7 @@ test('xxx', (t) => {
 });
 ```
 
-#### ✅ Example of correct code
+### ✅ Example of correct code
 
 ```js
 import {test, stub} from 'supertape';
@@ -546,26 +504,6 @@ 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
@@ -600,55 +538,6 @@ test('xxx', (t) => {
 });
 ```
 
-## add-await-to-re-import
-
-### ❌ Example of incorrect code
-
-```js
-test('stop-all: should be called', (t) => {
-    const read = reImport('./read');
-    t.end();
-});
-```
-
-### ✅ Example of correct code
-
-```js
-test('stop-all: should be called', async (t) => {
-    const read = await reImport('./read');
-    t.end();
-});
-```
-
-## add-stop-all
-
-When you write test mocking `ESM` with [`mockImport()`](https://github.com/coderaiser/mock-import#mockimportname-mock) never forget to call [`stopAll()`](https://github.com/coderaiser/mock-import#stopall) when you no longer need it. This leads to bugs in tests which are hard to find, each test should be checked with the one which pass when called alone but fail when called with others.
-
-### ❌ Example of incorrect code
-
-```js
-test('stop-all: should be called', (t) => {
-    mockImport('fs/promises', {
-        readFile: stub(),
-    });
-    
-    t.end();
-});
-```
-
-### ✅ Example of correct code
-
-```js
-test('stop-all: should be called', (t) => {
-    mockImport('fs/promises', {
-        readFile: stub(),
-    });
-    
-    stopAll();
-    t.end();
-});
-```
-
 ## remove-useless-t-end
 
 ### ❌ Example of incorrect code
@@ -853,75 +742,6 @@ test('some test', (t) => {
 });
 ```
 
-## remove-stop-all
-
-When `reImport()` or `reRequire` not called, `stopAll()` is redundant and should be removed.
-
-### ❌ Example of incorrect code
-
-```js
-test('some test', (t) => {
-    stopAll();
-    t.end();
-});
-```
-
-### ✅ Example of correct code
-
-```js
-test('some test', (t) => {
-    t.end();
-});
-```
-
-## convert-mock-import-to-require
-
-Convert [mockRequire](https://github.com/boblauer/mock-require) to [mockImport](https://github.com/coderaiser/mock-import).
-
-### ❌ Example of incorrect code
-
-```js
-const mockRequire = require('mock-require');
-
-const {reRequire, stopAll} = mockRequire;
-
-test('', (t) => {
-    mockRequire('fs/promises', {
-        unlink: stub(),
-    });
-    
-    const fn = reRequire('..');
-    fn();
-    
-    stopAll();
-    t.end();
-});
-```
-
-### ✅ Example of correct code
-
-```js
-import {createMockImport} from 'mock-import';
-
-const {
-    mockImport,
-    reImport,
-    stopAll,
-} = createMockImport(import.meta.url);
-
-test('', async (t) => {
-    mockImport('fs/promises', {
-        unlink: stub(),
-    });
-    
-    const fn = await reImport('..');
-    fn();
-    
-    stopAll();
-    t.end();
-});
-```
-
 ## License
 
 MIT

package/lib/apply-destructuring/index.js

@@ -1,16 +1,5 @@
-import {operator} from 'putout';
-
-const {remove} = operator;
-
-export const report = () => `Use destructuring when using 'stub()' in 'test()'`;
-
-export const match = () => ({
-    'const test = require("supertape")': (vars, path) => path.scope.bindings.stub,
-});
+export const report = () => `Use destructuring when require 'test' -> '{test}'`;
 
 export const replace = () => ({
-    'const test = require("supertape")': (vars, path) => {
-        remove(path.scope.bindings.stub.path);
-        return 'const {test, stub} = require("supertape")';
-    },
+    'const test = require("supertape")': 'const {test} = require("supertape")',
 });

package/lib/convert-deep-equal-to-equal/index.js

@@ -20,7 +20,7 @@ export const report = (path) => {
     if (args.length === 2)
         return `Use 't.equal(${args[0]}, ${args[1]})' instead of '${path}' when comparing with primitive`;
     
-    return `Use 't.equal(${args[0]}, ${args[1]}, ${args[2]})' instead of '${path} when comparin with primitive'`;
+    return `Use 't.equal(${args[0]}, ${args[1]}, ${args[2]})' instead of '${path} when comparing with primitive'`;
 };
 
 const check = ({__b}, path) => {

package/lib/convert-tape-to-supertape/index.js

@@ -1,4 +1,4 @@
-export const report = () => '"supertape" should be used';
+export const report = () => `Use 'supertape' instead of 'tape'`;
 
 export const replace = () => ({
     'const tryTo = __': '',

package/lib/declare/index.js

@@ -3,13 +3,4 @@ export const declare = () => ({
     stub: 'import {stub} from "supertape"',
     Test: 'import {Test} from "supertape"',
     Stub: 'import {Stub} from "supertape"',
-    mockImport: `const {mockImport} = createMockImport(import.meta.url)`,
-    reImport: `const {reImport} = createMockImport(import.meta.url)`,
-    mockRequire: `import mockRequire from 'mock-require'`,
-    reRequire: `const {reRequire} = mockRequire`,
-    stopAll: {
-        esm: `const {stopAll} = createMockImport(import.meta.url)`,
-        commonjs: `const {stopAll} = mockRequire`,
-    },
-    createMockImport: `import {createMockImport} from 'mock-import'`,
 });

package/lib/index.js

@@ -1,7 +1,6 @@
 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';
-import * as addAwaitToReImport from './add-await-to-re-import/index.js';
 import * as jest from './jest/index.js';
 import * as syncWithName from './sync-with-name/index.js';
 import * as convertTapeToSupertape from './convert-tape-to-supertape/index.js';
@@ -19,7 +18,6 @@ import * as convertEqualToDeepEqual from './convert-equal-to-deep-equal/index.js
 import * as convertEqualToOk from './convert-equal-to-ok/index.js';
 import * as convertEqualToCalledOnce from './convert-equal-to-called-once/index.js';
 import * as convertDeepEqualToEqual from './convert-deep-equal-to-equal/index.js';
-import * as convertMockRequireToMockImport from './convert-mock-require-to-mock-import/index.js';
 import * as addArgs from './add-args/index.js';
 import * as declare from './declare/index.js';
 import * as removeDefaultMessages from './remove-default-messages/index.js';
@@ -27,19 +25,15 @@ import * as removeUselessNotCalledArgs from './remove-useless-not-called-args/in
 import * as switchExpectedWithResult from './switch-expected-with-result/index.js';
 import * as addTEnd from './add-t-end/index.js';
 import * as addStopAll from './add-stop-all/index.js';
-import * as removeStopAll from './remove-stop-all/index.js';
 import * as removeUselessTEnd from './remove-useless-t-end/index.js';
 import * as removeOnly from './remove-only/index.js';
 import * as removeSkip from './remove-skip/index.js';
 import * as convertEqualsToEqual from './convert-equals-to-equal/index.js';
-import * as addNodePrefixToMockRequire from './add-node-prefix-to-mock-require/index.js';
 
 export const rules = {
-    'convert-mock-require-to-mock-import': ['off', convertMockRequireToMockImport],
     'apply-stub': applyStub,
     'apply-destructuring': applyDestructuring,
     'apply-with-name': applyWithName,
-    'add-await-to-re-import': addAwaitToReImport,
     'jest': jest,
     'sync-with-name': syncWithName,
     'convert-tape-to-supertape': convertTapeToSupertape,
@@ -65,9 +59,7 @@ export const rules = {
     'switch-expected-with-result': switchExpectedWithResult,
     '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": "19.3.0",
+  "version": "20.1.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with tests",
@@ -31,21 +31,21 @@
   ],
   "devDependencies": {
     "@putout/plugin-putout": "*",
-    "@putout/test": "^14.0.0",
+    "@putout/test": "^15.0.0",
     "c8": "^10.0.0",
-    "eslint": "^9.0.0",
+    "eslint": "^10.0.0-alpha.0",
     "eslint-plugin-n": "^17.0.0",
-    "eslint-plugin-putout": "^28.0.0",
-    "madrun": "^11.0.0",
+    "eslint-plugin-putout": "^30.0.0",
+    "madrun": "^12.0.0",
     "montag": "^1.2.1",
     "nodemon": "^3.0.1"
   },
   "peerDependencies": {
-    "putout": ">=40"
+    "putout": ">=41"
   },
   "license": "MIT",
   "engines": {
-    "node": ">=20"
+    "node": ">=22"
   },
   "publishConfig": {
     "access": "public"

package/lib/add-await-to-re-import/index.js

@@ -1,24 +0,0 @@
-import {types} from 'putout';
-
-const {isFunction} = types;
-
-export const report = () => `Call 'reImport()' using await`;
-
-export const match = () => ({
-    'reImport(__a)': (vars, path) => {
-        const {parentPath} = path;
-        
-        return !parentPath.isAwaitExpression();
-    },
-});
-
-export const replace = () => ({
-    'reImport(__a)': (vars, path) => {
-        const fnPath = path.findParent(isFunction);
-        
-        if (fnPath)
-            fnPath.node.async = true;
-        
-        return 'await reImport(__a)';
-    },
-});

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

@@ -1,38 +0,0 @@
-import {isBuiltin} from 'node:module';
-import {operator, types} from 'putout';
-
-const {isStringLiteral} = types;
-const {
-    getTemplateValues,
-    setLiteralValue,
-} = operator;
-
-export const report = ({__a}) => `Add 'node:' prefix: '${__a.value}' -> 'node:${__a.value}'`;
-
-export const fix = ({__a}) => {
-    setLiteralValue(__a, `node:${__a.value}`);
-};
-
-const MOCK_REQUIRE = 'mockRequire(__a, __b)';
-const MOCK_IMPORT = 'mockImport(__a, __b)';
-
-export const traverse = ({push}) => ({
-    [MOCK_REQUIRE]: createTraverser(push, MOCK_REQUIRE),
-    [MOCK_IMPORT]: createTraverser(push, MOCK_IMPORT),
-});
-
-const createTraverser = (push, template) => (path) => {
-    const {__a} = getTemplateValues(path, template);
-    
-    if (!isStringLiteral(__a))
-        return;
-    
-    if (__a.value.startsWith('node:'))
-        return;
-    
-    if (isBuiltin(__a.value))
-        push({
-            path,
-            __a,
-        });
-};

package/lib/convert-mock-require-to-mock-import/index.js

@@ -1,48 +0,0 @@
-import {operator, template} from 'putout';
-
-const {
-    remove,
-    getPathAfterImports,
-    insertBefore,
-} = operator;
-
-const initStopAll = template.ast(`const {stopAll} = createMockImport(import.meta.url)`);
-
-export const report = () => '"mockImport" should be used instead of "mockRequire"';
-
-export const replace = () => ({
-    'mockRequire(__a, __b)': 'mockImport(__a, __b)',
-    'reRequire(__a)': (vars, path) => {
-        const {scope} = path;
-        const {block} = scope;
-        
-        if (!block.async)
-            block.async = true;
-        
-        if (!scope.hasBinding('createMockImport')) {
-            const programScope = scope.getProgramParent();
-            const programPath = programScope.path;
-            
-            maybeRemoveOldStopAll(scope);
-            
-            const endOfImportPath = getPathAfterImports(programPath.get('body'));
-            insertBefore(endOfImportPath, initStopAll);
-        }
-        
-        return 'await reImport(__a)';
-    },
-});
-
-function maybeRemoveOldStopAll(scope) {
-    const binding = scope.getBinding('stopAll');
-    
-    if (!binding)
-        return;
-    
-    const {path} = binding;
-    
-    if (path) {
-        remove(path);
-        path.scope.crawl();
-    }
-}

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

@@ -1,78 +0,0 @@
-import {operator} from 'putout';
-import {
-    TEST,
-    TEST_ONLY,
-    TEST_SKIP,
-    TEST_ASYNC,
-    TEST_ASYNC_ONLY,
-    TEST_ASYNC_SKIP,
-} from '../test-signatures.js';
-
-const {remove} = operator;
-
-export const report = () => `Remove 'stopAll()' when you not calling 'reImport()'`;
-
-export const fix = (path) => {
-    remove(path);
-};
-
-export const traverse = ({push}) => ({
-    [TEST]: createTraverse(push),
-    [TEST_ONLY]: createTraverse(push),
-    [TEST_SKIP]: createTraverse(push),
-    [TEST_ASYNC]: createTraverse(push),
-    [TEST_ASYNC_ONLY]: createTraverse(push),
-    [TEST_ASYNC_SKIP]: createTraverse(push),
-});
-
-const createTraverse = (push) => (path) => {
-    const {
-        hasReImport,
-        hasMockImport,
-        stopAllPath,
-    } = check(path);
-    
-    if (hasReImport)
-        return;
-    
-    if (hasMockImport)
-        return;
-    
-    if (!stopAllPath)
-        return;
-    
-    push(stopAllPath);
-};
-
-function check(path) {
-    let hasReImport = false;
-    let hasMockImport = false;
-    let stopAllPath = null;
-    
-    operator.traverse(path, {
-        'reImport(__a)': () => {
-            hasReImport = true;
-        },
-        'mockRequire(__a, __b)': () => {
-            hasMockImport = true;
-        },
-        'mockImport(__a, __b)': () => {
-            hasMockImport = true;
-        },
-        'traceImport(__a, __b)': () => {
-            hasReImport = true;
-        },
-        'reRequire(__a)': () => {
-            hasReImport = true;
-        },
-        'stopAll()': (path) => {
-            stopAllPath = path;
-        },
-    });
-    
-    return {
-        hasReImport,
-        hasMockImport,
-        stopAllPath,
-    };
-}