Skip automatic CTest discovery when test explorer integration is disabled

Author: hanniavalera

CHANGELOG.md

@@ -10,6 +10,7 @@ Improvements:
 - Improve responsiveness to CMake path changes made by vendor extensions during configure-on-open retry. [#4908](https://github.com/microsoft/vscode-cmake-tools/pull/4908) Contributed by STMicroelectronics
 
 Bug Fixes:
+- Skip the automatic post-build CTest discovery when `cmake.ctest.testExplorerIntegrationEnabled` is `false`. [#4909](https://github.com/microsoft/vscode-cmake-tools/issues/4909)
 - Fix kit detection returning "unknown vendor" when using clang-cl compiler. [#4638](https://github.com/microsoft/vscode-cmake-tools/issues/4638)
 - Update testing framework to fix bugs when running tests of CMake Tools without a reliable internet connection. [#4891](https://github.com/microsoft/vscode-cmake-tools/pull/4891) [@cwalther](https://github.com/cwalther)
 

docs/cmake-settings.md

@@ -39,7 +39,7 @@ Options that support substitution, in the table below, allow variable references
 | `cmake.ctest.allowParallelJobs` | If `true`, allow running test jobs in parallel. When `false`, tests run sequentially in alphabetical order, matching the Test Explorer display order. | `false` | no |
 | `cmake.ctest.debugLaunchTarget` | Target to debug during CTest execution. | `null` | no |
 | `cmake.ctest.parallelJobs` | Specify the number of jobs to run in parallel for ctest. Using the value `0` will detect and use the number of CPUs. Using the value `1` will disable test parallelism. | `0` | no |
-| `cmake.ctest.testExplorerIntegrationEnabled` | If `true`, configure CMake to generate information needed by the test explorer. | `true` | no |
+| `cmake.ctest.testExplorerIntegrationEnabled` | If `true`, configure CMake to generate information needed by the test explorer. When `false`, the automatic CTest discovery that runs after each build is also skipped; the post-configure refresh and the manual `cmake.refreshTests` command still run. | `true` | no |
 | `cmake.ctest.testSuiteDelimiter` | Character(s) that separate test suite name components. | `null` | no |
 | `cmake.ctestArgs` | An array of additional arguments to pass to CTest. | `[]` | yes |
 | `cmake.ctestDefaultArgs` | Default arguments to pass to CTest. | `["-T", "test", "--output-on-failure"]` | no |

src/cmakeProject.ts

@@ -2384,7 +2384,9 @@ export class CMakeProject {
                             // remain from a previous build that had parsing enabled.
                             collections.build.clear();
                         }
-                        await this.cTestController.refreshTests(drv!);
+                        if (drv!.config.testExplorerIntegrationEnabled) {
+                            await this.cTestController.refreshTests(drv!);
+                        }
                         await this.refreshCompileDatabase(drv!.expansionOptions);
                         return {
                             exitCode: rc === null ? -1 : rc,