Associate CTest tests with add_test() wrappers (#4490) (#4491)

When CTest tests are associated directly with `add_test()` calls, then
using functions that wrap `add_test()` results in many tests linked to
the exact same line of code. Instead, walk the CTest backtrace from the
`add_test()` call and associate the test with the outermost stack frame.

Author: malsyned

CHANGELOG.md

@@ -16,6 +16,7 @@ Improvements:
 - Add name de-mangling for C++ symbols in the Test Explorer view when running tests with coverage. [#4340](https://github.com/microsoft/vscode-cmake-tools/pull/4340) [@rjaegers](https://github.com/rjaegers)
 - No longer convert paths on lowercase on MacOS to enable cpp tools to resolve them. [#4325](https://github.com/microsoft/vscode-cmake-tools/pull/4325) [@tringenbach](https://github.com/tringenbach)
 - Speedup & reduce heap allocations in shlex split module function. Significant gains for mid-large compile_commands.json - CompilationDatabase construction. [#4458](https://github.com/microsoft/vscode-cmake-tools/pull/4458) [@borjamunozf](https://github.com/borjamunozf)
+- In the Test Explorer, associate CTest tests with outermost function or macro invocation that calls `add_test()` instead of with the `add_test()` call itself. [#4490](https://github.com/microsoft/vscode-cmake-tools/issues/4490) [@malsyned](https://github.com/malsyned)
 
 Bug Fixes:
 

src/ctest.ts

@@ -835,11 +835,16 @@ export class CTestDriver implements vscode.Disposable {
                         }
                     }
 
-                    if (!testDefFile && test.backtrace !== undefined && this.tests!.backtraceGraph.nodes[test.backtrace] !== undefined) {
+                    const nodes = this.tests!.backtraceGraph.nodes;
+                    if (!testDefFile && test.backtrace !== undefined && nodes[test.backtrace] !== undefined) {
                         // Use the backtrace graph to find the file and line number
                         // This finds the CMake module's file and line number and not the test file and line number
-                        testDefFile = this.tests!.backtraceGraph.files[this.tests!.backtraceGraph.nodes[test.backtrace].file];
-                        testDefLine = this.tests!.backtraceGraph.nodes[test.backtrace].line;
+                        let node = nodes[test.backtrace];
+                        while (node.parent !== undefined && nodes[node.parent].command !== undefined) {
+                            node = nodes[node.parent];
+                        }
+                        testDefFile = this.tests!.backtraceGraph.files[node.file];
+                        testDefLine = node.line;
                     }
 
                     const testAndParentSuite = this.createTestItemAndSuiteTree(test.name, testExplorerRoot, initializedTestExplorer, testDefFile ? vscode.Uri.file(testDefFile) : undefined);