Skip to content

Comments

Data-35011 bug fix#906

Merged
SagarSathe merged 7 commits intomainfrom
Data-35011-fix-bug
Feb 23, 2026
Merged

Data-35011 bug fix#906
SagarSathe merged 7 commits intomainfrom
Data-35011-fix-bug

Conversation

@komalp2025
Copy link
Collaborator

@komalp2025 komalp2025 commented Feb 17, 2026

35011 spec's evaluation logic along with remediation resources is updated. Created this PR to update the new changes.

Results are updated here- https://github.com/microsoft/ztspecs/issues/65#issuecomment-3912746451

@komalp2025 komalp2025 added the ready for review PR is ready for review and merging label Feb 18, 2026
@alexandair alexandair requested a review from Copilot February 18, 2026 12:55
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

Updates Data test 35011 to align with the latest spec evaluation logic and remediation guidance for Azure Information Protection (AIP) Super User feature configuration.

Changes:

  • Updated pass/fail/investigate logic based on feature state and configured super user count.
  • Revised result messaging and expanded report details to always include configuration summary and member listing.
  • Refreshed remediation guidance to emphasize temporary enablement and Azure PIM just-in-time access.

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.35011.ps1 Updates evaluation logic and restructures markdown result generation for new spec outcomes.
src/powershell/tests/Test-Assessment.35011.md Updates remediation steps and best-practice guidance (temporary enablement + PIM).

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

@komalp2025 komalp2025 self-assigned this Feb 19, 2026
@komalp2025 komalp2025 added ready for review PR is ready for review and merging and removed ready for review PR is ready for review and merging labels Feb 19, 2026
Copy link
Collaborator

@alexandair alexandair left a comment

Choose a reason for hiding this comment

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

LGTM

@SagarSathe SagarSathe merged commit 7b7b35a into main Feb 23, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review PR is ready for review and merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants