Stop UTILITY targets from affecting IntelliSense

Some `UTILITY` targets, such as the ones created by `Doxygen.cmake`,
have a list of sources attached. Avoid passing them to
`updateConfigurationData()` so that they don't interfere with the
assumptions made by `getConfiguration()` in the case that `all` or some
other non-artifact target is the selected Build Target.

* fix for #4404

Author: malsyned

CHANGELOG.md

@@ -19,6 +19,7 @@ Bug Fixes:
 - Fixes for small bug string bugs. [#4319](https://github.com/microsoft/vscode-cmake-tools/issues/4319), [#4317](https://github.com/microsoft/vscode-cmake-tools/issues/4317), [#4312](https://github.com/microsoft/vscode-cmake-tools/issues/4312)
 - Fixes localization for "workspace is" string. [#4374](https://github.com/microsoft/vscode-cmake-tools/issues/4374)
 - Make tooltips for selecting Launch/Debug Target. [#4373](https://github.com/microsoft/vscode-cmake-tools/issues/4373)
+- Fix incorrect IntelliSense configuration when a `UTILITY` has source files. [#4404](https://github.com/microsoft/vscode-cmake-tools/issues/4404)
 
 ## 1.20.53
 

src/cpptools.ts

@@ -596,6 +596,10 @@ export class CppConfigurationProvider implements cpptools.CustomConfigurationPro
             if (config.name === opts.activeBuildTypeVariant || (!opts.activeBuildTypeVariant && config.name === "")) {
                 for (const project of config.projects) {
                     for (const target of project.targets) {
+                        if (target.type === 'UTILITY') {
+                            continue;
+                        }
+
                         // Now some shenanigans since header files don't have config data:
                         // 1. Accumulate some "defaults" based on the set of all options for each file group
                         // 2. Pass these "defaults" down when rebuilding the config data

test/unit-tests/cpptools.test.ts

@@ -213,6 +213,14 @@ suite('CppTools tests', () => {
                     name: 'cpptools-test-3',
                     sourceDirectory: smokeFolder,
                     targets: [
+                        {
+                            name: 'utilityTarget',
+                            type: 'UTILITY',
+                            fileGroups: [{
+                                sources: [sourceFile3],
+                                isGenerated: false
+                            }]
+                        },
                         {
                             name: 'target3',
                             type: 'EXECUTABLE',
@@ -319,6 +327,14 @@ suite('CppTools tests', () => {
                     name: 'cpptools-test2',
                     sourceDirectory: smokeFolder,
                     targets: [
+                        {
+                            name: 'utilityTarget',
+                            type: 'UTILITY',
+                            fileGroups: [{
+                                sources: [sourceFile3],
+                                isGenerated: false
+                            }]
+                        },
                         {
                             name: 'target3',
                             type: 'EXECUTABLE',