implement test tags in test controller

Author: d-biehl

CHANGELOG.md

@@ -10,6 +10,7 @@ All notable changes to the "robotcode" extension will be documented in this file
 - Added some more configuration options for log and debug messages when running tests in the debug console
 - debug console now shows source and line number from log messages
 - use of debugpy from vscode Python extension, no separate installation of debugpy required
+- implement test tags in test controller
 
 ##  0.2.7
 

package.json

@@ -19,7 +19,7 @@
     "url": "https://github.com/d-biehl/robotcode/issues"
   },
   "engines": {
-    "vscode": "^1.60.0"
+    "vscode": "^1.61.0"
   },
   "categories": [
     "Programming Languages",
@@ -586,8 +586,8 @@
   "devDependencies": {
     "@types/glob": "^7.1.4",
     "@types/mocha": "^9.0.0",
-    "@types/node": "^16.10.2",
-    "@types/vscode": "^1.60.0",
+    "@types/node": "^16.10.3",
+    "@types/vscode": "^1.61.0",
     "@typescript-eslint/eslint-plugin": "^4.33.0",
     "@typescript-eslint/parser": "^4.33.0",
     "eslint": "^7.32.0",
@@ -612,7 +612,7 @@
     "vscode-debugadapter-testsupport": "^1.49.0",
     "vscode-dts": "^0.3.1",
     "vscode-test": "^1.6.1",
-    "webpack": "^5.56.1",
-    "webpack-cli": "^4.8.0"
+    "webpack": "^5.58.0",
+    "webpack-cli": "^4.9.0"
   }
 }

vscode-client/testcontrollermanager.ts

@@ -266,9 +266,9 @@ export class TestControllerManager {
             }
             testItem.label = ri.label;
             testItem.error = ri.error;
-            // TODO: Tags
-            // const tags = this.convertTags(ri.tags);
-            // if (tags) testItem.tags = tags;
+
+            const tags = this.convertTags(ri.tags);
+            if (tags) testItem.tags = tags;
 
             await this.refreshItem(testItem);
           }
@@ -315,9 +315,9 @@ export class TestControllerManager {
             testItem.canResolveChildren = ri.children !== undefined && ri.children.length > 0;
             testItem.label = ri.label;
             testItem.error = ri.error;
-            // TODO: Tags
-            // const tags = this.convertTags(ri.tags);
-            // if (tags) testItem.tags = tags;
+
+            const tags = this.convertTags(ri.tags);
+            if (tags) testItem.tags = tags;
 
             await this.refreshItem(testItem);
           }
@@ -338,24 +338,23 @@ export class TestControllerManager {
     }
   }
 
-  // TODO: Tags
-  // private testTags = new WeakValueMap<string, vscode.TestTag>();
+  private testTags = new WeakValueMap<string, vscode.TestTag>();
 
-  // private convertTags(tags: string[] | undefined): vscode.TestTag[] | undefined {
-  // if (tags === undefined) return undefined;
+  private convertTags(tags: string[] | undefined): vscode.TestTag[] | undefined {
+    if (tags === undefined) return undefined;
 
-  // const result: vscode.TestTag[] = [];
+    const result: vscode.TestTag[] = [];
 
-  // for (const tag of tags) {
-  //   if (!this.testTags.has(tag)) {
-  //     this.testTags.set(tag, new vscode.TestTag(tag));
-  //   }
-  //   const vstag = this.testTags.get(tag);
-  //   if (vstag !== undefined) result.push(vstag);
-  // }
+    for (const tag of tags) {
+      if (!this.testTags.has(tag)) {
+        this.testTags.set(tag, new vscode.TestTag(tag));
+      }
+      const vstag = this.testTags.get(tag);
+      if (vstag !== undefined) result.push(vstag);
+    }
 
-  // return result;
-  // }
+    return result;
+  }
 
   private readonly refreshFromUriMutex = new Mutex();