@putout/test 9.0.1 → 10.0.0

package/README.md

@@ -100,6 +100,8 @@ test('remove usless variables: for-of', (t) => {
 });
 ```
 
+*☝️When input and output the same test fails. Use [`noTransform()`](#notransformfilename) for such cases.*
+
 ### `transformCode(input, output)`
 
 Check transform of `input` -> `output` code:
@@ -372,6 +374,8 @@ test('test: eslint: transform', async ({process}) => {
 });
 ```
 
+*☝️When input and output the same test fails. Use [`noProcess()`](#noprocessfilename--overrides) for such cases.*
+
 ### `noProcess(filename [, overrides])`
 
 Check that filename would not be processed.

package/lib/processor/index.js

@@ -1,12 +1,17 @@
 'use strict';
 
-const {readFile, writeFile} = require('fs/promises');
+const tryToCatch = require('try-to-catch');
+const {
+    readFile,
+    writeFile,
+    unlink,
+} = require('node:fs/promises');
 
 const {
     join,
     extname,
     basename,
-} = require('path');
+} = require('node:path');
 
 const test = require('supertape');
 const processFile = require('putout/process-file');
@@ -20,6 +25,16 @@ const update = async (a, b) => {
     await write(a, b);
 };
 
+const remove = async (a) => {
+    const remove = global.unlink || unlink;
+    await tryToCatch(remove, a);
+};
+
+const read = async (a, b) => {
+    const read = global.readFile || readFile;
+    return await read(a, b);
+};
+
 const buildOptions = ({options, plugins, processors}) => ({
     ...options,
     plugins: plugins || options.plugins,
@@ -28,6 +43,11 @@ const buildOptions = ({options, plugins, processors}) => ({
 
 const addDot = (a) => a ? `.${a}` : '';
 
+const fail = (t, message) => {
+    const {__putout_test_fail = t.fail} = global;
+    return __putout_test_fail(message);
+};
+
 module.exports._addDot = addDot;
 
 module.exports.createTest = (dir, options) => {
@@ -40,10 +60,11 @@ module.exports.createTest = (dir, options) => {
 
 const createProcess = (dir, options) => (operator) => async (filename, plugins, processors) => {
     if (!isStr(filename))
-        return operator.fail(`Expected filename to be string!`);
+        return fail(operator, `Expected filename to be string!`);
     
     const {
         processedSource,
+        rawSource,
         output,
     } = await process(
         filename,
@@ -58,6 +79,9 @@ const createProcess = (dir, options) => (operator) => async (filename, plugins,
     if (isUpdate())
         return operator.pass('fixtures updated');
     
+    if (rawSource === processedSource)
+        return fail(operator, `'input' === 'output', use 'noProcess()'`);
+    
     return operator.equal(processedSource, output, 'fixtures should equal');
 };
 
@@ -98,16 +122,24 @@ const createComparePlaces = (dir, options) => (operator) => async (filename, exp
 
 module.exports._createComparePlaces = createComparePlaces;
 
-async function process(filename, dir, {printer, processors, plugins, extension, fix = true, noChange = false, processorRunners}) {
+async function process(filename, dir, config) {
+    let {extension} = config;
+    const {
+        printer,
+        processors,
+        plugins,
+        fix = true,
+        noChange = false,
+        processorRunners,
+    } = config;
+    
     extension = addDot(extname(filename).slice(1) || extension);
     filename = basename(filename, String(extension));
     
     const inputName = join(dir, 'fixture', `${filename}${extension}`);
     const outputName = join(dir, 'fixture', `${filename}-fix${extension}`);
     
-    const rawSource = await readFile(inputName, 'utf8');
-    const output = !fix || noChange ? '' : await readFile(outputName, 'utf8');
-    
+    const rawSource = await read(inputName, 'utf8');
     const options = {
         dir,
         processors,
@@ -129,8 +161,13 @@ async function process(filename, dir, {printer, processors, plugins, extension,
         processorRunners,
     });
     
-    if (isUpdate() && !noChange && fix)
-        await update(outputName, processedSource);
+    if (isUpdate())
+        if (!noChange && fix)
+            await update(outputName, processedSource);
+        else if (noChange)
+            await remove(outputName);
+    
+    const output = !fix || noChange ? '' : await read(outputName, 'utf8');
     
     return {
         processedSource,

package/lib/test.js

@@ -36,6 +36,11 @@ global.__putout_test_fs = {
 const isUpdate = () => Boolean(Number(process.env.UPDATE));
 const getPrinter = () => process.env.PUTOUT_PRINTER;
 
+const fail = (t, message) => {
+    const {__putout_test_fail = t.fail} = global;
+    return __putout_test_fail(message);
+};
+
 const TS = {
     ENABLED: true,
     DISABLED: false,
@@ -344,6 +349,9 @@ const transform = currify((dir, options, t, name, transformed = null, addons = {
         }],
     });
     
+    if (input === code)
+        return fail(t, `'input' === 'output', use 'noTransform()'`);
+    
     if (isUpdate() && !isStr) {
         writeFixture({
             full,
@@ -553,7 +561,6 @@ const noReportWithOptions = currify((dir, options, t, name, ruleOptions) => {
     rmFixture(`${full}-fix`);
     
     const rule = parseRule(options);
-    
     const rules = {
         [rule]: ['on', ruleOptions],
     };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/test",
-  "version": "9.0.1",
+  "version": "10.0.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "Test runner for 🐊Putout plugins ",
@@ -66,8 +66,8 @@
     "@putout/plugin-remove-empty": "*",
     "@putout/plugin-remove-unused-variables": "*",
     "c8": "^9.0.0",
-    "eslint": "^9.0.0-alpha.0",
-    "eslint-plugin-n": "^17.0.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",