Skip to content

Conversation

givinalis
Copy link
Collaborator

Changes

  • Added Authentication checks for these cmdlets under DirectoryManagement submodule for Entra:

    • Add-EntraAdministrativeUnitMember
    • Add-EntraCustomSecurityAttributeDefinitionAllowedValue
    • Add-EntraDeviceRegisteredOwner
    • Add-EntraDeviceRegisteredUser
    • Add-EntraDirectoryRoleMember
    • Add-EntraScopedRoleMembership
    • Confirm-EntraDomain
    • Enable-EntraDirectoryRole
    • Get-EntraAccountSku
    • Get-EntraAdministrativeUnit
    • Get-EntraAdministrativeUnitMember
    • Get-EntraAttributeSet
    • Get-EntraContact
    • Get-EntraContactDirectReport
    • Get-EntraContactManager
    • Get-EntraContactMembership
    • Get-EntraContract
    • Get-EntraCustomSecurityAttributeDefinition
    • Get-EntraCustomSecurityAttributeDefinitionAllowedValue
    • Get-EntraDeletedAdministrativeUnit
    • Get-EntraDeletedDevice
    • Get-EntraDeletedDirectoryObject
    • Get-EntraDevice
    • Get-EntraDeviceRegisteredOwner
    • Get-EntraDeviceRegisteredUser
    • Get-EntraDirectoryObject
    • Get-EntraDirectoryObjectOnPremisesProvisioningError
    • Get-EntraDirectoryRole
    • Get-EntraDirectoryRoleMember
    • Get-EntraDirectoryRoleTemplate
    • Get-EntraDirSyncConfiguration
    • Get-EntraDirSyncFeature
    • Get-EntraDomain
    • Get-EntraDomainFederationSettings
    • Get-EntraDomainNameReference
    • Get-EntraDomainServiceConfigurationRecord
    • Get-EntraDomainVerificationDnsRecord
    • Get-EntraExtensionProperty
    • Get-EntraFederationProperty
    • Get-EntraPartnerInformation
    • Get-EntraPasswordPolicy
    • Get-EntraScopedRoleMembership
    • Get-EntraSubscribedSku
    • Get-EntraSubscription
    • Get-EntraTenantDetail
    • Get-EntraUnsupportedCommand
    • New-EntraAdministrativeUnit
    • New-EntraAttributeSet
    • New-EntraCustomHeaders
    • New-EntraCustomSecurityAttributeDefinition
    • New-EntraDevice
    • New-EntraDomain
    • Remove-EntraAdministrativeUnit
    • Remove-EntraAdministrativeUnitMember
    • Remove-EntraContact
    • Remove-EntraDevice
    • Remove-EntraDeviceRegisteredOwner
    • Remove-EntraDeviceRegisteredUser
    • Remove-EntraDirectoryRoleMember
    • Remove-EntraDomain
    • Remove-EntraScopedRoleMembership
    • Resolve-EntraTenant
    • Restore-EntraDeletedDirectoryObject
    • Set-EntraAdministrativeUnit
    • Set-EntraAttributeSet
    • Set-EntraCustomSecurityAttributeDefinition
    • Set-EntraCustomSecurityAttributeDefinitionAllowedValue
    • Set-EntraDevice
    • Set-EntraDirSyncConfiguration
    • Set-EntraDirSyncEnabled
    • Set-EntraDirSyncFeature
    • Set-EntraDomain
    • Set-EntraDomainFederationSettings
    • Set-EntraPartnerInformation
    • Set-EntraTenantDetail
  • Updated tests to mock Get-EntraContext for all the cmdlets listed above.

  • Updated ALL tests for Entra to include authentication checks to validate the scenario.

Issue

Link: #356

Copy link

Learn Build status updates of commit 85dfc14:

❌ 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/tfbm-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 marked this pull request as ready for review September 8, 2025 17:30
@givinalis givinalis requested a review from a team as a code owner September 8, 2025 17:30
@givinalis givinalis added the Ready For Review The PR Ready for Review label Sep 8, 2025
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