Skip to content

Commit fae8ece

Browse files
committed
Merge pull request Azure#165 from huangpf/crp2
fix tests
2 parents a3a5b97 + 8fd98d4 commit fae8ece

File tree

3 files changed

+2004
-1599
lines changed

3 files changed

+2004
-1599
lines changed

src/ResourceManager/Compute/Commands.Compute.Test/ScenarioTests/VirtualMachineTests.ps1

Lines changed: 19 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -332,10 +332,10 @@ function Test-VirtualMachineImageList
332332
Assert-ThrowsContains { $s2 = Get-AzureVMImageOffer -Location $locStr -PublisherName $publisherName; } "$publisherName was not found";
333333

334334
$offerName = Get-ComputeTestResourceName;
335-
Assert-ThrowsContains { $s3 = Get-AzureVMImageSku -Location $locStr -PublisherName $publisherName -Offer $offerName; } "$offerName was not found";
335+
Assert-ThrowsContains { $s3 = Get-AzureVMImageSku -Location $locStr -PublisherName $publisherName -Offer $offerName; } "was not found";
336336

337337
$skusName = Get-ComputeTestResourceName;
338-
Assert-ThrowsContains { $s4 = Get-AzureVMImage -Location $locStr -PublisherName $publisherName -Offer $offerName -Skus $skusName; } "$skusName was not found";
338+
Assert-ThrowsContains { $s4 = Get-AzureVMImage -Location $locStr -PublisherName $publisherName -Offer $offerName -Skus $skusName; } "was not found";
339339

340340
$filter = "name eq 'latest'";
341341
Assert-ThrowsContains { $s5 = Get-AzureVMImage -Location $locStr -PublisherName $publisherName -Offer $offerName -Skus $skusName -FilterExpression $filter; } "was not found";
@@ -372,27 +372,18 @@ function Test-VirtualMachineSizeAndUsage
372372
try
373373
{
374374
# Common
375-
$loc = 'West US';
375+
$loc = 'eastasia';
376376
New-AzureResourceGroup -Name $rgname -Location $loc;
377-
377+
378378
# Availability Set
379379
$asetName = 'aset' + $rgname;
380380
New-AzureAvailabilitySet -ResourceGroupName $rgname -Name $asetName -Location $loc;
381-
382-
$asets = Get-AzureAvailabilitySet -ResourceGroupName $rgname;
383-
Assert-NotNull $asets;
384-
Assert-AreEqual $asetName $asets[0].Name;
385-
386381
$aset = Get-AzureAvailabilitySet -ResourceGroupName $rgname -Name $asetName;
387-
Assert-NotNull $aset;
388-
Assert-AreEqual $asetName $aset.Name;
389-
390-
$asetId = ('/subscriptions/' + (Get-AzureSubscription -Current).SubscriptionId + '/resourceGroups/' + $rgname + '/providers/Microsoft.Compute/availabilitySets/' + $asetName);
391382

392383
# VM Profile & Hardware
393-
$vmsize = 'Standard_A2';
384+
$vmsize = 'Standard_A1';
394385
$vmname = 'vm' + $rgname;
395-
$p = New-AzureVMConfig -VMName $vmname -VMSize $vmsize -AvailabilitySetId $asetId;
386+
$p = New-AzureVMConfig -VMName $vmname -VMSize $vmsize -AvailabilitySetId $aset.Id;
396387
Assert-AreEqual $p.HardwareProfile.VirtualMachineSize $vmsize;
397388

398389
# NRP
@@ -453,31 +444,27 @@ function Test-VirtualMachineSizeAndUsage
453444
$vhdContainer = "https://$stoname.blob.core.windows.net/test";
454445
$img = 'a699494373c04fc0bc8f2bb1389d6106__Windows-Server-2012-Datacenter-201503.01-en.us-127GB.vhd';
455446

456-
$p.StorageProfile.OSDisk = $null;
447+
# $p.StorageProfile.OSDisk = $null;
457448
$p = Set-AzureVMOperatingSystem -VM $p -Windows -ComputerName $computerName -Credential $cred;
458-
$p = Set-AzureVMSourceImage -VM $p -Name $img;
459449

460450
Assert-AreEqual $p.OSProfile.AdminUsername $user;
461451
Assert-AreEqual $p.OSProfile.ComputerName $computerName;
462452
Assert-AreEqual $p.OSProfile.AdminPassword $password;
463-
Assert-AreEqual $p.StorageProfile.SourceImage.ReferenceUri ('/' + (Get-AzureSubscription -Current).SubscriptionId + '/services/images/' + $img);
464-
453+
454+
# Image Reference;
455+
$p.StorageProfile.SourceImage = $null;
456+
$imgRef = Get-DefaultCRPImage;
457+
$p = Set-AzureVMSourceImage -VM $p -ImageReference $imgRef;
458+
Assert-NotNull $p.StorageProfile.ImageReference;
459+
Assert-Null $p.StorageProfile.SourceImageId;
460+
465461
# TODO: Remove Data Disks for now
466462
$p.StorageProfile.DataDisks = $null;
467463

468-
# Create VM
464+
# Virtual Machine
465+
# TODO: Still need to do retry for New-AzureVM for SA, even it's returned in Get-.
469466
New-AzureVM -ResourceGroupName $rgname -Location $loc -Name $vmname -VM $p;
470467

471-
# Get VM
472-
$vm1 = Get-AzureVM -Name $vmname -ResourceGroupName $rgname;
473-
Assert-AreEqual $vm1.Name $vmname;
474-
Assert-AreEqual $vm1.NetworkProfile.NetworkInterfaces.Count 1;
475-
Assert-AreEqual $vm1.NetworkProfile.NetworkInterfaces[0].ReferenceUri $nicId;
476-
Assert-AreEqual $vm1.StorageProfile.SourceImage.ReferenceUri ('/' + (Get-AzureSubscription -Current).SubscriptionId + '/services/images/' + $img);
477-
Assert-AreEqual $vm1.OSProfile.AdminUsername $user;
478-
Assert-AreEqual $vm1.OSProfile.ComputerName $computerName;
479-
Assert-AreEqual $vm1.HardwareProfile.VirtualMachineSize $vmsize;
480-
481468
# Test Sizes
482469
$s1 = Get-AzureVMSize -Location ($loc -replace ' ');
483470
Assert-NotNull $s1;
@@ -487,13 +474,14 @@ function Test-VirtualMachineSizeAndUsage
487474
Assert-NotNull $s2;
488475
Validate-VirtualMachineSize $vmsize $s2;
489476

477+
$asetName = $aset.Name;
490478
$s3 = Get-AzureVMSize -ResourceGroupName $rgname -AvailabilitySetName $asetName;
491479
Assert-NotNull $s3;
492480
Validate-VirtualMachineSize $vmsize $s3;
493481

494482
# Test Usage
495483
$u1 = Get-AzureVMUsage -Location ($loc -replace ' ');
496-
Validate-VirtualMachineUsage $u1.Usages;
484+
Validate-VirtualMachineUsage $u1;
497485

498486
$passed = $true;
499487
}

0 commit comments

Comments
 (0)