-
Notifications
You must be signed in to change notification settings - Fork 118
35005 - Add test for sensitivity labels in SharePoint Online assessment #729
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 adds a new assessment test (ID 35005) to check whether sensitivity labels are enabled in SharePoint Online and OneDrive for Business. The test verifies that the EnableAIPIntegration setting is enabled, which allows users to apply Microsoft Information Protection labels to documents stored in these services.
- Implements assessment function
Test-Assessment-35005to check SharePoint Online sensitivity label integration - Adds comprehensive unit tests covering success, failure, and error scenarios
- Provides remediation documentation with PowerShell commands and Microsoft Learn references
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
src/powershell/tests/Test-Assessment.35005.ps1 |
Implements the main assessment logic to query SharePoint tenant settings and evaluate EnableAIPIntegration status |
src/powershell/tests/Test-Assessment.35005.md |
Provides documentation explaining the security importance and remediation steps |
code-tests/test-assessments/Test-Assessment.35005.Tests.ps1 |
Adds Pester tests for three scenarios: query errors, disabled integration, and enabled integration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
contant --> content Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this 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 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…iles Rename $testResultDetail to $params for consistency with other test files Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this 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 3 out of 3 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.
Fix https://github.com/microsoft/ztspecs/issues/51