Skip to content

Commit be80c19

Browse files
Fix duplicate Example 3 numbering and use space-separated Boolean syntax in tests
1 parent 10034c7 commit be80c19

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

src/Functions/Functions.Autorest/docs/Update-AzFunctionApp.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ Update-AzFunctionApp -Name MyUniqueFunctionAppName -ResourceGroupName MyResource
5454

5555
This command updates function app Application Insights.
5656

57-
### Example 3: Remove managed identity from a function app.
57+
### Example 4: Remove managed identity from a function app.
5858
```powershell
5959
Update-AzFunctionApp -Name MyUniqueFunctionAppName -ResourceGroupName MyResourceGroupName -EnableSystemAssignedIdentity $false -Force
6060
```

src/Functions/Functions.Autorest/examples/Update-AzFunctionApp.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ This command sets a SystemAssigned managed identity for a function app.
1313
Update-AzFunctionApp -Name MyUniqueFunctionAppName -ResourceGroupName MyResourceGroupName -ApplicationInsightsName ApplicationInsightsProjectName -Force
1414
```
1515
This command updates function app Application Insights.
16-
### Example 3: Remove managed identity from a function app.
16+
### Example 4: Remove managed identity from a function app.
1717
```powershell
1818
Update-AzFunctionApp -Name MyUniqueFunctionAppName -ResourceGroupName MyResourceGroupName -EnableSystemAssignedIdentity $false -Force
1919
```

src/Functions/Functions.Autorest/test/New-Update-Remove-AzFunctionApp.Tests.ps1

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -239,11 +239,11 @@ Describe 'New-AzFunctionApp, Update-AzFunctionApp, and Remove-AzFunctionApp E2E'
239239
$functionApp.AppServicePlan | Should -Be $newPlanName
240240

241241
# Update test to use -InputObject when https://github.com/Azure/azure-powershell/issues/23266 is fixed
242-
# Update-AzFunctionApp -InputObject $functionApp -EnableSystemAssignedIdentity:$false -Force
243-
# Update-AzFunctionApp -Name $appName -ResourceGroupName $resourceGroupName -EnableSystemAssignedIdentity:$false -Force
242+
# Update-AzFunctionApp -InputObject $functionApp -EnableSystemAssignedIdentity $false -Force
243+
# Update-AzFunctionApp -Name $appName -ResourceGroupName $resourceGroupName -EnableSystemAssignedIdentity $false -Force
244244

245245
Write-Verbose "Update function -> remove SystemAssigned managed identity" -Verbose
246-
Update-AzFunctionApp -Name $appName -ResourceGroupName $resourceGroupName -EnableSystemAssignedIdentity:$false -Force
246+
Update-AzFunctionApp -Name $appName -ResourceGroupName $resourceGroupName -EnableSystemAssignedIdentity $false -Force
247247

248248
$functionApp = Get-AzFunctionApp -Name $appName -ResourceGroupName $resourceGroupName
249249
Write-Verbose "FunctionApp after identity removal. Validate IdentityType" -Verbose

src/Functions/Functions/help/Update-AzFunctionApp.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ Update-AzFunctionApp -Name MyUniqueFunctionAppName -ResourceGroupName MyResource
5454

5555
This command updates function app Application Insights.
5656

57-
### Example 3: Remove managed identity from a function app.
57+
### Example 4: Remove managed identity from a function app.
5858
```powershell
5959
Update-AzFunctionApp -Name MyUniqueFunctionAppName -ResourceGroupName MyResourceGroupName -EnableSystemAssignedIdentity $false -Force
6060
```

0 commit comments

Comments
 (0)