autorun pick fix

CHANGELOG.md

@@ -6,6 +6,12 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ## [Unreleased]
 
+## [4.4.3]
+
+### Fixed
+
+- issue related to autorun feature. It was not picking up some level of tests. [related](https://github.com/matepek/vscode-catch2-test-adapter/issues/387).
+
 ## [4.4.2] - 2023-04-28
 
 Improved .env file parser

README.md

@@ -21,7 +21,7 @@ It also have basic support for [Google Benchmark](https://github.com/google/benc
 - Runs executables parallel (_testMate.cpp.test.parallelExecutionLimit_).
 - Sorts tests and suites (_testExplorer.sort_).
 - Supports popular **debuggers** such as `vadimcn.vscode-lldb`, `webfreak.debug` and `ms-vscode.cpptools` out of the box.
-- Retire tests and "Autorun" them. (Coming soon..)
+- Retire tests and "Autorun" them.
 
 ### Screenshots
 
@@ -87,8 +87,6 @@ Plenty of more **fine-tuning options** are available under [test.advancedExecuta
 | `testMate.cmd.reload-workspaces` | Force reload workspaces (clean slate) |
 | `testing.refreshTests`           | Force reload workspaces (clean slate) |
 
-## About [Sentry.io](https://github.com/matepek/vscode-catch2-test-adapter/blob/master/documents/configuration/log.logSentry.md) integration
-
 ## [License](https://github.com/matepek/vscode-catch2-test-adapter/blob/master/LICENSE)
 
 ## [Support](https://github.com/matepek/vscode-catch2-test-adapter/blob/master/documents/support.md)

src/framework/AbstractExecutable.ts

@@ -85,6 +85,17 @@ export abstract class AbstractExecutable<TestT extends AbstractTest = AbstractTe
     return this._tests.values();
   }
 
+  public hasTestWithId(id: string): boolean {
+    if (this._execItem.getItem()?.id === id) return true;
+    const found = this._tests.get(id);
+    if (found) return true;
+    for (const test of this._tests.values()) {
+      const found = test.hasSubTest(id);
+      if (found) return true;
+    }
+    return false;
+  }
+
   private async _getOrCreateChildGroup(
     idIn: string | undefined,
     label: string,

src/framework/AbstractTest.ts

@@ -151,6 +151,17 @@ export abstract class AbstractTest {
 
   private _subTests: Map<string /*id*/, SubTest> | undefined = undefined;
 
+  public hasSubTest(id: string): boolean {
+    if (this._subTests === undefined) return false;
+    const found = this._subTests.get(id);
+    if (found !== undefined) return true;
+    for (const subTest of this._subTests.values()) {
+      const found = subTest.hasSubTest(id);
+      if (found) return true;
+    }
+    return false;
+  }
+
   async getOrCreateSubTest(
     id: string,
     label: string | undefined,

src/main.ts

@@ -175,9 +175,12 @@ export async function activate(context: vscode.ExtensionContext): Promise<void>
               else for (const t of e.getTests()) include.push(t.item);
             }
           } else {
-            for (const e of executables) {
-              for (const t of e.getTests()) {
-                if (request.include.indexOf(t.item) !== -1) include.push(t.item);
+            for (const item of request.include) {
+              for (const e of executables) {
+                if (e.hasTestWithId(item.id)) {
+                  include.push(item);
+                  break;
+                }
               }
             }
           }