feature: @putout/cli-process-file: get rid of mock-require

Author: coderaiser

packages/cli-process-file/lib/process-file.js

@@ -2,11 +2,11 @@
 
 const process = require('node:process');
 const tryToCatch = require('try-to-catch');
-const eslint = require('@putout/eslint');
+const _eslint = require('@putout/eslint');
 const {parseMatch} = require('putout/parse-match');
 const {mergeOptions} = require('putout/merge-options');
 const parseError = require('putout/parse-error');
-const {putoutAsync} = require('putout');
+const {putoutAsync: _putoutAsync} = require('putout');
 const once = require('once');
 
 const {simpleImport: _simpleImport} = require('./simple-import');
@@ -29,6 +29,8 @@ module.exports = ({fix, fixCount, logError, raw}) => async function processFile(
         again,
         env = getEnv(),
         simpleImport = _simpleImport,
+        eslint = _eslint,
+        putoutAsync = _putoutAsync,
     } = overrides;
 
     const {configurePrinter} = await import('./printer/printer.mjs');

packages/cli-process-file/lib/process-file.spec.js

@@ -2,15 +2,13 @@
 
 const montag = require('montag');
 const {test, stub} = require('supertape');
-const mockRequire = require('mock-require');
 
 const parseOptions = require('putout/parse-options');
-const putout = require('putout');
+
 const processFile = require('./process-file');
 
 const {assign} = Object;
 const noop = () => {};
-const {reRequire, stopAll} = mockRequire;
 const {stringify} = JSON;
 
 test('putout: cli: process-file: eslint', async (t) => {
@@ -22,14 +20,10 @@ test('putout: cli: process-file: eslint', async (t) => {
     const log = stub();
     const write = stub();
 
-    mockRequire('@putout/eslint', eslint);
-    
     const options = {
         dir: '.',
     };
 
-    const processFile = reRequire('./process-file');
-    
     const fn = processFile({
         fix,
         log,
@@ -42,10 +36,9 @@ test('putout: cli: process-file: eslint', async (t) => {
         index: 0,
         length: 1,
         options,
+        eslint,
     });
 
-    stopAll();
-    
     const expected = {
         code: source,
         fix,
@@ -65,14 +58,10 @@ test('putout: cli: process-file: ts from preProcessor', async (t) => {
     const log = stub();
     const write = stub();
 
-    mockRequire('@putout/eslint', eslint);
-    
     const options = {
         dir: '.',
     };
 
-    const processFile = reRequire('./process-file');
-    
     const fn = processFile({
         fix,
         log,
@@ -85,10 +74,9 @@ test('putout: cli: process-file: ts from preProcessor', async (t) => {
         index: 0,
         length: 1,
         options,
+        eslint,
     });
 
-    stopAll();
-    
     const expected = {
         code: source,
         fix,
@@ -110,8 +98,6 @@ test('putout: cli: process-file: tsx from preProcessor', async (t) => {
         dir: '.',
     };
 
-    const processFile = reRequire('./process-file');
-    
     const fn = processFile({
         fix,
         log,
@@ -156,10 +142,6 @@ test('putout: cli: process-file: options for inner data', async (t) => {
         plugins: ['putout-config'],
     };
 
-    mockRequire('@putout/eslint', eslint);
-    
-    const processFile = reRequire('./process-file');
-    
     const fn = processFile({
         fix,
         log,
@@ -172,10 +154,9 @@ test('putout: cli: process-file: options for inner data', async (t) => {
         index: 0,
         length: 1,
         options,
+        eslint,
     });
 
-    stopAll();
-    
     const expected = [{
         message: `Use 'String (on/off)' instead of 'Boolean (true/false)'`,
         position: {
@@ -223,14 +204,10 @@ test('putout: cli: process-file: ruler', async (t) => {
         plugins: ['putout-config'],
     };
 
-    mockRequire('@putout/eslint', eslint);
-    
     const ruler = {
         enableAll: true,
     };
 
-    const processFile = reRequire('./process-file');
-    
     const fn = processFile({
         fix,
         log,
@@ -243,11 +220,10 @@ test('putout: cli: process-file: ruler', async (t) => {
         source,
         index: 0,
         length: 1,
+        eslint,
         options,
     });
 
-    stopAll();
-    
     const expected = [{
         message: `Use 'String (on/off)' instead of 'Boolean (true/false)'`,
         position: {
@@ -280,16 +256,10 @@ test('putout: cli: process-file: configurePrinter', async (t) => {
     const log = stub();
     const write = stub();
 
-    mockRequire('putout', {
-        putoutAsync,
-    });
-    
     const options = {
         dir: '.',
     };
 
-    const processFile = reRequire('./process-file');
-    
     const fn = processFile({
         fix,
         log,
@@ -304,10 +274,9 @@ test('putout: cli: process-file: configurePrinter', async (t) => {
         length: 1,
 
         source,
+        putoutAsync,
     });
 
-    stopAll();
-    
     const expected = ['log123("hello")', {
         dir: '.',
         fix: false,
@@ -342,11 +311,6 @@ test('putout: cli: process-file: configurePrinter: overrides', async (t) => {
     const log = stub();
     const write = stub();
 
-    mockRequire('putout', {
-        ...putout,
-        putoutAsync,
-    });
-    
     const options = {
         dir: '.',
         printer: ['putout', {
@@ -356,8 +320,6 @@ test('putout: cli: process-file: configurePrinter: overrides', async (t) => {
         }],
     };
 
-    const processFile = reRequire('./process-file');
-    
     const fn = processFile({
         fix,
         log,
@@ -372,10 +334,9 @@ test('putout: cli: process-file: configurePrinter: overrides', async (t) => {
         length: 1,
 
         source,
+        putoutAsync,
     });
 
-    stopAll();
-    
     const expected = [
         source, {
             dir: '.',
@@ -408,19 +369,10 @@ test('putout: cli: process-file: plugin not found', async (t) => {
     const log = stub();
     const write = stub();
 
-    mockRequire('putout', {
-        ...putout,
-        putoutAsync,
-    });
-    
     const options = {
         dir: '.',
     };
 
-    reRequire('@putout/eslint');
-    
-    const processFile = reRequire('./process-file');
-    
     const fn = processFile({
         fix,
         log,
@@ -435,10 +387,9 @@ test('putout: cli: process-file: plugin not found', async (t) => {
         length: 1,
 
         source,
+        putoutAsync,
     });
 
-    stopAll();
-    
     const expected = {
         code: `__putout_processor_yaml({hello: 'world'});`,
         places: [{
@@ -734,13 +685,6 @@ test('putout: cli: process-file: recursion: infinite loop', async (t) => {
         lint,
     });
 
-    mockRequire('./simple-import', {
-        simpleImport,
-    });
-    
-    reRequire('putout');
-    const processFile = reRequire('./process-file');
-    
     const fn = processFile({
         fix,
         log,
@@ -755,6 +699,7 @@ test('putout: cli: process-file: recursion: infinite loop', async (t) => {
         options: {
             dir: '.',
         },
+        simpleImport,
     });
 
     const expected = {
@@ -776,8 +721,6 @@ test('putout: cli: process-file: recursion: infinite loop', async (t) => {
         }],
     };
 
-    stopAll();
-    
     t.deepEqual(result, expected);
     t.end();
 });

packages/cli-process-file/package.json

@@ -37,15 +37,12 @@
     "cli"
   ],
   "devDependencies": {
-    "@putout/eslint-flat": "^3.0.0",
     "@putout/operate": "^13.0.0",
     "c8": "^10.0.0",
     "eslint": "^9.0.0",
     "eslint-plugin-n": "^17.0.0",
     "eslint-plugin-putout": "^26.0.0",
     "madrun": "^11.0.0",
-    "mock-import": "^4.2.1",
-    "mock-require": "^3.0.3",
     "montag": "^1.2.1",
     "nodemon": "^3.0.1",
     "putout": "*",