@reporters/testwatch 1.1.0 → 1.2.0

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # Changelog
 
+## [1.2.0](https://github.com/MoLow/reporters/compare/testwatch-v1.1.1...testwatch-v1.2.0) (2023-06-19)
+
+
+### Features
+
+* add test:watch script ([4357fbd](https://github.com/MoLow/reporters/commit/4357fbde5f337cfd39226497f8ce0f6760f8a62c))
+* change default file filter ([5968235](https://github.com/MoLow/reporters/commit/596823529cd9a86e5eeada0523825fa215e49efe))
+
+
+### Bug Fixes
+
+* don't exit when no tests on initial run ([66bcc2b](https://github.com/MoLow/reporters/commit/66bcc2bc6436b544900cfbceb4bbfb0d93973490))
+
+## [1.1.1](https://github.com/MoLow/reporters/compare/testwatch-v1.1.0...testwatch-v1.1.1) (2023-06-12)
+
+
+### Bug Fixes
+
+* increas maximum listeners of event emitters ([b16d2ab](https://github.com/MoLow/reporters/commit/b16d2ab3b55554e1aaa9b9ca2222254bac364803))
+* support partial file name filter ([2b3f0ab](https://github.com/MoLow/reporters/commit/2b3f0abe0be37450c7b9189667ad47d4d39a4252))
+
 ## [1.1.0](https://github.com/MoLow/reporters/compare/testwatch-v1.0.0...testwatch-v1.1.0) (2023-06-08)
 
 

package/index.js

@@ -2,7 +2,9 @@
 
 const { run } = require('node:test');
 const { pipeline } = require('node:stream/promises');
-const { on, once, EventEmitter } = require('node:events');
+const {
+  on, once, EventEmitter, setMaxListeners,
+} = require('node:events');
 const { glob } = require('glob');
 const chalk = require('chalk');
 const { isSupported } = require('./nodeVersion');
@@ -28,6 +30,7 @@ const KEY_NAMES = {
 };
 const UnknownCommand = Symbol('UnknownKey');
 
+setMaxListeners(Infinity);
 process.stdout.setMaxListeners(Infinity);
 process.setMaxListeners(Infinity);
 process.stdin.setEncoding('utf8');
@@ -36,7 +39,7 @@ process.stdin.setRawMode?.(true);
 class REPL {
   #controller = new AbortController();
 
-  #filesFilter = '';
+  #filesFilter = process.argv[2] || '';
 
   #testsFilter = '';
 
@@ -56,7 +59,7 @@ class REPL {
   async #runTests() {
     this.#controller.abort();
     this.#controller = new AbortController();
-    const filter = this.#filesFilter ? `**/${this.#filesFilter}.*` : '**/*.test.js';
+    const filter = this.#filesFilter ? `**/${this.#filesFilter}*.*` : '**/?(*.)+(spec|test).[jt]s';
     const files = await glob(filter, { ignore: 'node_modules/**' });
 
     if (!files.length) {
@@ -200,7 +203,7 @@ class REPL {
     if (this.#filesFilter || this.#testsFilter) {
       const message = `
 ${chalk.white.bold('Active Filters:')} \
-${this.#filesFilter ? `file name ${chalk.gray('**/')}${chalk.yellow(this.#filesFilter)}${chalk.gray('.*')}` : ''}\
+${this.#filesFilter ? `file name ${chalk.gray('**/')}${chalk.yellow(this.#filesFilter)}${chalk.gray('*.*')}` : ''}\
 ${(this.#testsFilter && this.#filesFilter) ? ', ' : ''}\
 ${this.#testsFilter ? `test name ${chalk.yellow(`/${this.#testsFilter}/`)}` : ''}
 `;
@@ -217,8 +220,7 @@ ${Object.entries(this.#currentCommands)
   }
 
   async run() {
-    await this.#runTests();
-    await once(this.#emitter, 'drained');
+    await Promise.all([once(this.#emitter, 'drained'), this.#runTests()]);
     this.#emitter.on('drained', () => this.#compactHelp());
     this.#help();
     for await (const data of on(process.stdin, 'data')) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reporters/testwatch",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "An interactive repl for `node:test`",
   "keywords": [
     "node:test",
@@ -10,7 +10,8 @@
   ],
   "bin": "./index.js",
   "scripts": {
-    "test": "node --test-reporter=spec --test-reporter-destination=stdout --test-reporter=../github/index.js --test-reporter-destination=stdout --test tests/index.test.js"
+    "test": "node --test-reporter=spec --test-reporter-destination=stdout --test-reporter=../github/index.js --test-reporter-destination=stdout --test tests/index.test.js",
+    "test:watch": "testwatch tests/index"
   },
   "bugs": {
     "url": "https://github.com/MoLow/reporters/issues"

package/tests/fixtures/not-test.js

@@ -0,0 +1,6 @@
+const { test } = require('node:test');
+const assert = require('node:assert');
+
+test('should not run', () => {
+  assert.strictEqual(1 + 2, 3);
+});

package/tests/index.test.js

@@ -35,11 +35,11 @@ Filter Test
  pattern › `;
 const filterFilesPrompt = filterTestsPrompt.replace('test', 'file').replace('Test', 'File');
 
-async function spawnInteractive(commandSequence = 'q') {
+async function spawnInteractive(commandSequence = 'q', args = []) {
   let stderr = '';
   let stdout = '';
-  const child = spawn(process.execPath, ['../../index.js'], {
-    env: { }, cwd: path.resolve(__dirname, 'fixtures'),
+  const child = spawn(process.execPath, ['../../index.js', ...args], {
+    env: {}, cwd: path.resolve(__dirname, 'fixtures'),
   });
   child.stdin.setEncoding('utf8');
   let writing = false;
@@ -62,7 +62,7 @@ async function spawnInteractive(commandSequence = 'q') {
   child.stdout.setEncoding('utf8');
   child.stdout.on('data', (data) => {
     stdout += data;
-    if (stdout.includes(mainMenu)) {
+    if (stdout.includes(mainMenu) || stdout.includes(mainMenuWithFilters)) {
       writeInput();
     }
   });
@@ -103,7 +103,7 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
   });
   it('should exit on sigkill', async () => {
     const child = spawn(process.execPath, ['../../index.js'], {
-      env: { }, cwd: path.resolve(__dirname, 'fixtures'),
+      env: {}, cwd: path.resolve(__dirname, 'fixtures'),
     });
     let stderr = '';
     let stdout = '';
@@ -155,21 +155,57 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
       ]);
     });
 
-    it('should filter files on "p"', async () => {
-      const { outputs, stderr } = await spawnInteractive(['p', 'index', '\r', 'w', 'q'].join(''));
-      const activeFilters = '\nActive Filters: file name **/index.*\n';
-      assert.strictEqual(stderr, '');
-      assert.deepStrictEqual(outputs, [
-        '',
-        `${tests}\n${mainMenu}`,
-        `${filterFilesPrompt}index`,
-        '',
-        `${testsRun[1]}\n${compactMenu}\n${clearLines}${activeFilters}${mainMenuWithFilters}\n`,
-      ]);
+    describe('files filter', () => {
+      it('should not exit if no test found on first run', async () => {
+        const { outputs, stderr } = await spawnInteractive('q', ['notexist']);
+        const activeFilters = '\nActive Filters: file name **/notexist*.*\n';
+        const notFound = '\nNo files found for pattern **/notexist*.*';
+        assert.strictEqual(stderr, '');
+        assert.deepStrictEqual(outputs, [
+          '',
+          `${notFound}\n${activeFilters}${mainMenuWithFilters}\n`,
+        ]);
+      });
+
+      it('should set first argument as file filter', async () => {
+        const { outputs, stderr } = await spawnInteractive('q', ['ind']);
+        const activeFilters = '\nActive Filters: file name **/ind*.*\n';
+        assert.strictEqual(stderr, '');
+        assert.deepStrictEqual(outputs, [
+          '',
+          `${testsRun[1]}\n${activeFilters}${mainMenuWithFilters}\n`,
+        ]);
+      });
+
+      it('should filter files on "p"', async () => {
+        const { outputs, stderr } = await spawnInteractive(['p', 'index', '\r', 'w', 'q'].join(''));
+        const activeFilters = '\nActive Filters: file name **/index*.*\n';
+        assert.strictEqual(stderr, '');
+        assert.deepStrictEqual(outputs, [
+          '',
+          `${tests}\n${mainMenu}`,
+          `${filterFilesPrompt}index`,
+          '',
+          `${testsRun[1]}\n${compactMenu}\n${clearLines}${activeFilters}${mainMenuWithFilters}\n`,
+        ]);
+      });
+
+      it('should filter partial file names on "p"', async () => {
+        const { outputs, stderr } = await spawnInteractive(['p', 'ind', '\r', 'w', 'q'].join(''));
+        const activeFilters = '\nActive Filters: file name **/ind*.*\n';
+        assert.strictEqual(stderr, '');
+        assert.deepStrictEqual(outputs, [
+          '',
+          `${tests}\n${mainMenu}`,
+          `${filterFilesPrompt}ind`,
+          '',
+          `${testsRun[1]}\n${compactMenu}\n${clearLines}${activeFilters}${mainMenuWithFilters}\n`,
+        ]);
+      });
     });
     it('should filter tests and files togetheer', async () => {
       const { outputs, stderr } = await spawnInteractive(['p', 'index', '\r', 't', 'sum', '\r', 'w', 'q'].join(''));
-      const activeFilters = '\nActive Filters: file name **/index.*, test name /sum/\n';
+      const activeFilters = '\nActive Filters: file name **/index*.*, test name /sum/\n';
       assert.strictEqual(stderr, '');
       assert.strictEqual(outputs.length, 8);
       assert.strictEqual(outputs[7], `${testsRun[1].replace('✔ index - subtraction (*ms)', '﹣ index - subtraction (*ms) # SKIP')}\n${compactMenu}\n${clearLines}${activeFilters}${mainMenuWithFilters}\n`);
@@ -177,8 +213,8 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
 
     it('should mention when no files found', async () => {
       const { outputs, stderr } = await spawnInteractive(['p', 'nothing', '\r', 'w', 'q'].join(''));
-      const activeFilters = '\nActive Filters: file name **/nothing.*\n';
-      const notFound = '\nNo files found for pattern **/nothing.*';
+      const activeFilters = '\nActive Filters: file name **/nothing*.*\n';
+      const notFound = '\nNo files found for pattern **/nothing*.*';
       assert.strictEqual(stderr, '');
       assert.deepStrictEqual(outputs, [
         '',
@@ -194,16 +230,16 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
       assert.strictEqual(stderr, '');
       assert.strictEqual(outputs.length, 9);
 
-      assert.match(outputs[4], /Active Filters: file name \*\*\/index\.\*/);
-      assert.match(outputs[5], /Active Filters: file name \*\*\/index\.\*/);
-      assert.match(outputs[7], /Active Filters: file name \*\*\/index\.\*, test name \/sum\//);
+      assert.match(outputs[4], /Active Filters: file name \*\*\/index\*\.\*/);
+      assert.match(outputs[5], /Active Filters: file name \*\*\/index\*\.\*/);
+      assert.match(outputs[7], /Active Filters: file name \*\*\/index\*\.\*, test name \/sum\//);
       assert.strictEqual(outputs[8], `${tests}\n${compactMenu}\n${clearLines}${mainMenu}\n`);
     });
 
     it('prompt ESC should preserve previous state', async () => {
       const { outputs } = await spawnInteractive(['p', esc, 'p', 'filter', '\r', 'p', esc, 'q'].join(''));
-      const notFound = '\nNo files found for pattern **/filter.*';
-      const activeFilters = '\nActive Filters: file name **/filter.*\n';
+      const notFound = '\nNo files found for pattern **/filter*.*';
+      const activeFilters = '\nActive Filters: file name **/filter*.*\n';
       assert.deepStrictEqual(outputs, [
         '',
         `${tests}\n${mainMenu}`,
@@ -224,7 +260,7 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
       const { outputs, stderr } = await spawnInteractive(['p', `noth123${backspace}${backspace}ing`, '\r', 'w', 'q'].join(''));
       assert.strictEqual(stderr, '');
       assert.strictEqual(outputs.length, 5);
-      assert.match(outputs[4], /No files found for pattern \*\*\/noth1ing\.\*/);
+      assert.match(outputs[4], /No files found for pattern \*\*\/noth1ing\*\.\*/);
     });
   });
 });