fix(#92): activate extension on all commands (#94)

microsoft · Feb 16, 2022 · 108ace3 · 108ace3
1 parent 26adb02
commit 108ace3
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/package.json b/package.json
@@ -18,7 +18,9 @@
   ],
   "activationEvents": [
     "workspaceContains:**/*playwright*.config.{ts,js,mjs}",
-    "onCommand:pw.extension.install"
+    "onCommand:pw.extension.install",
+    "onCommand:pw.extension.recordTest",
+    "onCommand:pw.extension.refreshTests"
   ],
   "main": "./out/extension.js",
   "contributes": {

diff --git a/src/extension.ts b/src/extension.ts
@@ -114,10 +114,18 @@ export class Extension {
       vscode.window.onDidChangeVisibleTextEditors(() => {
         this._updateVisibleEditorItems();
       }),
-      vscode.commands.registerCommand('pw.extension.refreshTests', () => {
-        this._rebuildModel();
+      vscode.commands.registerCommand('pw.extension.refreshTests', async () => {
+        await this._rebuildModel();
+        if (!this._models.length) {
+          vscode.window.showWarningMessage('No Playwright Test config files found.');
+          return;
+        }
       }),
       vscode.commands.registerCommand('pw.extension.recordTest', async () => {
+        if (!this._models.length) {
+          vscode.window.showWarningMessage('No Playwright Test config files found.');
+          return;
+        }
         this._recorder.record(this._models);
       }),
       vscode.commands.registerCommand('pw.extension.install', () => {