Skip to content

Conversation

givinalis
Copy link
Collaborator

Changes

  • Added Authentication checks for governance cmdlets namely:
    • Get-EntraDirectoryRoleAssignment
    • Get-EntraDirectoryRoleDefinition
    • Get-EntraUnsupportedCommand
    • New-EntraCustomHeaders
    • New-EntraDirectoryRoleAssignment
    • New-EntraDirectoryRoleDefinition
    • Remove-EntraDirectoryRoleAssignment
    • Remove-EntraDirectoryRoleDefinition
    • Set-EntraDirectoryRoleDefinition
  • Updated tests to mock Get-EntraContext for all the cmdlets listed above.

Issue

Link: #356

@givinalis givinalis requested a review from a team as a code owner August 27, 2025 18:55
Copy link

Learn Build status updates of commit 0bc5f18:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • Line 0, Column 0: [Error: PSMD2Yaml_FileLoadFailed] Failed to load file: C:/LocalRun/W/qjhz-s/module/mapping/monikerMapping.json. PackageRoot, ReferenceTocUrl, and ConceptualTocUrl are required for every moniker. PackageRoot should be a valid relative path to docset root.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Copy link

Learn Build status updates of commit eaf421d:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • Line 0, Column 0: [Error: PSMD2Yaml_FileLoadFailed] Failed to load file: C:/LocalRun/W/nzwa-s/module/mapping/monikerMapping.json. PackageRoot, ReferenceTocUrl, and ConceptualTocUrl are required for every moniker. PackageRoot should be a valid relative path to docset root.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@givinalis givinalis added the Ready For Review The PR Ready for Review label Aug 29, 2025
@givinalis givinalis merged commit 3a7ed33 into main Sep 2, 2025
3 of 4 checks passed
@givinalis givinalis deleted the giomachar/356-EntraGovernance branch September 2, 2025 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review The PR Ready for Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants