Fix: Always export AdvancedSettings as array in SCSensitivityLabel

[Harikumar4]

Pull Request (PR) description

This PR ensures that the AdvancedSettings property in the SCSensitivityLabel resource is always exported as an array block in the generated DSC configuration, even when there is only a single item. This improves consistency and prevents downstream issues when consuming the exported configuration.

This Pull Request (PR) fixes the following issues

• Fixes [SCSensitivityLabel] AdvancedSettings seems to export a single item as a single item instead of an arrya #6321

Task list

• Added an entry to the change log under the Unreleased section of the file CHANGELOG.md.
• Resource parameter descriptions added/updated in the schema.mof. (Not applicable for this PR)
• Resource documentation added/updated in README.md. (Not applicable for this PR)
• Resource settings.json file contains all required permissions. (Not applicable for this PR)
• Examples appropriately added/updated. (Not applicable for this PR)
• Unit tests added/updated. (Not applicable for this PR)
• New/changed code adheres to DSC Community Style Guidelines.

[Harikumar4]

@Harikumar4 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

• (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

• (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

[FabienTschanz]

@Harikmar4 The CIM instance conversion you're trying to fix is based on the actual value of the property. To not meddle with the CIM conversion at the end, it's more appropriate to ensure that the property is an array. To fix this, update the following line:

# lines 361 to 363
if ($null -ne $label.Settings)
{
    # Add the [array] type indicator to the variable to enforce conversion to an array
    [array]$advancedSettingsValue = Convert-StringToAdvancedSettings -AdvancedSettings $label.Settings
}

This way, the export will look correctly:

SCSensitivityLabel "SCSensitivityLabel-defa4170-0d19-0005-0000-bc88714345d2"
{
    AdvancedSettings      = @(
         MSFT_SCLabelSetting{
             Key = "isparent"
             Value = "False"
        }
   );
...

This also reduces the PR size massively 😉

[Harikumar4]

Thanks for the clarification.
I have updated the logic to

[array]$advancedSettingsValue = Convert-StringToAdvancedSettings -AdvancedSettings $label.Settings

This ensures that AdvancedSettings is always an array. Since $advancedSettingsValue is assigned directly in the export block ($result = @{} at line 642), a separate assignment like $Results.AdvancedSettings = $advancedSettingsValue isn’t necessary.
Let me know if you need any further changes.

[FabienTschanz]

LGTM, now we have to wait for approval and merge. Thank you for the fix 👍

[FabienTschanz]

@Harikumar4 Can you please update the issue link in the Pull Request description? The link is currently invalid. You can just type - Fixes #6321 without providing any link and GitHub will automatically determine the correct issue. This will allow us to automatically close the other issue once this Pull Request is merged. Thank you.