feature: @putout/test: add ability to override ext

coderaiser · coderaiser · commit 115d824d06db · 2025-04-20T19:11:08.000+03:00
diff --git a/packages/processor-wasm/lib/lint.js b/packages/processor-wasm/lib/lint.js
@@ -4,7 +4,7 @@ import {parse} from '@webassemblyjs/wast-parser';
 import {print} from '@webassemblyjs/wast-printer';
 import {rules} from './rules/index.js';
 
-export const lint = (source, {fix} = {}) => {
+export const lint = (source, {fix = true} = {}) => {
     const [error, ast] = tryCatch(parse, source);
     
     if (error)
diff --git a/packages/processor-wasm/lib/rules/convert-get-local-to-local-get/fixture/convert-get-local-to-local-get-fix.wast b/packages/processor-wasm/lib/rules/convert-get-local-to-local-get/fixture/convert-get-local-to-local-get-fix.wast
@@ -0,0 +1,15 @@
+(module
+  ;; this is simple function that adds a couple of parameters
+
+  (func (param $a i32) (param $b i32)
+    (local.get $a)
+    (local.get $b)
+    (i32.add)
+  )
+  (func (param $a i32) (param $b i32)
+    (i32.add (local.get $a) (local.get $b))
+  )
+  ;; this statement exports the function to the host environment
+
+  (export "add" (func $add))
+)
diff --git a/packages/processor-wasm/lib/rules/convert-get-local-to-local-get/fixture/convert-get-local-to-local-get.wast b/packages/processor-wasm/lib/rules/convert-get-local-to-local-get/fixture/convert-get-local-to-local-get.wast
@@ -0,0 +1,16 @@
+(module
+  ;; this is simple function that adds a couple of parameters
+  (func (param $a i32) (param $b i32)
+    (get_local $a)
+    (get_local $b)
+    (i32.add)
+  )
+  
+  (func (param $a i32) (param $b i32)
+   (i32.add (get_local $a)(get_local $b))
+    
+  )
+  
+  ;; this statement exports the function to the host environment
+  (export "add" (func $add))
+)
diff --git a/packages/processor-wasm/lib/rules/convert-get-local-to-local-get/index.js b/packages/processor-wasm/lib/rules/convert-get-local-to-local-get/index.js
@@ -1,4 +1,4 @@
-export const report = () => `Use 'local.get' instead of get_local`;
+export const report = () => `Use 'local.get' instead of 'get_local'`;
 
 export const fix = (path) => {
     path.node.id = 'get';
diff --git a/packages/processor-wasm/lib/rules/convert-get-local-to-local-get/index.spec.js b/packages/processor-wasm/lib/rules/convert-get-local-to-local-get/index.spec.js
@@ -0,0 +1,18 @@
+import {createTest} from '#test';
+import * as plugin from './index.js';
+
+const test = createTest(import.meta.url, {
+    plugins: [
+        ['convert-get-local-t-local-get', plugin],
+    ],
+});
+
+test('putout: processor-wasm: convert-get-local-to-local-get: report', (t) => {
+    t.report('convert-get-local-to-local-get', `Use 'local.get' instead of 'get_local'`);
+    t.end();
+});
+
+test('putout: processor-wasm: convert-get-local-to-local-get: transform', (t) => {
+    t.transform('convert-get-local-to-local-get');
+    t.end();
+});
diff --git a/packages/processor-wasm/lib/rules/convert-set-local-to-local-set/fixture/convert-set-local-to-local-set-fix.wast b/packages/processor-wasm/lib/rules/convert-set-local-to-local-set/fixture/convert-set-local-to-local-set-fix.wast
@@ -0,0 +1,12 @@
+(module
+  ;; this is simple function that adds a couple of parameters
+
+  (func $add (param $a i32) (param $b i32) (result i32)
+    (local.set $a)
+    (local.set $b)
+    (i32.add)
+  )
+  ;; this statement exports the function to the host environment
+
+  (export "add" (func $add))
+)
diff --git a/packages/processor-wasm/lib/rules/convert-set-local-to-local-set/fixture/convert-set-local-to-local-set.wast b/packages/processor-wasm/lib/rules/convert-set-local-to-local-set/fixture/convert-set-local-to-local-set.wast
@@ -0,0 +1,12 @@
+(module
+
+  ;; this is simple function that adds a couple of parameters
+  (func $add (param $a i32) (param $b i32) (result i32)
+    (set_local $a)
+    (set_local $b)
+    (i32.add)
+  )
+
+  ;; this statement exports the function to the host environment
+  (export "add" (func $add))
+)
diff --git a/packages/processor-wasm/lib/rules/convert-set-local-to-local-set/index.js b/packages/processor-wasm/lib/rules/convert-set-local-to-local-set/index.js
@@ -1,4 +1,4 @@
-export const report = () => `Use 'local.set' instead of set_local`;
+export const report = () => `Use 'local.set' instead of 'set_local'`;
 
 export const fix = (path) => {
     path.node.id = 'set';
diff --git a/packages/processor-wasm/lib/rules/convert-set-local-to-local-set/index.spec.js b/packages/processor-wasm/lib/rules/convert-set-local-to-local-set/index.spec.js
@@ -0,0 +1,18 @@
+import {createTest} from '#test';
+import * as plugin from './index.js';
+
+const test = createTest(import.meta.url, {
+    plugins: [
+        ['convert-set-local-t-local-set', plugin],
+    ],
+});
+
+test('putout: processor-wasm: convert-set-local-to-local-set: report', (t) => {
+    t.report('convert-set-local-to-local-set', `Use 'local.set' instead of 'set_local'`);
+    t.end();
+});
+
+test('putout: processor-wasm: convert-set-local-to-local-set: transform', (t) => {
+    t.transform('convert-set-local-to-local-set');
+    t.end();
+});
diff --git a/packages/processor-wasm/lib/rules/index.js b/packages/processor-wasm/lib/rules/index.js
@@ -1,5 +1,5 @@
-import * as convertGetLocalToLocalGet from './convert-get-local-to-local-get.js';
-import * as convertSetLocalToLocalSet from './convert-set-local-to-local-set.js';
+import * as convertGetLocalToLocalGet from './convert-get-local-to-local-get/index.js';
+import * as convertSetLocalToLocalSet from './convert-set-local-to-local-set/index.js';
 
 export const rules = [
     ['convert-get-local-to-local-get', convertGetLocalToLocalGet],
diff --git a/packages/processor-wasm/lib/wasm.js b/packages/processor-wasm/lib/wasm.js
@@ -6,7 +6,9 @@ export const files = [
 ];
 
 export const find = (source) => {
-    const result = lint(source);
+    const result = lint(source, {
+        fix: false,
+    });
     
     return result.places;
 };
diff --git a/packages/processor-wasm/package.json b/packages/processor-wasm/package.json
@@ -6,6 +6,11 @@
   "description": "🐊Putout processor adds ability to parse wasm files and lint js *.wast files",
   "homepage": "https://github.com/coderaiser/putout/tree/master/packages/processor-wasm#readme",
   "main": "lib/wasm.js",
+  "exports": {
+    ".": "./lib/wasm.js",
+    "./lint": "./lib/lint.js",
+    "./plugin": "./lib/rules/index.js"
+  },
   "release": false,
   "tag": false,
   "changelog": false,
@@ -46,6 +51,11 @@
     "nodemon": "^3.0.1",
     "putout": "*"
   },
+  "imports": {
+    "#test": {
+      "default": "./test/create-test.js"
+    }
+  },
   "peerDependencies": {
     "putout": ">=32"
   },
diff --git a/packages/processor-wasm/test/create-test.js b/packages/processor-wasm/test/create-test.js
@@ -0,0 +1,10 @@
+import {createTest as createPutoutTest} from '@putout/test';
+import {lint} from '../lib/lint.js';
+
+export const createTest = (url, options) => {
+    return createPutoutTest(url, {
+        ext: '.wast',
+        lint,
+        ...options,
+    });
+};
diff --git a/packages/processor-wasm/test/wasm.js b/packages/processor-wasm/test/wasm.js
@@ -29,14 +29,14 @@ test('putout: processor: wasm: parser error', async ({comparePlaces}) => {
 
 test('putout: processor: wasm: places: set-local', async ({comparePlaces}) => {
     await comparePlaces('set-local.wast', [{
-        message: `Use 'local.set' instead of set_local`,
+        message: `Use 'local.set' instead of 'set_local'`,
         position: {
             column: 5,
             line: 5,
         },
         rule: 'convert-set-local-to-local-set (wasm)',
     }, {
-        message: `Use 'local.set' instead of set_local`,
+        message: `Use 'local.set' instead of 'set_local'`,
         position: {
             column: 5,
             line: 6,
@@ -47,14 +47,14 @@ test('putout: processor: wasm: places: set-local', async ({comparePlaces}) => {
 
 test('putout: processor: wasm: places: get-local', async ({comparePlaces}) => {
     await comparePlaces('get-local.wast', [{
-        message: `Use 'local.get' instead of get_local`,
+        message: `Use 'local.get' instead of 'get_local'`,
         position: {
             column: 5,
             line: 5,
         },
         rule: 'convert-get-local-to-local-get (wasm)',
     }, {
-        message: `Use 'local.get' instead of get_local`,
+        message: `Use 'local.get' instead of 'get_local'`,
         position: {
             column: 5,
             line: 6,
diff --git a/packages/test/README.md b/packages/test/README.md
@@ -58,6 +58,18 @@ const test = createTest(import.meta.url, {
 });
 ```
 
+You can also pass extensions to read from `fixture` directory:
+
+```js
+const test = createTest(import.meta.url, {
+    ext: '.wast',
+    lint: putout,
+    plugins: [
+        ['remove-unused-variables', rmVars],
+    ],
+});
+```
+
 ### `report(filename, message: string | string[], plugins?: PutoutPlugin[])`
 
 Check error message (or messages) of a plugin:
diff --git a/packages/test/lib/test.js b/packages/test/lib/test.js
diff --git a/packages/test/package.json b/packages/test/package.json
diff --git a/packages/test/test/ext.js b/packages/test/test/ext.js
diff --git a/packages/test/test/fixture/ext-no-transform.wast b/packages/test/test/fixture/ext-no-transform.wast
diff --git a/packages/test/test/fixture/ext.wast b/packages/test/test/fixture/ext.wast
diff --git a/packages/test/test/report.js b/packages/test/test/report.js

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		-export const report = () => `Use 'local.get' instead of get_local`;
	`1`	+export const report = () => `Use 'local.get' instead of 'get_local'`;
`2`	`2`
`3`	`3`	`export const fix = (path) => {`
`4`	`4`	`path.node.id = 'get';`