@playwright/test 1.32.0-beta-1679517005000 → 1.32.0-beta-1679601059000

package/lib/runner/uiMode.js

@@ -34,6 +34,7 @@ class UIMode {
     this._page = void 0;
     this._testRun = void 0;
     this.globalCleanup = void 0;
+    this._globalWatcher = void 0;
     this._testWatcher = void 0;
     this._originalStderr = void 0;
     this._config = config;
@@ -53,25 +54,19 @@ class UIMode {
       sources: false
     };
     this._originalStderr = process.stderr.write.bind(process.stderr);
-    this._installGlobalWatcher();
-  }
-  _installGlobalWatcher() {
-    const projectDirs = new Set();
-    for (const p of this._config.projects) projectDirs.add(p.testDir);
-    let coalescingTimer;
-    const watcher = _utilsBundle.chokidar.watch([...projectDirs], {
-      ignoreInitial: true,
-      persistent: true
-    }).on('all', async event => {
-      if (event !== 'add' && event !== 'change' && event !== 'unlink') return;
-      if (coalescingTimer) clearTimeout(coalescingTimer);
-      coalescingTimer = setTimeout(() => {
-        this._dispatchEvent({
-          method: 'listChanged'
-        });
-      }, 200);
+    this._globalWatcher = new Watcher('deep', () => this._dispatchEvent({
+      method: 'listChanged'
+    }));
+    this._testWatcher = new Watcher('flat', events => {
+      const collector = new Set();
+      events.forEach(f => (0, _compilationCache.collectAffectedTestFiles)(f.file, collector));
+      this._dispatchEvent({
+        method: 'testFilesChanged',
+        params: {
+          testFileNames: [...collector]
+        }
+      });
     });
-    return watcher;
   }
   async runGlobalSetup() {
     const reporter = await (0, _reporters.createReporter)(this._config, 'watch');
@@ -175,6 +170,9 @@ class UIMode {
     reporter.onExit({
       status
     });
+    const projectDirs = new Set();
+    for (const p of this._config.projects) projectDirs.add(p.testDir);
+    this._globalWatcher.update([...projectDirs], false);
   }
   async _runTests(testIds) {
     await this._stopTests();
@@ -212,43 +210,7 @@ class UIMode {
       files.add(fileName);
       (0, _compilationCache.dependenciesForTestFile)(fileName).forEach(file => files.add(file));
     }
-    const watchedFiles = [...files].sort();
-    if (this._testWatcher && JSON.stringify(this._testWatcher.watchedFiles) === JSON.stringify(watchedFiles)) return;
-    if (this._testWatcher) {
-      if (this._testWatcher.collector.size) this._dispatchEvent({
-        method: 'filesChanged',
-        params: {
-          fileNames: [...this._testWatcher.collector]
-        }
-      });
-      clearTimeout(this._testWatcher.timer);
-      this._testWatcher.watcher.close().then(() => {});
-      this._testWatcher = undefined;
-    }
-    if (!watchedFiles.length) return;
-    const collector = new Set();
-    const watcher = _utilsBundle.chokidar.watch(watchedFiles, {
-      ignoreInitial: true
-    }).on('all', async (event, file) => {
-      if (event !== 'add' && event !== 'change') return;
-      (0, _compilationCache.collectAffectedTestFiles)(file, collector);
-      if (this._testWatcher.timer) clearTimeout(this._testWatcher.timer);
-      this._testWatcher.timer = setTimeout(() => {
-        const fileNames = [...collector];
-        collector.clear();
-        this._dispatchEvent({
-          method: 'filesChanged',
-          params: {
-            fileNames
-          }
-        });
-      }, 250);
-    });
-    this._testWatcher = {
-      watcher,
-      watchedFiles,
-      collector
-    };
+    this._testWatcher.update([...files], true);
   }
   async _stopTests() {
     var _this$_testRun, _this$_testRun$stop, _this$_testRun2;
@@ -276,4 +238,44 @@ function chunkToPayload(type, chunk) {
     type,
     text: chunk
   };
+}
+class Watcher {
+  constructor(mode, onChange) {
+    this._onChange = void 0;
+    this._watchedFiles = [];
+    this._collector = [];
+    this._fsWatcher = void 0;
+    this._throttleTimer = void 0;
+    this._mode = void 0;
+    this._mode = mode;
+    this._onChange = onChange;
+  }
+  update(watchedFiles, reportPending) {
+    var _this$_fsWatcher;
+    if (JSON.stringify(this._watchedFiles) === JSON.stringify(watchedFiles)) return;
+    if (reportPending) this._reportEventsIfAny();
+    this._watchedFiles = watchedFiles;
+    (_this$_fsWatcher = this._fsWatcher) === null || _this$_fsWatcher === void 0 ? void 0 : _this$_fsWatcher.close().then(() => {});
+    this._fsWatcher = undefined;
+    this._collector.length = 0;
+    clearTimeout(this._throttleTimer);
+    this._throttleTimer = undefined;
+    if (!this._watchedFiles.length) return;
+    this._fsWatcher = _utilsBundle.chokidar.watch(watchedFiles, {
+      ignoreInitial: true
+    }).on('all', async (event, file) => {
+      if (this._throttleTimer) clearTimeout(this._throttleTimer);
+      if (this._mode === 'flat' && event !== 'add' && event !== 'change') return;
+      if (this._mode === 'deep' && event !== 'add' && event !== 'change' && event !== 'unlink' && event !== 'addDir' && event !== 'unlinkDir') return;
+      this._collector.push({
+        event,
+        file
+      });
+      this._throttleTimer = setTimeout(() => this._reportEventsIfAny(), 250);
+    });
+  }
+  _reportEventsIfAny() {
+    if (this._collector.length) this._onChange(this._collector.slice());
+    this._collector.length = 0;
+  }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@playwright/test",
-  "version": "1.32.0-beta-1679517005000",
+  "version": "1.32.0-beta-1679601059000",
   "description": "A high-level API to automate web browsers",
   "repository": "github:Microsoft/playwright",
   "homepage": "https://playwright.dev",
@@ -34,7 +34,7 @@
   "license": "Apache-2.0",
   "dependencies": {
     "@types/node": "*",
-    "playwright-core": "1.32.0-beta-1679517005000"
+    "playwright-core": "1.32.0-beta-1679601059000"
   },
   "optionalDependencies": {
     "fsevents": "2.3.2"