feature: @putout/engine-reporter: get rid of mock-require

Author: coderaiser

packages/engine-reporter/lib/formatter/formatter.cjs

@@ -1,7 +1,7 @@
 'use strict';
 
-const {simpleImport} = require('putout/simple-import');
-const {createAsyncLoader} = require('@putout/engine-loader');
+const {simpleImport: _simpleImport} = require('putout/simple-import');
+const {createAsyncLoader: _createAsyncLoader} = require('@putout/engine-loader');
 
 const tryToCatch = require('try-to-catch');
 const {
@@ -12,7 +12,12 @@ const {
 const {isArray} = Array;
 const maybeArray = (a) => isArray(a) ? a : [a, {}];
 
-module.exports.getFormatter = async (formatterOptional, exit) => {
+module.exports.getFormatter = async (formatterOptional, exit, overrides = {}) => {
+    const {
+        createAsyncLoader = _createAsyncLoader,
+        simpleImport = _simpleImport,
+    } = overrides;
+    
     const [formatterName = 'none', formatterOptions] = maybeArray(formatterOptional);
     const loadFormatter = createAsyncLoader('formatter');
 

packages/engine-reporter/lib/formatter/formatter.spec.cjs

@@ -1,7 +1,6 @@
 'use strict';
 
 const {test, stub} = require('supertape');
-const mockRequire = require('mock-require');
 
 const {
     NO_FORMATTER,
@@ -10,8 +9,6 @@ const {
 
 const {getFormatter} = require('./formatter.cjs');
 
-const {reRequire, stopAll} = mockRequire;
-
 test('putout: cli: formatter: get formatter', async (t) => {
     const exit = stub();
 
@@ -53,15 +50,10 @@ test('putout: cli: formatter: get formatter: options', async (t) => {
 
 test('putout: cli: formatter: get reporter: exit: NO_FORMATTER', async (t) => {
     const exit = stub();
-    const {getFormatter} = reRequire('./formatter.cjs');
-    
     await getFormatter('xxx', exit);
 
     const expected = [NO_FORMATTER, Error(`Cannot find package 'putout-formatter-xxx'`)];
 
-    stopAll();
-    reRequire('./formatter.cjs');
-    
     t.calledWith(exit, expected, 'should call exit');
     t.end();
 });
@@ -70,18 +62,12 @@ test('putout: cli: formatter: get reporter: exit: CANNOT_LOAD_FORMATTER', async
     const exit = stub();
     const createAsyncLoader = stub().returns(stub().rejects(Error('@putout/formatter-xxx: Syntax error')));
 
-    mockRequire('@putout/engine-loader', {
+    await getFormatter('xxx', exit, {
         createAsyncLoader,
     });
 
-    const {getFormatter} = reRequire('./formatter.cjs');
-    await getFormatter('xxx', exit);
-    
     const expected = [CANNOT_LOAD_FORMATTER, Error(`@putout/formatter-xxx: Syntax error`)];
 
-    stopAll();
-    reRequire('./formatter.cjs');
-    
     t.calledWith(exit, expected, 'should call exit');
     t.end();
 });
@@ -90,18 +76,12 @@ test('putout: cli: formatter: get reporter: pass load', async (t) => {
     const exit = stub();
     const simpleImport = stub().rejects(Error('simple import'));
 
-    mockRequire('putout/simple-import', {
+    await getFormatter('xxx', exit, {
         simpleImport,
     });
 
-    const {getFormatter} = reRequire('./formatter.cjs');
-    await getFormatter('xxx', exit);
-    
     const expected = [CANNOT_LOAD_FORMATTER, Error(`@putout/formatter-xxx: simple import`)];
 
-    stopAll();
-    reRequire('./formatter.cjs');
-    
     t.calledWith(exit, expected, 'should call exit');
     t.end();
 });

packages/engine-reporter/package.json

@@ -49,7 +49,6 @@
     "eslint-plugin-n": "^17.0.0",
     "eslint-plugin-putout": "^29.0.0",
     "madrun": "^11.0.0",
-    "mock-require": "^3.0.3",
     "montag": "^1.0.0",
     "nodemon": "^3.0.1",
     "supertape": "^11.0.3"