feature: @putout/plugin-putout: replace-test-message: no-report-after-transform: add

Author: coderaiser

packages/plugin-conditions/lib/apply-consistent-blocks/index.spec.js

@@ -64,7 +64,7 @@ test('putout: apply-consistent-blocks: no report: else-statement', (t) => {
     t.end();
 });
 
-test('putout: apply-consistent-blocks: no report after transform', (t) => {
+test('putout: apply-consistent-blocks: no report after transform: nested', (t) => {
     t.noReportAfterTransform('nested');
     t.end();
 });

packages/plugin-putout/lib/apply-fixture-name-to-message/fixture/no-report-after-transform-fix.js

@@ -2,3 +2,8 @@ test('compiler: convert-dec-to-hex: transform: hello', (t) => {
     t.noReportAfterTransform('hello');
     t.end();
 });
+
+test('compiler: convert-dec-to-hex: no report after transform: hello', (t) => {
+    t.noReportAfterTransform('hello');
+    t.end();
+});

packages/plugin-putout/lib/apply-fixture-name-to-message/fixture/no-report-after-transform.js

@@ -2,3 +2,8 @@ test('compiler: convert-dec-to-hex: transform', (t) => {
     t.noReportAfterTransform('hello');
     t.end();
 });
+
+test('compiler: convert-dec-to-hex: no report after transform', (t) => {
+    t.noReportAfterTransform('hello');
+    t.end();
+});

packages/plugin-putout/lib/apply-fixture-name-to-message/index.js

@@ -14,6 +14,7 @@ const NAMES = [
     ...FIXTURE,
     'no report',
     'no transform',
+    'no report after transform',
 ];
 
 export const report = () => `Apply 'fixture' name to 'message'`;

packages/plugin-putout/lib/check-replace-code/index.spec.js

@@ -147,7 +147,7 @@ test('plugin-putout: check-replace-code: report: __json: json-left', (t) => {
     t.end();
 });
 
-test('plugin-putout: check-replace-code: no report: after-transform: once', (t) => {
+test('plugin-putout: check-replace-code: no report after transform: after-transform: once', (t) => {
     t.noReportAfterTransform('after-transform');
     t.end();
 });

packages/plugin-putout/lib/replace-test-message/fixture/no-report-after-transform-fix.js

@@ -0,0 +1,9 @@
+test('compiler: convert-dec-to-hex: no report after transform', (t) => {
+    t.noReportAfterTransform('convert-dec-to-hex');
+    t.end();
+});
+
+test('compiler: convert-dec-to-hex: transform', (t) => {
+    t.transform('convert-dec-to-hex');
+    t.end();
+});

packages/plugin-putout/lib/replace-test-message/fixture/no-report-after-transform.js

@@ -0,0 +1,9 @@
+test('compiler: convert-dec-to-hex: transform', (t) => {
+    t.noReportAfterTransform('convert-dec-to-hex');
+    t.end();
+});
+
+test('compiler: convert-dec-to-hex: no report after transform', (t) => {
+    t.transform('convert-dec-to-hex');
+    t.end();
+});

packages/plugin-putout/lib/replace-test-message/index.js

@@ -13,10 +13,11 @@ export const fix = ({path, incorrect, correct}) => {
 };
 
 const INCORRECT = {
-    TRANSFORM: /: (no transform|report|no report)/,
-    NO_TRANSFORM: /: (transform|report|no report)/,
-    REPORT: /: (no report|transform|no transform)/,
-    NO_REPORT: /: (report|transform|no transform)/,
+    TRANSFORM: /: (no report after transform|no transform|report|no report)/,
+    NO_TRANSFORM: /: (no report after transform|transform|report|no report)/,
+    REPORT: /: (no report after transform|no report|transform|no transform)/,
+    NO_REPORT: /: (no report after transform|report|transform|no transform)/,
+    NO_REPORT_AFTER_TRANSFORM: /: (report|transform|no transform|no report)/,
 };
 
 export const traverse = ({push}) => ({
@@ -45,12 +46,18 @@ export const traverse = ({push}) => ({
         incorrect: INCORRECT.NO_REPORT,
         correct: ': no report',
     }),
+    't.noReportAfterTransform(__a)': convert({
+        push,
+        incorrect: INCORRECT.NO_REPORT_AFTER_TRANSFORM,
+        correct: ': no report after transform',
+    }),
 });
 
 const convert = ({push, correct, incorrect}) => (path) => {
     const [is, messagePath] = isCorrect({
         incorrect,
         path,
+        correct,
     });
 
     if (is)
@@ -66,7 +73,7 @@ const convert = ({push, correct, incorrect}) => (path) => {
 
 const CORRECT = true;
 
-function isCorrect({path, incorrect}) {
+function isCorrect({path, correct, incorrect}) {
     const calleePath = path.findParent(isCallExpression);
 
     if (!calleePath)
@@ -78,6 +85,10 @@ function isCorrect({path, incorrect}) {
         return [CORRECT];
 
     const {value} = messagePath.node;
+    
+    if (value.includes(correct))
+        return [CORRECT];
+    
     const is = !incorrect.test(value);
 
     return [is, messagePath];

packages/plugin-putout/lib/replace-test-message/index.spec.js

@@ -51,3 +51,8 @@ test('plugin-putout: replace-test-message: transform: transform-with-plugins', (
     t.transform('transform-with-plugins');
     t.end();
 });
+
+test('plugin-putout: replace-test-message: transform: no-report-after-transform', (t) => {
+    t.transform('no-report-after-transform');
+    t.end();
+});

packages/plugin-regexp/lib/optimize/index.spec.js

@@ -40,12 +40,12 @@ test('plugin-regexp/optimize: transform: replace-all', (t) => {
     t.end();
 });
 
-test('plugin-regexp/optimize: escape', (t) => {
+test('plugin-regexp/optimize: no transform: escape', (t) => {
     t.noTransform('escape');
     t.end();
 });
 
-test('plugin-regexp/optimize: false positive after fix', (t) => {
+test('plugin-regexp/optimize: no-report-after-transform: after-fix', (t) => {
     t.noReportAfterTransform('after-fix');
     t.end();
 });