Skip to content

Commit 48b7f02

Browse files
DeepakRajendranMsftHovsep
authored andcommitted
Minor fixes and enable tests (Azure#2662)
1 parent aa032d2 commit 48b7f02

File tree

8 files changed

+1753
-1748
lines changed

8 files changed

+1753
-1748
lines changed

src/ResourceManager/Network/Commands.Network.Test/ScenarioTests/LoadBalancerTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ public LoadBalancerTests(ITestOutputHelper output)
2626
XunitTracingInterceptor.AddToContext(new XunitTracingInterceptor(output));
2727
}
2828

29-
[Fact(Skip = "ClientRuntime upgrade - Please re-record")]
29+
[Fact]
3030
[Trait(Category.AcceptanceType, Category.CheckIn)]
3131
public void TestLoadBalancerCRUDPublic()
3232
{
@@ -131,7 +131,7 @@ public void TestLoadBalancerMultiVipPublic()
131131
NetworkResourcesController.NewInstance.RunPsTest("Test-LoadBalancerMultiVip-Public");
132132
}
133133

134-
[Fact(Skip = "ClientRuntime upgrade - Please re-record")]
134+
[Fact]
135135
[Trait(Category.AcceptanceType, Category.CheckIn)]
136136
public void TestLoadBalancerMultiVipInternal()
137137
{

src/ResourceManager/Network/Commands.Network.Test/ScenarioTests/NetworkInterfaceTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ public void TestNetworkInterfaceExpandResource()
8282
NetworkResourcesController.NewInstance.RunPsTest("Test-NetworkInterfaceExpandResource");
8383
}
8484

85-
[Fact(Skip ="ClientRuntime upgrade - Please re-record")]
85+
[Fact]
8686
[Trait(Category.AcceptanceType, Category.CheckIn)]
8787
public void TestNetworkInterfaceIpv6()
8888
{

src/ResourceManager/Network/Commands.Network.Test/ScenarioTests/NetworkInterfaceTests.ps1

Lines changed: 35 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -591,6 +591,7 @@ function Test-NetworkInterfaceIpv6
591591
$rgname = Get-ResourceGroupName
592592
$vnetName = Get-ResourceName
593593
$subnetName = Get-ResourceName
594+
$subnet2Name = Get-ResourceName
594595
$publicIpName = Get-ResourceName
595596
$nicName = Get-ResourceName
596597
$ipconfigName = Get-ResourceName
@@ -606,7 +607,8 @@ function Test-NetworkInterfaceIpv6
606607

607608
# Create the Virtual Network
608609
$subnet = New-AzureRmVirtualNetworkSubnetConfig -Name $subnetName -AddressPrefix 10.0.1.0/24
609-
$vnet = New-AzureRmvirtualNetwork -Name $vnetName -ResourceGroupName $rgname -Location $location -AddressPrefix 10.0.0.0/16 -Subnet $subnet
610+
$subnet2 = New-AzureRmVirtualNetworkSubnetConfig -Name $subnet2Name -AddressPrefix 10.0.2.0/24
611+
$vnet = New-AzureRmvirtualNetwork -Name $vnetName -ResourceGroupName $rgname -Location $location -AddressPrefix 10.0.0.0/16 -Subnet $subnet,$subnet2
610612

611613
# Create the publicip
612614
$publicip = New-AzureRmPublicIpAddress -ResourceGroupName $rgname -name $publicIpName -location $location -AllocationMethod Dynamic -DomainNameLabel $domainNameLabel
@@ -638,33 +640,33 @@ function Test-NetworkInterfaceIpv6
638640
$nic = Get-AzureRmNetworkInterface -Name $nicName -ResourceGroupName $rgname | Add-AzureRmNetworkInterfaceIpConfig -Name $ipconfigName -PrivateIpAddressVersion ipv6 | Set-AzureRmNetworkInterface
639641
Assert-AreEqual 2 @($nic.IpConfigurations).Count
640642

641-
Assert-AreEqual $expectedNic.IpConfigurations[0].Name $nic.IpConfigurations[1].Name
642-
Assert-AreEqual $publicip.Id $nic.IpConfigurations[1].PublicIpAddress.Id
643-
Assert-AreEqual $vnet.Subnets[0].Id $nic.IpConfigurations[1].Subnet.Id
644-
Assert-NotNull $nic.IpConfigurations[1].PrivateIpAddress
645-
Assert-AreEqual "Dynamic" $nic.IpConfigurations[1].PrivateIpAllocationMethod
646-
Assert-AreEqual $nic.IpConfigurations[1].PrivateIpAddressVersion IPv4
643+
Assert-AreEqual $expectedNic.IpConfigurations[0].Name $nic.IpConfigurations[0].Name
644+
Assert-AreEqual $publicip.Id $nic.IpConfigurations[0].PublicIpAddress.Id
645+
Assert-AreEqual $vnet.Subnets[0].Id $nic.IpConfigurations[0].Subnet.Id
646+
Assert-NotNull $nic.IpConfigurations[0].PrivateIpAddress
647+
Assert-AreEqual "Dynamic" $nic.IpConfigurations[0].PrivateIpAllocationMethod
648+
Assert-AreEqual $nic.IpConfigurations[0].PrivateIpAddressVersion IPv4
647649

648-
Assert-AreEqual $ipconfigName $nic.IpConfigurations[0].Name
649-
Assert-Null $nic.IpConfigurations[0].PublicIpAddress
650-
Assert-Null $nic.IpConfigurations[0].Subnet
651-
Assert-AreEqual $nic.IpConfigurations[0].PrivateIpAddressVersion IPv6
650+
Assert-AreEqual $ipconfigName $nic.IpConfigurations[1].Name
651+
Assert-Null $nic.IpConfigurations[1].PublicIpAddress
652+
Assert-Null $nic.IpConfigurations[1].Subnet
653+
Assert-AreEqual $nic.IpConfigurations[1].PrivateIpAddressVersion IPv6
652654

653655
# Set Ipconfig
654-
$nic = Get-AzureRmNetworkInterface -Name $nicName -ResourceGroupName $rgname | Set-AzureRmNetworkInterfaceIpConfig -Name $nic.IpConfigurations[1].Name -PrivateIpAddressVersion ipv4 -Subnet $nic.IpConfigurations[1].Subnet | Set-AzureRmNetworkInterface
656+
$nic = Get-AzureRmNetworkInterface -Name $nicName -ResourceGroupName $rgname | Set-AzureRmNetworkInterfaceIpConfig -Name $nic.IpConfigurations[0].Name -Subnet $vnet.Subnets[1] -PrivateIpAddress "10.0.2.10" | Set-AzureRmNetworkInterface
655657
Assert-AreEqual 2 @($nic.IpConfigurations).Count
656658

657-
Assert-AreEqual $expectedNic.IpConfigurations[0].Name $nic.IpConfigurations[1].Name
658-
Assert-Null $nic.IpConfigurations[1].PublicIpAddress
659-
Assert-AreEqual $vnet.Subnets[0].Id $nic.IpConfigurations[1].Subnet.Id
660-
Assert-NotNull $nic.IpConfigurations[1].PrivateIpAddress
661-
Assert-AreEqual "Dynamic" $nic.IpConfigurations[1].PrivateIpAllocationMethod
662-
Assert-AreEqual $nic.IpConfigurations[1].PrivateIpAddressVersion IPv4
663-
664-
Assert-AreEqual $ipconfigName $nic.IpConfigurations[0].Name
659+
Assert-AreEqual $expectedNic.IpConfigurations[0].Name $nic.IpConfigurations[0].Name
665660
Assert-Null $nic.IpConfigurations[0].PublicIpAddress
666-
Assert-Null $nic.IpConfigurations[0].Subnet
667-
Assert-AreEqual $nic.IpConfigurations[0].PrivateIpAddressVersion IPv6
661+
Assert-AreEqual $vnet.Subnets[1].Id $nic.IpConfigurations[0].Subnet.Id
662+
Assert-NotNull $nic.IpConfigurations[0].PrivateIpAddress
663+
Assert-AreEqual "Static" $nic.IpConfigurations[0].PrivateIpAllocationMethod
664+
Assert-AreEqual $nic.IpConfigurations[0].PrivateIpAddressVersion IPv4
665+
666+
Assert-AreEqual $ipconfigName $nic.IpConfigurations[1].Name
667+
Assert-Null $nic.IpConfigurations[1].PublicIpAddress
668+
Assert-Null $nic.IpConfigurations[1].Subnet
669+
Assert-AreEqual $nic.IpConfigurations[1].PrivateIpAddressVersion IPv6
668670

669671
# Get IpConfig
670672
$ipconfigv6 = Get-AzureRmNetworkInterface -Name $nicName -ResourceGroupName $rgname | Get-AzureRmNetworkInterfaceIpConfig -Name $ipconfigName
@@ -679,17 +681,16 @@ function Test-NetworkInterfaceIpv6
679681

680682
Assert-AreEqual 2 @($ipconfigList).Count
681683

682-
Assert-AreEqual $ipconfigList[1].Name $nic.IpConfigurations[1].Name
683-
Assert-Null $ipconfigList[1].PublicIpAddress
684-
Assert-AreEqual $vnet.Subnets[0].Id $ipconfigList[1].Subnet.Id
685-
Assert-NotNull $ipconfigList[1].PrivateIpAddress
686-
Assert-AreEqual "Dynamic" $ipconfigList[1].PrivateIpAllocationMethod
687-
Assert-AreEqual $ipconfigList[1].PrivateIpAddressVersion IPv4
684+
Assert-AreEqual $expectedNic.IpConfigurations[0].Name $ipconfigList[0].Name
685+
Assert-Null $ipconfigList[0].PublicIpAddress.Id
686+
Assert-NotNull $ipconfigList[0].PrivateIpAddress
687+
Assert-AreEqual "Static" $nic.IpConfigurations[0].PrivateIpAllocationMethod
688+
Assert-AreEqual $ipconfigList[0].PrivateIpAddressVersion IPv4
688689

689-
Assert-AreEqual $ipconfigList[0].Name $nic.IpConfigurations[0].Name
690-
Assert-Null $ipconfigList[0].PublicIpAddress
691-
Assert-Null $ipconfigList[0].Subnet
692-
Assert-AreEqual $ipconfigList[0].PrivateIpAddressVersion IPv6
690+
Assert-AreEqual $ipconfigName $ipconfigList[1].Name
691+
Assert-Null $ipconfigList[1].PublicIpAddress
692+
Assert-Null $ipconfigList[1].Subnet
693+
Assert-AreEqual $ipconfigList[1].PrivateIpAddressVersion IPv6
693694

694695
# Remove IpConfig
695696
$nic = Get-AzureRmNetworkInterface -Name $nicName -ResourceGroupName $rgname | Remove-AzureRmNetworkInterfaceIpConfig -Name $ipconfigName | Set-AzureRmNetworkInterface
@@ -698,9 +699,9 @@ function Test-NetworkInterfaceIpv6
698699

699700
Assert-AreEqual $expectedNic.IpConfigurations[0].Name $nic.IpConfigurations[0].Name
700701
Assert-Null $nic.IpConfigurations[0].PublicIpAddress
701-
Assert-AreEqual $vnet.Subnets[0].Id $nic.IpConfigurations[0].Subnet.Id
702+
Assert-AreEqual $vnet.Subnets[1].Id $nic.IpConfigurations[0].Subnet.Id
702703
Assert-NotNull $nic.IpConfigurations[0].PrivateIpAddress
703-
Assert-AreEqual "Dynamic" $nic.IpConfigurations[0].PrivateIpAllocationMethod
704+
Assert-AreEqual "Static" $nic.IpConfigurations[0].PrivateIpAllocationMethod
704705
Assert-AreEqual $nic.IpConfigurations[0].PrivateIpAddressVersion IPv4
705706

706707
# Delete NetworkInterface

0 commit comments

Comments
 (0)