Update command

Chenglong Liu · Chenglong Liu · commit f2c758f142a3 · 2022-03-07T15:18:44.000+08:00
diff --git a/src/Cdn/Cdn.Autorest/custom/Get-AzCdnProfile.ps1 b/src/Cdn/Cdn.Autorest/custom/Get-AzCdnProfile.ps1
@@ -82,34 +82,25 @@ function Get-AzCdnProfile {
 
     process {
         $internalProfiles = Az.Cdn.internal\Get-AzCdnProfile @PSBoundParameters
-        Write-Host -ForegroundColor Green $internalProfiles.GetType()
-
-        if ($null -eq $internalProfiles )
-        {
-            $profiles = $null
-        }else
+ 
+        if ($null -ne $internalProfiles )
         {
             if($internalProfiles -is [array]){
-                $profiles = @()
                 foreach ($oneInternalProfile in $internalProfiles)
                 {
                     if(-Not (ISFrontDoorCdnProfile($oneInternalProfile.SkuName)))
                     {
-                        $profiles += $oneInternalProfile
+                        Write-Output $oneInternalProfile
                     }
                 }
             }else
             {
                 $oneInternalProfile = $internalProfiles
                 if(-Not (ISFrontDoorCdnProfile($oneInternalProfile.SkuName)))
                 {
-                    $profiles = $internalProfiles
-                }else{
-                    $profiles = $null
+                    Write-Output $oneInternalProfile
                 }
             }
         } 
-
-        Write-Output $profiles
     }
 }
diff --git a/src/Cdn/Cdn.Autorest/custom/Get-AzFrontDoorCdnProfile.ps1 b/src/Cdn/Cdn.Autorest/custom/Get-AzFrontDoorCdnProfile.ps1
@@ -82,32 +82,24 @@ function Get-AzFrontDoorCdnProfile {
 
     process {
         $internalProfiles = Az.Cdn.internal\Get-AzCdnProfile @PSBoundParameters
-        if ($null -eq $internalProfiles )
-        {
-            $profiles = $null
-        }else
+        if ($null -ne $internalProfiles )
         {
             if($internalProfiles -is [array]){
-                $profiles = @()
                 foreach ($oneInternalProfile in $internalProfiles)
                 {
                     if(ISFrontDoorCdnProfile($oneInternalProfile.SkuName))
                     {
-                        $profiles += $oneInternalProfile
+                        Write-Output $oneInternalProfile
                     }
                 }
             }else
             {
                 $oneInternalProfile = $internalProfiles
                 if(ISFrontDoorCdnProfile($oneInternalProfile.SkuName))
                 {
-                    $profiles = $oneInternalProfile
-                }else{
-                    $profiles = $null
+                    Write-Output $oneInternalProfile
                 }
             }
         } 
-
-        Write-Output $profiles
     }
 }
diff --git a/src/Cdn/Cdn.Autorest/custom/Remove-AzFrontDoorCdnProfile.ps1 b/src/Cdn/Cdn.Autorest/custom/Remove-AzFrontDoorCdnProfile.ps1
@@ -156,11 +156,19 @@ function Remove-AzFrontDoorCdnProfile {
     
     process {
 
-        $frontDoorCdnProfile = Get-AzFrontDoorCdnProfile -ResourceGroupName ${ResourceGroupName} -Name ${Name}
+        if ($PSCmdlet.ParameterSetName -eq 'Delete') {
+            $frontDoorCdnProfile = Get-AzFrontDoorCdnProfile -ResourceGroupName ${ResourceGroupName} -Name ${Name}
+        } elseif ($PSCmdlet.ParameterSetName -eq 'DeleteViaIdentity') {
+            $frontDoorCdnProfile = $InputObject
+        }else {
+            throw "Not supported ParameterSetName."
+        }
+
         if($null -eq $frontDoorCdnProfile)
         {
             throw "Provided FrontDoorCdnProfile does not exist."
         }else{
+            Write-Host -ForegroundColor Green "Skuname $($frontDoorCdnProfile.SkuName)"
             if(ISFrontDoorCdnProfile($frontDoorCdnProfile.SkuName)){
                 Az.Cdn.internal\Remove-AzCdnProfile @PSBoundParameters
             }else{
diff --git a/src/Cdn/Cdn.Autorest/test/Remove-AzFrontDoorCdnProfile.Tests.ps1 b/src/Cdn/Cdn.Autorest/test/Remove-AzFrontDoorCdnProfile.Tests.ps1
@@ -15,7 +15,7 @@ if(($null -eq $TestName) -or ($TestName -contains 'Remove-AzFrontDoorCdnProfile'
 }
 
 Describe 'Remove-AzFrontDoorCdnProfile' {
-    It 'Delete' {
+    It 'Delete' -skip{
         { 
             $ResourceGroupName = 'testps-rg-' + (RandomString -allChars $false -len 6)
             try
@@ -28,15 +28,32 @@ Describe 'Remove-AzFrontDoorCdnProfile' {
 
                 $profileSku = "Standard_AzureFrontDoor";
                 New-AzFrontDoorCdnProfile -SkuName $profileSku -Name $frontDoorCdnProfileName -ResourceGroupName $ResourceGroupName -Location Global
-                #Remove-AzFrontDoorCdnProfile -Name $frontDoorCdnProfileName -ResourceGroupName $ResourceGroupName
+                Remove-AzFrontDoorCdnProfile -Name $frontDoorCdnProfileName -ResourceGroupName $ResourceGroupName
             } Finally
             {
-                #Remove-AzResourceGroup -Name $ResourceGroupName -NoWait
+                Remove-AzResourceGroup -Name $ResourceGroupName -NoWait
             }
         } | Should -Not -Throw
     }
 
-    It 'DeleteViaIdentity' -skip {
-        { throw [System.NotImplementedException] } | Should -Not -Throw
+    It 'DeleteViaIdentity' {
+        { 
+            $ResourceGroupName = 'testps-rg-' + (RandomString -allChars $false -len 6)
+            try
+            {
+                Write-Host -ForegroundColor Green "Create test group $($ResourceGroupName)"
+                New-AzResourceGroup -Name $ResourceGroupName -Location $env.location
+
+                $frontDoorCdnProfileName = 'fdp-' + (RandomString -allChars $false -len 6);
+                Write-Host -ForegroundColor Green "Use frontDoorCdnProfileName : $($frontDoorCdnProfileName)"
+
+                $profileSku = "Standard_AzureFrontDoor";
+                New-AzFrontDoorCdnProfile -SkuName $profileSku -Name $frontDoorCdnProfileName -ResourceGroupName $ResourceGroupName -Location Global
+                Get-AzFrontDoorCdnProfile -ResourceGroupName $ResourceGroupName -Name $frontDoorCdnProfileName | Remove-AzFrontDoorCdnProfile
+            } Finally
+            {
+                Remove-AzResourceGroup -Name $ResourceGroupName -NoWait
+            }
+        } | Should -Not -Throw
     }
 }
diff --git a/src/Cdn/Cdn.Autorest/test/localEnv.json b/src/Cdn/Cdn.Autorest/test/localEnv.json
@@ -1,5 +1,5 @@
 {
-  "Tenant": "72f988bf-86f1-41af-91ab-2d7cd011db47",
+  "location": "westus",
   "SubscriptionId": "d7cfdb98-c118-458d-8bdf-246be66b1f5e",
-  "location": "westus"
+  "Tenant": "72f988bf-86f1-41af-91ab-2d7cd011db47"
 }

Original file line number	Diff line number	Diff line change
`@@ -82,34 +82,25 @@ function Get-AzCdnProfile {`
`82`	`82`
`83`	`83`	`process {`
`84`	`84`	`$internalProfiles = Az.Cdn.internal\Get-AzCdnProfile @PSBoundParameters`
`85`		`- Write-Host -ForegroundColor Green $internalProfiles.GetType()`
`86`		`-`
`87`		`- if ($null -eq $internalProfiles )`
`88`		`- {`
`89`		`- $profiles = $null`
`90`		`- }else`
	`85`	`+`
	`86`	`+ if ($null -ne $internalProfiles )`
`91`	`87`	`{`
`92`	`88`	`if($internalProfiles -is [array]){`
`93`		`- $profiles = @()`
`94`	`89`	`foreach ($oneInternalProfile in $internalProfiles)`
`95`	`90`	`{`
`96`	`91`	`if(-Not (ISFrontDoorCdnProfile($oneInternalProfile.SkuName)))`
`97`	`92`	`{`
`98`		`- $profiles += $oneInternalProfile`
	`93`	`+ Write-Output $oneInternalProfile`
`99`	`94`	`}`
`100`	`95`	`}`
`101`	`96`	`}else`
`102`	`97`	`{`
`103`	`98`	`$oneInternalProfile = $internalProfiles`
`104`	`99`	`if(-Not (ISFrontDoorCdnProfile($oneInternalProfile.SkuName)))`
`105`	`100`	`{`
`106`		`- $profiles = $internalProfiles`
`107`		`- }else{`
`108`		`- $profiles = $null`
	`101`	`+ Write-Output $oneInternalProfile`
`109`	`102`	`}`
`110`	`103`	`}`
`111`	`104`	`}`
`112`		`-`
`113`		`- Write-Output $profiles`
`114`	`105`	`}`
`115`	`106`	`}`
Original file line number	Diff line number	Diff line change
`@@ -82,32 +82,24 @@ function Get-AzFrontDoorCdnProfile {`
`82`	`82`
`83`	`83`	`process {`
`84`	`84`	`$internalProfiles = Az.Cdn.internal\Get-AzCdnProfile @PSBoundParameters`
`85`		`- if ($null -eq $internalProfiles )`
`86`		`- {`
`87`		`- $profiles = $null`
`88`		`- }else`
	`85`	`+ if ($null -ne $internalProfiles )`
`89`	`86`	`{`
`90`	`87`	`if($internalProfiles -is [array]){`
`91`		`- $profiles = @()`
`92`	`88`	`foreach ($oneInternalProfile in $internalProfiles)`
`93`	`89`	`{`
`94`	`90`	`if(ISFrontDoorCdnProfile($oneInternalProfile.SkuName))`
`95`	`91`	`{`
`96`		`- $profiles += $oneInternalProfile`
	`92`	`+ Write-Output $oneInternalProfile`
`97`	`93`	`}`
`98`	`94`	`}`
`99`	`95`	`}else`
`100`	`96`	`{`
`101`	`97`	`$oneInternalProfile = $internalProfiles`
`102`	`98`	`if(ISFrontDoorCdnProfile($oneInternalProfile.SkuName))`
`103`	`99`	`{`
`104`		`- $profiles = $oneInternalProfile`
`105`		`- }else{`
`106`		`- $profiles = $null`
	`100`	`+ Write-Output $oneInternalProfile`
`107`	`101`	`}`
`108`	`102`	`}`
`109`	`103`	`}`
`110`		`-`
`111`		`- Write-Output $profiles`
`112`	`104`	`}`
`113`	`105`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`		`- "Tenant": "72f988bf-86f1-41af-91ab-2d7cd011db47",`
	`2`	`+ "location": "westus",`
`3`	`3`	`"SubscriptionId": "d7cfdb98-c118-458d-8bdf-246be66b1f5e",`
`4`		`- "location": "westus"`
	`4`	`+ "Tenant": "72f988bf-86f1-41af-91ab-2d7cd011db47"`
`5`	`5`	`}`