fix: @putout/plugin-madrun: add-function: report

Author: coderaiser

packages/engine-loader/test/load-plugins.js

@@ -299,21 +299,27 @@ test('putout: loader: enable part of rule', (t) => {
         const {run} = require('madrun');
         
         module.exports = {
-            'lint': () => 'bin/putout.js .',
-            'fix:lint': () => run('lint', '--fix'),
+            'lint': 'putout .',
         };
     `;
 
     const {places} = putout(code, {
         fix: false,
         rules: {
             'madrun': 'off',
-            'madrun/add-madrun-to-lint': 'on',
+            'madrun/add-function': 'on',
         },
         plugins: ['madrun'],
     });
 
-    const expected = [];
+    const expected = [{
+        message: `Use 'function' instead of 'string' in script: 'lint'`,
+        position: {
+            column: 12,
+            line: 4,
+        },
+        rule: 'madrun/add-function',
+    }];
 
     t.deepEqual(places, expected, 'should disable all but couple of rules in plugin');
     t.end();

packages/plugin-madrun/lib/add-function/index.js

@@ -3,14 +3,13 @@
 const {types, operator} = require('putout');
 
 const {replaceWith, extract} = operator;
-
 const {
     isLiteral,
     isCallExpression,
     arrowFunctionExpression,
 } = types;
 
-module.exports.report = ({name}) => `function should be used instead of string in script "${name}`;
+module.exports.report = ({name}) => `Use 'function' instead of 'string' in script: '${name}'`;
 
 module.exports.fix = ({path}) => {
     replaceWith(path, arrowFunctionExpression([], path.node));

packages/plugin-madrun/lib/add-function/index.spec.js

@@ -10,7 +10,7 @@ const test = createTest(__dirname, {
 });
 
 test('madrun: add function: report: string', (t) => {
-    t.report('string', 'function should be used instead of string in script "hello');
+    t.report('string', `Use 'function' instead of 'string' in script: 'hello'`);
     t.end();
 });
 
@@ -40,6 +40,6 @@ test('madrun: add function: transform: export-default', (t) => {
 });
 
 test('madrun: add function: report: identifier', (t) => {
-    t.report('identifier', 'function should be used instead of string in script "build');
+    t.report('identifier', `Use 'function' instead of 'string' in script: 'build'`);
     t.end();
 });