Added Test and more name changes

src/ResourceManager/Websites/Commands.Websites.Test/Commands.Websites.Test.csproj

@@ -131,6 +131,7 @@
     <None Include="SessionRecords\Microsoft.Azure.Commands.WebApp.Test.ScenarioTests.WebAppTests\TestCreatesNewSimpleWebApp.json">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
+    <None Include="SessionRecords\Microsoft.Azure.Commands.WebApp.Test.ScenarioTests.WebAppTests\TestSetNewAppServicePlan.json" />
   </ItemGroup>
   <ItemGroup>
     <ProjectReference Include="..\..\..\Common\Commands.Common.Test\Commands.Common.Test.csproj">

src/ResourceManager/Websites/Commands.Websites.Test/ScenarioTests/Common.ps1

@@ -18,7 +18,6 @@ Gets a website name for testing.
 #>
 function Get-WebsiteName
 {
-# need to remove hardcoding
     return getAssetName
 }
 
@@ -28,7 +27,6 @@ Gets a website name for testing.
 #>
 function Get-WebHostPlanName
 {
-# need to remove hardcoding
     return getAssetName 
 }
 

src/ResourceManager/Websites/Commands.Websites.Test/ScenarioTests/WebAppTests.cs

@@ -33,5 +33,12 @@ public void TestCreatesNewAppServicePlan()
         {
             WebsitesController.NewInstance.RunPsTest("Test-CreatesNewAppServicePlan");
         }
+
+        [Fact]
+        [Trait(Category.AcceptanceType, Category.CheckIn)]
+        public void TestSetNewAppServicePlan()
+        {
+            WebsitesController.NewInstance.RunPsTest("Test-SetNewAppServicePlan");
+        }
     }
 }

src/ResourceManager/Websites/Commands.Websites.Test/ScenarioTests/WebAppTests.ps1

@@ -78,4 +78,47 @@ function Test-CreatesNewAppServicePlan
 		Remove-AzureAppServicePlan -ResourceGroupName $rgname -Name  $whpName -Force
 		Remove-AzureResourceGroup -Name $rgname -Force
     }
+}
+
+<#
+.SYNOPSIS
+Tests creating a new Web Hosting Plan.
+#>
+function Test-SetNewAppServicePlan
+{
+	# Setup
+	$rgname = Get-ResourceGroupName
+	$whpName = Get-WebHostPlanName
+	$location = Get-Location
+
+	try
+	{
+		#Setup
+		New-AzureResourceGroup -Name $rgname -Location $location
+		# Test
+		$actual = New-AzureAppServicePlan -ResourceGroupName $rgname -Name  $whpName -Location  $location 
+		$result = Get-AzureAppServicePlan -ResourceGroupName $rgname -Name  $whpName
+		# Assert
+		Assert-AreEqual $whpName $result.WebHostingPlan.Name
+		Assert-AreEqual 1 $result.WebHostingPlan.Properties.NumberOfWorkers
+		Assert-AreEqual "Standard" $result.WebHostingPlan.Properties.Sku
+		Assert-AreEqual "Small" $result.WebHostingPlan.Properties.WorkerSize
+
+		# Test setting the created service plan
+		$newresult = Set-AzureAppServicePlan  -ResourceGroupName $rgname -Name  $whpName -Location  $location -Sku Premium -NumberofWorkers 12 -WorkerSize Medium
+		# due to a bug Set and New are not returning the appropriate object so need to get.
+		$newresult = Get-AzureAppServicePlan -ResourceGroupName $rgname -Name  $whpName
+
+		# Assert
+		Assert-AreEqual $whpName $result.WebHostingPlan.Name
+		Assert-AreEqual 12 $newresult.WebHostingPlan.Properties.NumberOfWorkers
+		Assert-AreEqual "Premium" $newresult.WebHostingPlan.Properties.Sku
+		Assert-AreEqual "Medium" $newresult.WebHostingPlan.Properties.WorkerSize
+	}
+    finally
+    {
+		# Cleanup
+		Remove-AzureAppServicePlan -ResourceGroupName $rgname -Name  $whpName -Force
+		Remove-AzureResourceGroup -Name $rgname -Force
+    }
 }

src/ResourceManager/Websites/Commands.Websites/Cmdlets/AppServicePlan/GetAzureAppServicePlan.cs

@@ -56,12 +56,12 @@ public override void ExecuteCmdlet()
 
         private void GetByWebHostingPlan()
         {
-            WriteObject(WebsitesClient.GetWebHostingPlan(ResourceGroupName, Name));
+            WriteObject(WebsitesClient.GetAppServicePlan(ResourceGroupName, Name));
         }
 
         private void GetByResourceGroup()
         {
-            WriteObject(WebsitesClient.ListWebHostingPlan(ResourceGroupName));
+            WriteObject(WebsitesClient.ListAppServicePlan(ResourceGroupName));
         }
     }
 }

src/ResourceManager/Websites/Commands.Websites/Cmdlets/AppServicePlan/NewAzureAppServicePlan.cs

@@ -116,7 +116,7 @@ public override void ExecuteCmdlet()
                 }
             }
 
-            WriteObject(WebsitesClient.CreateWebHostingPlan(ResourceGroupName, Name, Location, adminSiteName, NumberofWorkers, skuInput, workerSizeInput));
+            WriteObject(WebsitesClient.CreateAppServicePlan(ResourceGroupName, Name, Location, adminSiteName, NumberofWorkers, skuInput, workerSizeInput));
 
         }
 

src/ResourceManager/Websites/Commands.Websites/Cmdlets/AppServicePlan/RemoveAppServicePlan.cs

@@ -48,10 +48,10 @@ public override void ExecuteCmdlet()
         {
             ConfirmAction(
                     Force.IsPresent,
-                    string.Format(Microsoft.Azure.Commands.WebApp.Properties.Resources.RemovingWebHostPlan, Name),
-                    Microsoft.Azure.Commands.WebApp.Properties.Resources.RemovingWebHostPlan,
+                    string.Format(Microsoft.Azure.Commands.WebApp.Properties.Resources.RemovingAppServicePlan, Name),
+                    Microsoft.Azure.Commands.WebApp.Properties.Resources.RemovingAppServicePlan,
                     Name,
-                    () => WebsitesClient.RemoveWebHostingPlan(ResourceGroupName, Name));
+                    () => WebsitesClient.RemoveAppServicePlan(ResourceGroupName, Name));
         }
     }
 }

src/ResourceManager/Websites/Commands.Websites/Cmdlets/AppServicePlan/SetAzureAppServicePlan.cs

@@ -115,7 +115,7 @@ public override void ExecuteCmdlet()
                 }
             }
 
-            WriteObject(WebsitesClient.CreateWebHostingPlan(ResourceGroupName, Name, Location, adminSiteName, NumberofWorkers, skuInput, workerSizeInput));
+            WriteObject(WebsitesClient.CreateAppServicePlan(ResourceGroupName, Name, Location, adminSiteName, NumberofWorkers, skuInput, workerSizeInput));
 
         }
 

src/ResourceManager/Websites/Commands.Websites/Properties/Resources.resx

@@ -117,16 +117,16 @@
   <resheader name="writer">
     <value>System.Resources.ResXResourceWriter, System.Windows.Forms, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
   </resheader>
-  <data name="RemovingWebHostPlan" xml:space="preserve">
-    <value>Are you sure you want to remove web hosting plan '{0}'</value>
+  <data name="RemovingAppServicePlan" xml:space="preserve">
+    <value>Are you sure you want to remove app service plan '{0}'</value>
   </data>
   <data name="RemoveWebsiteMessage" xml:space="preserve">
-    <value>Removing website</value>
+    <value>Removing web app</value>
   </data>
   <data name="RemoveWebsiteWarning" xml:space="preserve">
-    <value>Are you sure you want to remove the website "{0}"</value>
+    <value>Are you sure you want to remove the web app "{0}"</value>
   </data>
   <data name="RemovingWebsite" xml:space="preserve">
-    <value>Are you sure you want to remove website '{0}'</value>
+    <value>Are you sure you want to remove web app '{0}'</value>
   </data>
 </root>

src/ResourceManager/Websites/Commands.Websites/Utilities/WebsitesClient.cs

@@ -125,7 +125,7 @@ public WebSiteGetHistoricalUsageMetricsResponse GetWebAppUsageMetrics(string res
             return usageMetrics;
         }
 
-        public WebHostingPlanCreateOrUpdateResponse CreateWebHostingPlan(string resourceGroupName, string whpName, string location, string adminSiteName, int numberOfWorkers, SkuOptions sku, WorkerSizeOptions workerSize)
+        public WebHostingPlanCreateOrUpdateResponse CreateAppServicePlan(string resourceGroupName, string whpName, string location, string adminSiteName, int numberOfWorkers, SkuOptions sku, WorkerSizeOptions workerSize)
         {
             WebHostingPlanProperties webHostingPlanProperties = new WebHostingPlanProperties();
             webHostingPlanProperties.Sku = sku;
@@ -143,24 +143,21 @@ public WebHostingPlanCreateOrUpdateResponse CreateWebHostingPlan(string resource
             return createdWHP;
         }
 
-        public AzureOperationResponse RemoveWebHostingPlan(string resourceGroupName, string whpName)
+        public AzureOperationResponse RemoveAppServicePlan(string resourceGroupName, string whpName)
         {
             var response = WrappedWebsitesClient.WebHostingPlans.Delete(resourceGroupName, whpName);
-            //proper return type need to be discussed
             return response;
         }
 
-        public WebHostingPlanGetResponse GetWebHostingPlan(string resourceGroupName, string whpName)
+        public WebHostingPlanGetResponse GetAppServicePlan(string resourceGroupName, string whpName)
         {
             var response = WrappedWebsitesClient.WebHostingPlans.Get(resourceGroupName, whpName);
-            //proper return type need to be discussed
             return response;
         }
 
-        public WebHostingPlanListResponse ListWebHostingPlan(string resourceGroupName)
+        public WebHostingPlanListResponse ListAppServicePlan(string resourceGroupName)
         {       
             var response = WrappedWebsitesClient.WebHostingPlans.List(resourceGroupName);
-            //proper return type need to be discussed
             return response;
         }