@reporters/testwatch 1.1.1 → 1.2.1

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Changelog
 
+## [1.2.1](https://github.com/MoLow/reporters/compare/testwatch-v1.2.0...testwatch-v1.2.1) (2023-07-02)
+
+
+### Bug Fixes
+
+* clear screen before tests ([3d3f67c](https://github.com/MoLow/reporters/commit/3d3f67cddd7047c857e001066021933eb1381ec3))
+
+## [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)
 
 

package/index.js

@@ -39,7 +39,7 @@ process.stdin.setRawMode?.(true);
 class REPL {
   #controller = new AbortController();
 
-  #filesFilter = '';
+  #filesFilter = process.argv[2] || '';
 
   #testsFilter = '';
 
@@ -57,13 +57,13 @@ class REPL {
   #emitter = new EventEmitter();
 
   async #runTests() {
+    this.#clear();
     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) {
-      this.#clear();
       process.stdout.write(chalk.red(`\nNo files found for pattern ${filter}\n`));
       this.#emitter.emit('drained');
       return;
@@ -220,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.1",
+  "version": "1.2.1",
   "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();
     }
   });
@@ -89,21 +89,21 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
   it('should run all tests on initialization', async () => {
     const { outputs, stderr } = await spawnInteractive('q');
     assert.strictEqual(stderr, '');
-    assert.deepStrictEqual(outputs, ['', `${tests}\n${mainMenu}\n`]);
+    assert.deepStrictEqual(outputs, ['', '', `${tests}\n${mainMenu}\n`]);
   });
   it('should handle CTR + C', async () => {
     const { outputs, stderr } = await spawnInteractive('\x03');
     assert.strictEqual(stderr, '');
-    assert.deepStrictEqual(outputs, ['', `${tests}\n${mainMenu}\n`]);
+    assert.deepStrictEqual(outputs, ['', '', `${tests}\n${mainMenu}\n`]);
   });
   it('should handle CTR + D', async () => {
     const { outputs, stderr } = await spawnInteractive('\x04');
     assert.strictEqual(stderr, '');
-    assert.deepStrictEqual(outputs, ['', `${tests}\n${mainMenu}\n`]);
+    assert.deepStrictEqual(outputs, ['', '', `${tests}\n${mainMenu}\n`]);
   });
   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 = '';
@@ -121,19 +121,21 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
   it('should run all tests on "a"', async () => {
     const { outputs, stderr } = await spawnInteractive('aq');
     assert.strictEqual(stderr, '');
-    assert.deepStrictEqual(outputs, ['', `${tests}\n${mainMenu}`, `${tests}\n${compactMenu}\n`]);
+    assert.deepStrictEqual(outputs, ['', '', `${tests}\n${mainMenu}`, '', `${tests}\n${compactMenu}\n`]);
   });
   it('should run all tests on Enter', async () => {
     const { outputs, stderr } = await spawnInteractive('\rq');
     assert.strictEqual(stderr, '');
-    assert.deepStrictEqual(outputs, ['', `${tests}\n${mainMenu}`, `${tests}\n${compactMenu}\n`]);
+    assert.deepStrictEqual(outputs, ['', '', `${tests}\n${mainMenu}`, '', `${tests}\n${compactMenu}\n`]);
   });
   it('should show full menu on "w" after running tests', async () => {
     const { outputs, stderr } = await spawnInteractive('awq');
     assert.strictEqual(stderr, '');
     assert.deepStrictEqual(outputs, [
+      '',
       '',
       `${tests}\n${mainMenu}`,
+      '',
       `${tests}\n${compactMenu}\n${clearLines}${mainMenu}\n`,
     ]);
   });
@@ -144,10 +146,12 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
       const activeFilters = '\nActive Filters: test name /sub/\n';
       assert.strictEqual(stderr, '');
       assert.deepStrictEqual(outputs, [
+        '',
         '',
         `${tests}\n${mainMenu}`,
         `${filterTestsPrompt}sub`,
         '',
+        '',
         `${tests
           .replace('✔ j - sum (*ms)', '﹣ j - sum (*ms) # SKIP')
           .replace('✔ index - sum (*ms)', '﹣ index - sum (*ms) # SKIP')
@@ -156,15 +160,39 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
     });
 
     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`,
         ]);
       });
@@ -174,10 +202,12 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
         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`,
         ]);
       });
@@ -186,8 +216,8 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
       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';
       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`);
+      assert.strictEqual(outputs.length, 11);
+      assert.strictEqual(outputs[10], `${testsRun[1].replace('✔ index - subtraction (*ms)', '﹣ index - subtraction (*ms) # SKIP')}\n${compactMenu}\n${clearLines}${activeFilters}${mainMenuWithFilters}\n`);
     });
 
     it('should mention when no files found', async () => {
@@ -196,6 +226,7 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
       const notFound = '\nNo files found for pattern **/nothing*.*';
       assert.strictEqual(stderr, '');
       assert.deepStrictEqual(outputs, [
+        '',
         '',
         `${tests}\n${mainMenu}`,
         `${filterFilesPrompt}nothing`,
@@ -207,12 +238,12 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
     it('should clear filters on "c"', async () => {
       const { outputs, stderr } = await spawnInteractive(['p', 'index', '\r', 'w', 't', 'sum', '\r', 'w', 'c', 'w', 'q'].join(''));
       assert.strictEqual(stderr, '');
-      assert.strictEqual(outputs.length, 9);
+      assert.strictEqual(outputs.length, 13);
 
-      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`);
+      assert.match(outputs[6], /Active Filters: file name \*\*\/index\*\.\*/);
+      assert.match(outputs[7], /Active Filters: file name \*\*\/index\*\.\*/);
+      assert.match(outputs[10], /Active Filters: file name \*\*\/index\*\.\*, test name \/sum\//);
+      assert.strictEqual(outputs[12], `${tests}\n${compactMenu}\n${clearLines}${mainMenu}\n`);
     });
 
     it('prompt ESC should preserve previous state', async () => {
@@ -220,10 +251,12 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
       const notFound = '\nNo files found for pattern **/filter*.*';
       const activeFilters = '\nActive Filters: file name **/filter*.*\n';
       assert.deepStrictEqual(outputs, [
+        '',
         '',
         `${tests}\n${mainMenu}`,
         `${filterFilesPrompt}`,
         '',
+        '',
         `${tests}\n${compactMenu}`,
         `${filterFilesPrompt}filter`,
         '',
@@ -234,12 +267,12 @@ describe('testwatch', { concurrency: true, skip: !isSupported ? 'unsupported nod
       ]);
     });
 
-    it('backspace shoud remove last character', async () => {
+    it('backspace should remove last character', async () => {
       const backspace = '\x7f';
       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.strictEqual(outputs.length, 6);
+      assert.match(outputs[5], /No files found for pattern \*\*\/noth1ing\*\.\*/);
     });
   });
 });