Merge pull request Azure#2153 from ravbhatnagar/dev

Adding positional params to Resource group cmdlets and updating help

Author: Hovsep

src/ResourceManager/Resources/AzureRM.Resources.psd1

@@ -54,11 +54,14 @@ RequiredAssemblies = @()
 ScriptsToProcess = @()  
 
 # Type files (.ps1xml) to be loaded when importing this module  
-TypesToProcess = @()
+TypesToProcess = @(
+	'Microsoft.Azure.Commands.ResourceManager.Cmdlets.Types.ps1xml'
+)
 
 # Format files (.ps1xml) to be loaded when importing this module  
 FormatsToProcess = @(
-	'.\Microsoft.Azure.Commands.Resources.format.ps1xml'
+	'.\Microsoft.Azure.Commands.Resources.format.ps1xml',
+	'.\Microsoft.Azure.Commands.ResourceManager.Cmdlets.format.ps1xml'
 )
 
 # Modules to import as nested modules of the module specified in ModuleToProcess  

src/ResourceManager/Resources/Commands.ResourceManager/Cmdlets/Microsoft.Azure.Commands.ResourceManager.Cmdlets.dll-Help.xml

@@ -409,6 +409,9 @@
     <None Include="SessionRecords\Microsoft.Azure.Commands.Resources.Test.ScenarioTests.ResourceGroupTests\TestRemoveNonExistingResourceGroup.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
+    <None Include="SessionRecords\Microsoft.Azure.Commands.Resources.Test.ScenarioTests.ResourceGroupTests\TestResourceGroupWithPositionalParams.json">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
     <None Include="SessionRecords\Microsoft.Azure.Commands.Resources.Test.ScenarioTests.ResourceGroupTests\TestUpdatesExistingResourceGroup.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>

src/ResourceManager/Resources/Commands.Resources.Test/Commands.Resources.Test.csproj

@@ -76,6 +76,13 @@ public void TestExportResourceGroup()
             ResourcesController.NewInstance.RunPsTest("Test-ExportResourceGroup");
         }
 
+        [Fact]
+        [Trait(Category.AcceptanceType, Category.CheckIn)]
+        public void TestResourceGroupWithPositionalParams()
+        {
+            ResourcesController.NewInstance.RunPsTest("Test-ResourceGroupWithPositionalParams");
+        }
+
         [Fact (Skip = "TODO: Fix the broken test.")]
         public void TestAzureTagsEndToEnd()
         {

src/ResourceManager/Resources/Commands.Resources.Test/ScenarioTests/ResourceGroupTests.cs

@@ -350,4 +350,34 @@ function Test-ExportResourceGroup
         # Cleanup
         Clean-ResourceGroup $rgname
     }
+}
+
+<#
+.SYNOPSIS
+Tests resource group new, get and remove using positional parameters.
+#>
+function Test-ResourceGroupWithPositionalParams
+{
+    # Setup
+    $rgname = Get-ResourceGroupName
+    $location = "West US"
+
+    try
+    {
+        $ErrorActionPreference = "SilentlyContinue"
+        $Error.Clear()
+        # Test
+        $actual = New-AzureRmResourceGroup $rgname $location
+        $expected = Get-AzureRmResourceGroup $rgname
+
+        # Assert
+        Assert-AreEqual $expected.ResourceGroupName $actual.ResourceGroupName
+
+        #Test
+        Remove-AzureRmResourceGroup $rgname -Force
+    }
+    catch
+    {
+        Assert-True { $Error[0].Contains("Provided resource group does not exist.") }
+    }
 }