Skip to content

fix: category with naming mismatches to reduce duplicate categories in diffrent naming variants#1171

Open
Mynster9361 wants to merge 3 commits into
microsoft:mainfrom
Mynster9361:uniform-category-naming
Open

fix: category with naming mismatches to reduce duplicate categories in diffrent naming variants#1171
Mynster9361 wants to merge 3 commits into
microsoft:mainfrom
Mynster9361:uniform-category-naming

Conversation

@Mynster9361
Copy link
Copy Markdown

Currently after running the assessment if you wnt to parse the json file "ZeroTrustAssessmentReport.json" and want to look at categories a hinderance i see is the diffrent naming schemes for the same topic example:
Sensitivity Labels
Sensitivity Labels Configuration
sensitivity-labels

3 different naming schemes for Sensitivity Labels.

I am adding an output of the diffrent categories below to show how it looks before this pr:

$t.Tests.TestCategory | select -Unique | sort
Access control
Access control; Credential management
Advanced Classification
Advanced Label Features
Application management
Application Proxy
Applications management
Azure Network Security
Credential management
Credential management, Privileged access
Data
Data Loss Prevention (DLP)
Data Security Posture Management
Device
Devices
Encryption
Entra
External collaboration
External Identities
Global Secure Access
Hybrid infrastructure
Identity
Identity governance
Information Protection
Label Policy Configuration
Microsoft Purview Message Encryption
Monitoring
Network
Network security
Private Access
Privileged access
Rights Management Service
Rights Management Service (RMS)
Role management
Sensitivity Labels
Sensitivity Labels Configuration
sensitivity-labels
SharePoint Online
Tenant

Copilot AI review requested due to automatic review settings April 19, 2026 18:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR standardizes [ZtTest()] Category metadata values across several PowerShell assessment tests to reduce duplicate categories caused by inconsistent naming (e.g., hyphenated vs. title case, singular vs. plural).

Changes:

  • Normalizes Sensitivity Labels categories to Sensitivity Labels (removing variants like sensitivity-labels and Sensitivity Labels Configuration).
  • Normalizes device-related categories from Devices to Device.
  • Aligns a few remaining category name variants (e.g., Rights Management ServiceRights Management Service (RMS), Applications managementApplication management).

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/powershell/tests/Test-Assessment.35024.ps1 Aligns category to the existing Rights Management Service (RMS) naming.
src/powershell/tests/Test-Assessment.35015.ps1 Changes category from sensitivity-labels to Sensitivity Labels.
src/powershell/tests/Test-Assessment.35013.ps1 Changes category to Sensitivity Labels (but leaves header note category text outdated).
src/powershell/tests/Test-Assessment.35012.ps1 Changes category to Sensitivity Labels (but leaves header note category text outdated).
src/powershell/tests/Test-Assessment.35003.ps1 Changes category from sensitivity-labels to Sensitivity Labels.
src/powershell/tests/Test-Assessment.21955.ps1 Changes category from Devices to Device.
src/powershell/tests/Test-Assessment.21954.ps1 Changes category from Devices to Device.
src/powershell/tests/Test-Assessment.21953.ps1 Changes category from Devices to Device.
src/powershell/tests/Test-Assessment.21886.ps1 Changes category from Applications management to Application management.
src/powershell/tests/Test-Assessment.21837.ps1 Changes category from Devices to Device.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/powershell/tests/Test-Assessment.35012.ps1
Comment thread src/powershell/tests/Test-Assessment.35013.ps1
Copilot AI review requested due to automatic review settings April 19, 2026 19:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants