Skip to content

Commit 3e3ce29

Browse files
committed
Addressed PR feedback
1 parent f921a7c commit 3e3ce29

File tree

12 files changed

+2103
-1738
lines changed

12 files changed

+2103
-1738
lines changed

src/ResourceManager/Network/Commands.Network.Test/ScenarioTests/Generated/ApplicationSecurityGroupTestsGenerated.ps1

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ function Test-ApplicationSecurityGroupCRUDMinimalParameters
4646
$rgname = Get-ResourceGroupName;
4747
$rglocation = Get-ProviderLocation ResourceManagement;
4848
$rname = Get-ResourceName;
49-
$location = "westcentralus";
49+
$location = Get-ProviderLocation "Microsoft.Network/ApplicationSecurityGroups";
5050

5151
try
5252
{
@@ -67,6 +67,9 @@ function Test-ApplicationSecurityGroupCRUDMinimalParameters
6767
# Remove ApplicationSecurityGroup
6868
$removeApplicationSecurityGroup = Remove-AzureRmApplicationSecurityGroup -ResourceGroupName $rgname -Name $rname -PassThru -Force;
6969
Assert-AreEqual $true $removeApplicationSecurityGroup;
70+
71+
# Get ApplicationSecurityGroup should fail
72+
Assert-ThrowsContains { Get-AzureRmApplicationSecurityGroup -ResourceGroupName $rgname -Name $rname } "${rname} not found";
7073
}
7174
finally
7275
{
@@ -85,7 +88,7 @@ function Test-ApplicationSecurityGroupCRUDAllParameters
8588
$rgname = Get-ResourceGroupName;
8689
$rglocation = Get-ProviderLocation ResourceManagement;
8790
$rname = Get-ResourceName;
88-
$location = "westcentralus";
91+
$location = Get-ProviderLocation "Microsoft.Network/ApplicationSecurityGroups";
8992
# Resource's parameters
9093
$Tag = @{tag1='test'};
9194

@@ -110,6 +113,9 @@ function Test-ApplicationSecurityGroupCRUDAllParameters
110113
# Remove ApplicationSecurityGroup
111114
$removeApplicationSecurityGroup = Remove-AzureRmApplicationSecurityGroup -ResourceGroupName $rgname -Name $rname -PassThru -Force;
112115
Assert-AreEqual $true $removeApplicationSecurityGroup;
116+
117+
# Get ApplicationSecurityGroup should fail
118+
Assert-ThrowsContains { Get-AzureRmApplicationSecurityGroup -ResourceGroupName $rgname -Name $rname } "${rname} not found";
113119
}
114120
finally
115121
{

src/ResourceManager/Network/Commands.Network.Test/ScenarioTests/Generated/RouteTableTestsGenerated.ps1

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,9 @@ function Test-RouteTableCRUDMinimalParameters
7575
# Remove RouteTable
7676
$removeRouteTable = Remove-AzureRmRouteTable -ResourceGroupName $rgname -Name $rname -PassThru -Force;
7777
Assert-AreEqual $true $removeRouteTable;
78+
79+
# Get RouteTable should fail
80+
Assert-ThrowsContains { Get-AzureRmRouteTable -ResourceGroupName $rgname -Name $rname } "${rname} not found";
7881
}
7982
finally
8083
{
@@ -143,6 +146,9 @@ function Test-RouteTableCRUDAllParameters
143146
# Remove RouteTable
144147
$removeRouteTable = Remove-AzureRmRouteTable -ResourceGroupName $rgname -Name $rname -PassThru -Force;
145148
Assert-AreEqual $true $removeRouteTable;
149+
150+
# Get RouteTable should fail
151+
Assert-ThrowsContains { Get-AzureRmRouteTable -ResourceGroupName $rgname -Name $rname } "${rname} not found";
146152
}
147153
finally
148154
{
@@ -215,8 +221,11 @@ function Test-RouteCRUDMinimalParameters
215221
Assert-AreEqual $AddressPrefixAdd $vRoute.AddressPrefix;
216222

217223
# Remove Route
218-
Remove-AzureRmRouteConfig -RouteTable $vRouteTable -Name $vRouteTable.Routes[-1].Name;
224+
Remove-AzureRmRouteConfig -RouteTable $vRouteTable -Name $rnameAdd;
219225
$vRouteTable = Set-AzureRmRouteTable -RouteTable $vRouteTable;
226+
227+
# Get Route should fail
228+
Assert-ThrowsContains { Get-AzureRmRouteConfig -RouteTable $vRouteTable -Name $rnameAdd } "Sequence contains no matching element";
220229
}
221230
finally
222231
{
@@ -296,8 +305,11 @@ function Test-RouteCRUDAllParameters
296305
Assert-AreEqual $NextHopTypeAdd $vRoute.NextHopType;
297306

298307
# Remove Route
299-
Remove-AzureRmRouteConfig -RouteTable $vRouteTable -Name $vRouteTable.Routes[-1].Name;
308+
Remove-AzureRmRouteConfig -RouteTable $vRouteTable -Name $rnameAdd;
300309
$vRouteTable = Set-AzureRmRouteTable -RouteTable $vRouteTable;
310+
311+
# Get Route should fail
312+
Assert-ThrowsContains { Get-AzureRmRouteConfig -RouteTable $vRouteTable -Name $rnameAdd } "Sequence contains no matching element";
301313
}
302314
finally
303315
{

src/ResourceManager/Network/Commands.Network.Test/SessionRecords/Commands.Network.Test.ScenarioTests.ApplicationSecurityGroupTestsGenerated/TestApplicationSecurityGroupCRUDAllParameters.json

Lines changed: 292 additions & 168 deletions
Large diffs are not rendered by default.

src/ResourceManager/Network/Commands.Network.Test/SessionRecords/Commands.Network.Test.ScenarioTests.ApplicationSecurityGroupTestsGenerated/TestApplicationSecurityGroupCRUDMinimalParameters.json

Lines changed: 290 additions & 166 deletions
Large diffs are not rendered by default.

src/ResourceManager/Network/Commands.Network.Test/SessionRecords/Commands.Network.Test.ScenarioTests.RouteTableTestsGenerated/TestRouteCRUDAllParameters.json

Lines changed: 405 additions & 441 deletions
Large diffs are not rendered by default.

src/ResourceManager/Network/Commands.Network.Test/SessionRecords/Commands.Network.Test.ScenarioTests.RouteTableTestsGenerated/TestRouteCRUDMinimalParameters.json

Lines changed: 421 additions & 457 deletions
Large diffs are not rendered by default.

src/ResourceManager/Network/Commands.Network.Test/SessionRecords/Commands.Network.Test.ScenarioTests.RouteTableTestsGenerated/TestRouteTableCRUDAllParameters.json

Lines changed: 368 additions & 283 deletions
Large diffs are not rendered by default.

src/ResourceManager/Network/Commands.Network.Test/SessionRecords/Commands.Network.Test.ScenarioTests.RouteTableTestsGenerated/TestRouteTableCRUDMinimalParameters.json

Lines changed: 292 additions & 207 deletions
Large diffs are not rendered by default.

src/ResourceManager/Network/Commands.Network.Test/SessionRecords/Commands.Network.Test.ScenarioTests.UsageTestsGenerated/TestUsageCRUDMinimalParameters.json

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,15 @@
77
"RequestBody": "",
88
"RequestHeaders": {
99
"x-ms-client-request-id": [
10-
"27840b46-6a74-455e-8019-9c7b3f2f34e7"
10+
"c0334623-627b-4192-8b6d-844b8f950ec9"
1111
],
1212
"accept-language": [
1313
"en-US"
1414
],
1515
"User-Agent": [
16-
"FxVersion/4.7.2633.0",
16+
"FxVersion/4.7.3131.0",
1717
"OSName/Windows10Enterprise",
18-
"OSVersion/6.3.16299",
18+
"OSVersion/6.3.17134",
1919
"Microsoft.Azure.Management.Network.NetworkManagementClient/19.1.0.0"
2020
]
2121
},
@@ -34,10 +34,10 @@
3434
"no-cache"
3535
],
3636
"x-ms-request-id": [
37-
"1f205b7a-3257-4c2a-a3c8-e1c6ac753233"
37+
"49822452-e103-4688-bfbe-95f3aefb3f6a"
3838
],
3939
"x-ms-correlation-request-id": [
40-
"ac462f22-893d-4f06-a62c-a6f00be687c4"
40+
"a4dcbd8b-7e03-4b9c-ba53-5a2ae840a75e"
4141
],
4242
"Strict-Transport-Security": [
4343
"max-age=31536000; includeSubDomains"
@@ -50,24 +50,24 @@
5050
"Microsoft-HTTPAPI/2.0"
5151
],
5252
"x-ms-ratelimit-remaining-subscription-reads": [
53-
"14999"
53+
"14981"
5454
],
5555
"x-ms-routing-request-id": [
56-
"FRANCECENTRAL:20180725T124245Z:ac462f22-893d-4f06-a62c-a6f00be687c4"
56+
"UKSOUTH2:20180801T111430Z:a4dcbd8b-7e03-4b9c-ba53-5a2ae840a75e"
5757
],
5858
"X-Content-Type-Options": [
5959
"nosniff"
6060
],
6161
"Date": [
62-
"Wed, 25 Jul 2018 12:42:44 GMT"
62+
"Wed, 01 Aug 2018 11:14:29 GMT"
6363
]
6464
},
6565
"StatusCode": 200
6666
}
6767
],
6868
"Names": {
6969
"Test-UsageCRUDMinimalParameters": [
70-
"ps4962"
70+
"ps5054"
7171
]
7272
},
7373
"Variables": {

src/ResourceManager/Network/Commands.Network/Generated/Usage/GetAzureRMUsageListCommand.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838

3939
namespace Microsoft.Azure.Commands.Network
4040
{
41-
[Cmdlet(VerbsCommon.Get, "AzureRmNetworkUsage"), OutputType(typeof(List<PSUsage>))]
41+
[Cmdlet(VerbsCommon.Get, "AzureRmNetworkUsage"), OutputType(typeof(PSUsage))]
4242
public partial class GetAzureRmNetworkUsage : NetworkBaseCmdlet
4343
{
4444
[Parameter(

src/ResourceManager/Network/Commands.Network/help/Remove-AzureRmApplicationSecurityGroup.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,9 @@ This command deletes an application security group named MyApplicationSecurityGr
3232
## PARAMETERS
3333

3434
### -AsJob
35-
Run cmdlet in the background```yaml
35+
Run cmdlet in the background
36+
37+
```yaml
3638
Type: System.Management.Automation.SwitchParameter
3739
Parameter Sets: (All)
3840
Aliases:

tools/StaticAnalysis/Exceptions/AzureRM.Network/BreakingChangeIssues.csv

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,4 +24,3 @@
2424
"Microsoft.Azure.Commands.Network.dll","Microsoft.Azure.Commands.Network.GetAzureVirtualNetworkGatewayBGPPeerStatus","Get-AzureRmVirtualNetworkGatewayBGPPeerStatus","0","1020","The cmdlet 'Get-AzureRmVirtualNetworkGatewayBGPPeerStatus' no longer has output type 'Microsoft.Azure.Commands.Network.Models.PSBGPPeerStatus[]'.","Make cmdlet 'Get-AzureRmVirtualNetworkGatewayBGPPeerStatus' return type 'Microsoft.Azure.Commands.Network.Models.PSBGPPeerStatus[]'."
2525
"Microsoft.Azure.Commands.Network.dll","Microsoft.Azure.Commands.Network.GetVirtualNetworkGatewayLearnedRouteCommand","Get-AzureRmVirtualNetworkGatewayLearnedRoute","0","1020","The cmdlet 'Get-AzureRmVirtualNetworkGatewayLearnedRoute' no longer has output type 'Microsoft.Azure.Commands.Network.Models.PSGatewayRoute[]'.","Make cmdlet 'Get-AzureRmVirtualNetworkGatewayLearnedRoute' return type 'Microsoft.Azure.Commands.Network.Models.PSGatewayRoute[]'."
2626
"Microsoft.Azure.Commands.Network.dll","Microsoft.Azure.Commands.Network.NewAzureRmRouteTable","New-AzureRmRouteTable","0","2090","The ValidateNotNullOrEmpty attribute has been added to parameter 'Location' for cmdlet 'New-AzureRmRouteTable'.","Remove the ValidateNotNullOrEmpty attribute from parameter 'Location'."
27-
"Microsoft.Azure.Commands.Network.dll","Microsoft.Azure.Commands.Network.Automation.GetAzureRmNetworkUsage","Get-AzureRmNetworkUsage","0","1020","The cmdlet 'Get-AzureRmNetworkUsage' no longer has output type 'Microsoft.Azure.Commands.Network.Models.PSUsage'.","Make cmdlet 'Get-AzureRmNetworkUsage' return type 'Microsoft.Azure.Commands.Network.Models.PSUsage'."

0 commit comments

Comments
 (0)