Skip to content
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

Usability Params for 'Set-EntraBetaUserLicense' and 'Set-EntraBetaUserPassword' #1181

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

emmanuel-karanja
Copy link
Collaborator

Usability Params for 'Set-EntraBetaUserLicense' and 'Set-EntraBetaUserPassword'

Copy link

Learn Build status updates of commit c351e08:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserExtension.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserLicense.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserPassword.md ✅Succeeded View (entra-powershell-beta-preview)
module/EntraBeta/customizations/Set-EntraBetaUserLicense.ps1 ✅Succeeded
module/EntraBeta/customizations/Set-EntraBetaUserPassword.ps1 ✅Succeeded
src/CompatibilityAdapterBuilder.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@emmanuel-karanja emmanuel-karanja added the P0 Highest priority label Oct 31, 2024
Copy link

Learn Build status updates of commit 62b4dba:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserExtension.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserLicense.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserPassword.md ✅Succeeded View (entra-powershell-beta-preview)
module/EntraBeta/customizations/Set-EntraBetaUserLicense.ps1 ✅Succeeded
module/EntraBeta/customizations/Set-EntraBetaUserPassword.ps1 ✅Succeeded
src/CompatibilityAdapterBuilder.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserLicense.Tests.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserPassword.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 17109ed:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserExtension.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserLicense.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserPassword.md ✅Succeeded View (entra-powershell-beta-preview)
module/EntraBeta/customizations/Set-EntraBetaUserLicense.ps1 ✅Succeeded
module/EntraBeta/customizations/Set-EntraBetaUserPassword.ps1 ✅Succeeded
src/CompatibilityAdapterBuilder.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserLicense.Tests.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserPassword.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Should -Invoke -CommandName Update-MgUser -ModuleName Microsoft.Graph.Entra.Beta -Times 1
}
It "Should fail when UserId is empty" {
$userUPN="[email protected]"
Copy link
Collaborator

Choose a reason for hiding this comment

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

We don't use this variable in this test

{ Set-EntraBetaUserPassword -UserId -Password $secPassword } | Should -Throw "Missing an argument for parameter 'UserId'*"
}
It "Should fail when UserId is invalid" {
$userUPN="[email protected]"
Copy link
Collaborator

Choose a reason for hiding this comment

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

We don't use this variable in this test

Copy link

Learn Build status updates of commit ece21ba:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserExtension.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserLicense.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserPassword.md ✅Succeeded View (entra-powershell-beta-preview)
module/EntraBeta/customizations/Set-EntraBetaUserLicense.ps1 ✅Succeeded
module/EntraBeta/customizations/Set-EntraBetaUserPassword.ps1 ✅Succeeded
src/CompatibilityAdapterBuilder.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserLicense.Tests.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserPassword.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit e028cc8:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserExtension.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserLicense.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserPassword.md ✅Succeeded View (entra-powershell-beta-preview)
module/EntraBeta/customizations/Set-EntraBetaUserLicense.ps1 ✅Succeeded
module/EntraBeta/customizations/Set-EntraBetaUserPassword.ps1 ✅Succeeded
src/CompatibilityAdapterBuilder.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserLicense.Tests.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserPassword.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@KenitoInc KenitoInc force-pushed the enganga/usability-params-for-beta branch from e028cc8 to 1c9dd3a Compare October 31, 2024 12:31
Copy link

Learn Build status updates of commit 1c9dd3a:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserExtension.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserLicense.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserPassword.md ✅Succeeded View (entra-powershell-beta-preview)
module/EntraBeta/customizations/Set-EntraBetaUserLicense.ps1 ✅Succeeded
module/EntraBeta/customizations/Set-EntraBetaUserPassword.ps1 ✅Succeeded
src/CompatibilityAdapterBuilder.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserLicense.Tests.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserPassword.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

KenitoInc
KenitoInc previously approved these changes Oct 31, 2024
Copy link

Learn Build status updates of commit 58c8b54:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserExtension.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserLicense.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserPassword.md ✅Succeeded View (entra-powershell-beta-preview)
module/EntraBeta/customizations/Set-EntraBetaUserLicense.ps1 ✅Succeeded
module/EntraBeta/customizations/Set-EntraBetaUserPassword.ps1 ✅Succeeded
src/CompatibilityAdapterBuilder.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserLicense.Tests.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserPassword.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 5080787:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserExtension.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserLicense.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserPassword.md ✅Succeeded View (entra-powershell-beta-preview)
module/EntraBeta/customizations/Set-EntraBetaUserLicense.ps1 ✅Succeeded
module/EntraBeta/customizations/Set-EntraBetaUserPassword.ps1 ✅Succeeded
src/CompatibilityAdapterBuilder.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserLicense.Tests.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserPassword.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@KenitoInc KenitoInc force-pushed the enganga/usability-params-for-beta branch from 5080787 to 08f0673 Compare November 13, 2024 11:50
Copy link

Learn Build status updates of commit 08f0673:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserExtension.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserLicense.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/Microsoft.Graph.Entra.Beta/Set-EntraBetaUserPassword.md ✅Succeeded View (entra-powershell-beta-preview)
module/EntraBeta/customizations/Set-EntraBetaUserLicense.ps1 ✅Succeeded
module/EntraBeta/customizations/Set-EntraBetaUserPassword.ps1 ✅Succeeded
src/CompatibilityAdapterBuilder.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserLicense.Tests.ps1 ✅Succeeded
test/module/EntraBeta/Set-EntraBetaUserPassword.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 Highest priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants