Skip to content

Conversation

@sandeepjha000
Copy link
Collaborator

Data-35016: adding test

@sandeepjha000 sandeepjha000 marked this pull request as draft January 8, 2026 22:16
@sandeepjha000 sandeepjha000 self-assigned this Jan 8, 2026
@sandeepjha000 sandeepjha000 added the ready for review PR is ready for review and merging label Jan 9, 2026
@sandeepjha000 sandeepjha000 marked this pull request as ready for review January 9, 2026 04:46
@alexandair alexandair requested a review from Copilot January 9, 2026 08:00
Copy link
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 pull request adds a new security assessment test (ID: 35016) to verify that mandatory labeling is enabled for sensitivity labels across Microsoft 365 workloads (Outlook, Teams, SharePoint/Groups, and Power BI). The test retrieves enabled label policies, parses their settings to check for mandatory labeling flags across different workloads, and generates a comprehensive report showing the current configuration status.

Key changes:

  • New PowerShell test script that queries label policies and evaluates mandatory labeling settings
  • Documentation explaining the security rationale and remediation steps
  • Support for checking mandatory labeling across four workloads with override detection

Reviewed changes

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

File Description
src/powershell/tests/Test-Assessment.35016.ps1 Implements the assessment logic to check sensitivity label policies for mandatory labeling configuration, including data collection, policy parsing, and markdown report generation
src/powershell/tests/Test-Assessment.35016.md Provides comprehensive documentation explaining security risks of not having mandatory labeling and detailed remediation steps for each workload

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

@alexandair
Copy link
Collaborator

@sandeepjha000
I've added a commit to refactor test code to use XML parsing instead of regex. Could you test, please?

@sandeepjha000 sandeepjha000 marked this pull request as draft January 12, 2026 17:12
@sandeepjha000 sandeepjha000 removed the ready for review PR is ready for review and merging label Jan 12, 2026
@sandeepjha000
Copy link
Collaborator Author

sandeepjha000 commented Jan 12, 2026

@sandeepjha000 I've added a commit to refactor test code to use XML parsing instead of regex. Could you test, please?

@alexandair - I am able to get the same test results. Thanks

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.

3 participants