[Do not merge!] Yanxu/for test #28149

YanaXu · 2025-07-08T05:56:56Z

Description

Mandatory Checklist

Please choose the target release of Azure PowerShell. (⚠️Target release is a different concept from API readiness. Please click below links for details.)
- General release
- Public preview
- Private preview
- Engineering build
- No need for a release
Check this box to confirm: I have read the Submitting Changes section of CONTRIBUTING.md and reviewed the following information:

SHOULD update ChangeLog.md file(s) appropriately
- Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
  - A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
- Should not change ChangeLog.md if no new release is required, such as fixing test case only.
SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
SHOULD have proper test coverage for changes in pull request.
SHOULD NOT adjust version of module manually in pull request

azure-client-tools-bot-prd · 2025-07-08T05:56:59Z

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

github-actions · 2025-07-08T05:57:32Z

‼️ DO NOT MERGE THIS PR ‼️
This PR was labeled "Do Not Merge" because it contains code change that cannot be merged. Please contact the reviewer for more information.

Copilot

Pull Request Overview

This PR adds temporary debug logging to the TestAutorest workflow, fixes several spelling mistakes in help markdown files, and updates module scaffolding, including ChangeLog entries and newly generated Authorization.Autorest artifacts.

Inserted Write-Host debug messages around the TestAutorest block in ExecuteCIStep.ps1
Corrected typos in multiple help .md files (Management, eligible, resourceType)
Updated ChangeLog.md with breaking-change announcements and scaffolded new Authorization.Autorest files

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
tools/ExecuteCIStep.ps1	Added `Write-Host` debug statements in TestAutorest
src/Resources/Resources/help/Save-AzManagementGroupDeploymentStackTemplate.md	Fixed typo in parameter set name
src/Resources/Resources/help/New-AzRoleEligibilityScheduleRequest.md	Fixed spelling of “eligible”
src/Resources/Resources/help/Get-AzRoleEligibleChildResource.md	Corrected filter variable name typo
src/Resources/Resources/help/Get-AzManagementGroupDeploymentStack.md	Fixed typo in parameter set name

Copilot · 2025-07-08T05:58:20Z

tools/ExecuteCIStep.ps1

+    Write-Host "--------------------------------- TestAutorest step 1 ---------------------------------"
    If (-not (Test-Path "$AutorestDirectory/test-module.ps1"))
    {
        Write-Warning "There is no test-module.ps1 found in the folder: $AutorestDirectory"
        Return
    }
+    Write-Host "--------------------------------- TestAutorest step 2 ---------------------------------"
    $ModuleName = Split-Path -Path $AutorestDirectory | Split-Path -Leaf
+    Write-Host "--------------------------------- TestAutorest step 3 ---------------------------------"
    $ModuleFolderName = $ModuleName.Split(".")[1]
+    Write-Host "--------------------------------- TestAutorest step 4 ---------------------------------"
    If (Test-Path $CIPlanPath)
    {
+        Write-Host "--------------------------------- TestAutorest step 5 ---------------------------------"
        $CIPlan = Get-Content $CIPlanPath | ConvertFrom-Json
+        Write-Host "--------------------------------- TestAutorest step 6 ---------------------------------"
        If (-not ($CIPlan.test.Contains($ModuleFolderName)))
        {
            Write-Debug "Skip test for $ModuleName because it is not in the test plan."
            Return
        }
+        Write-Host "--------------------------------- TestAutorest step 7 ---------------------------------"
        . $AutorestDirectory/test-module.ps1
+        Write-Host "--------------------------------- TestAutorest step 8 ---------------------------------"


[nitpick] These Write-Host statements appear to be temporary debug logs; consider converting them to Write-Verbose or removing them to keep CI logs clean.

Suggested change

Write-Host "--------------------------------- TestAutorest step 1 ---------------------------------"

If (-not (Test-Path "$AutorestDirectory/test-module.ps1"))

{

Write-Warning "There is no test-module.ps1 found in the folder: $AutorestDirectory"

Return

}

Write-Host "--------------------------------- TestAutorest step 2 ---------------------------------"

$ModuleName = Split-Path -Path $AutorestDirectory | Split-Path -Leaf

Write-Host "--------------------------------- TestAutorest step 3 ---------------------------------"

$ModuleFolderName = $ModuleName.Split(".")[1]

Write-Host "--------------------------------- TestAutorest step 4 ---------------------------------"

If (Test-Path $CIPlanPath)

{

Write-Host "--------------------------------- TestAutorest step 5 ---------------------------------"

$CIPlan = Get-Content $CIPlanPath | ConvertFrom-Json

Write-Host "--------------------------------- TestAutorest step 6 ---------------------------------"

If (-not ($CIPlan.test.Contains($ModuleFolderName)))

{

Write-Debug "Skip test for $ModuleName because it is not in the test plan."

Return

}

Write-Host "--------------------------------- TestAutorest step 7 ---------------------------------"

. $AutorestDirectory/test-module.ps1

Write-Host "--------------------------------- TestAutorest step 8 ---------------------------------"

Write-Verbose "--------------------------------- TestAutorest step 1 ---------------------------------"

If (-not (Test-Path "$AutorestDirectory/test-module.ps1"))

{

Write-Warning "There is no test-module.ps1 found in the folder: $AutorestDirectory"

Return

}

Write-Verbose "--------------------------------- TestAutorest step 2 ---------------------------------"

$ModuleName = Split-Path -Path $AutorestDirectory | Split-Path -Leaf

Write-Verbose "--------------------------------- TestAutorest step 3 ---------------------------------"

$ModuleFolderName = $ModuleName.Split(".")[1]

Write-Verbose "--------------------------------- TestAutorest step 4 ---------------------------------"

If (Test-Path $CIPlanPath)

{

Write-Verbose "--------------------------------- TestAutorest step 5 ---------------------------------"

$CIPlan = Get-Content $CIPlanPath | ConvertFrom-Json

Write-Verbose "--------------------------------- TestAutorest step 6 ---------------------------------"

If (-not ($CIPlan.test.Contains($ModuleFolderName)))

{

Write-Debug "Skip test for $ModuleName because it is not in the test plan."

Return

}

Write-Verbose "--------------------------------- TestAutorest step 7 ---------------------------------"

. $AutorestDirectory/test-module.ps1

Write-Verbose "--------------------------------- TestAutorest step 8 ---------------------------------"

Copilot · 2025-07-08T05:58:21Z

src/Resources/Resources/help/Save-AzManagementGroupDeploymentStackTemplate.md

@@ -12,7 +12,7 @@ Saves a Management Group scoped Deployment Stack Template.

 ## SYNTAX

-### SaveByNameAndManagementGroupId (Default)
+### SaveByNameAndManagmentGroupId (Default)


Typo in 'ManagmentGroupId'; it should be 'ManagementGroupId'.

Suggested change

### SaveByNameAndManagmentGroupId (Default)

### SaveByNameAndManagementGroupId (Default)

Copilot · 2025-07-08T05:58:21Z

src/Resources/Resources/help/New-AzRoleEligibilityScheduleRequest.md

@@ -27,7 +27,7 @@ Creates a role eligibility schedule request.

 ## EXAMPLES

-### Example 1: Create a new role eligible schedule request as Admin
+### Example 1: Create a new role eligibile schedule request as Admin


Typo in 'eligibile'; it should be 'eligible'.

Suggested change

### Example 1: Create a new role eligibile schedule request as Admin

### Example 1: Create a new role eligible schedule request as Admin

Copilot · 2025-07-08T05:58:22Z

src/Resources/Resources/help/Get-AzRoleEligibleChildResource.md

@@ -49,7 +49,7 @@ Get all child resources of a resource `scope` that the calling user has eligible
 ### Example 2: List all child resources filtered by resource type
 ```powershell
 $scope = "/subscriptions/38ab2ccc-3747-4567-b36b-9478f5602f0d/"
-$filter = "resourceType eq 'resourcegroup'"
+$filter = "resoureType eq 'resourcegroup'"


Typo in 'resoureType'; it should be 'resourceType'.

Suggested change

$filter = "resoureType eq 'resourcegroup'"

$filter = "resourceType eq 'resourcegroup'"

Copilot · 2025-07-08T05:58:22Z

src/Resources/Resources/help/Get-AzManagementGroupDeploymentStack.md

@@ -12,7 +12,7 @@ Gets Management Group scoped Deployment Stacks.

 ## SYNTAX

-### ListByManagementGroupId (Default)
+### ListByManagmentGroupId (Default)


Typo in 'ManagmentGroupId'; it should be 'ManagementGroupId'.

Suggested change

### ListByManagmentGroupId (Default)

### ListByManagementGroupId (Default)

YanaXu · 2025-07-14T05:37:58Z

Add a timestamp.

JoyerJin and others added 6 commits June 12, 2025 16:41

add breaking change announcement

c6c3da0

Update ChangeLog.md

f4e98e6

Merge branch 'main' into joyer/resource-breakingchange

54a3868

merge main

6a33026

Regenerate with latest autorest to sync the change

75eb752

add debug info

d75f42f

Copilot AI review requested due to automatic review settings July 8, 2025 05:56

YanaXu added the Do Not Merge 🚫 label Jul 8, 2025

Copilot AI reviewed Jul 8, 2025

View reviewed changes

YanaXu added 2 commits July 8, 2025 14:04

debug 2

80b5c26

debug 3

53e2679

YanaXu closed this Jul 21, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Do not merge!] Yanxu/for test #28149

[Do not merge!] Yanxu/for test #28149

Uh oh!

YanaXu commented Jul 8, 2025

Uh oh!

azure-client-tools-bot-prd bot commented Jul 8, 2025

Uh oh!

github-actions bot commented Jul 8, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jul 8, 2025

Uh oh!

Copilot AI Jul 8, 2025

Uh oh!

Copilot AI Jul 8, 2025

Uh oh!

Copilot AI Jul 8, 2025

Uh oh!

Copilot AI Jul 8, 2025

Uh oh!

YanaXu commented Jul 14, 2025

Uh oh!

Uh oh!

	### SaveByNameAndManagmentGroupId (Default)
	### SaveByNameAndManagementGroupId (Default)

	### Example 1: Create a new role eligibile schedule request as Admin
	### Example 1: Create a new role eligible schedule request as Admin

	$filter = "resoureType eq 'resourcegroup'"
	$filter = "resourceType eq 'resourcegroup'"

	### ListByManagmentGroupId (Default)
	### ListByManagementGroupId (Default)

[Do not merge!] Yanxu/for test #28149

[Do not merge!] Yanxu/for test #28149

Uh oh!

Conversation

YanaXu commented Jul 8, 2025

Description

Mandatory Checklist

Uh oh!

azure-client-tools-bot-prd bot commented Jul 8, 2025

Uh oh!

github-actions bot commented Jul 8, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

Uh oh!

YanaXu commented Jul 14, 2025

Uh oh!

Uh oh!