Updated code based on pull request feedback

Author: Matt-Westphal

src/ResourceManager/Commerce/Commands.UsageAggregates.Test/ScenarioTests/UsageAggregatesTests.cs

@@ -16,10 +16,8 @@
 
 namespace Microsoft.Azure.Commands.UsageAggregates.Test.ScenarioTests
 {
-
     public class UsageAggregatesTests 
     {
-      
          [Fact]
         public void TestGetUsageAggregatesWithDefaultParameters()
         {

src/ResourceManager/Commerce/Commands.UsageAggregates/GetUsageAggregatesCommand.cs

@@ -27,6 +27,8 @@ namespace Microsoft.Azure.Commands.UsageAggregates
     [Cmdlet(VerbsCommon.Get, "UsageAggregates"), OutputType(typeof(UsageAggregationGetResponse))]
     public class GetUsageAggregatesCommand : AzurePSCmdlet
     {
+        private UsageAggregationManagementClient _theClient;
+
         [Parameter(Mandatory = true, HelpMessage = "The start of the time range to retrieve data for.")]
         public DateTime ReportedStartTime { get; set; }
 
@@ -50,16 +52,15 @@ public bool ShowDetails {
         [Parameter(Mandatory = false, HelpMessage = "Retrieved from previous calls, this is the bookmark used for progress when the responses are paged.")]
         public string ContinuationToken { get; set; }
 
-        private UsageAggregationManagementClient theClient;
         public override void ExecuteCmdlet()
         {
-            if (theClient == null)
+            if (_theClient == null)
             {
-                theClient = AzureSession.ClientFactory.CreateClient<UsageAggregationManagementClient>(Profile,
+                _theClient = AzureSession.ClientFactory.CreateClient<UsageAggregationManagementClient>(Profile,
                     Profile.Context.Subscription, AzureEnvironment.Endpoint.ResourceManager);
             }
 
-            UsageAggregationGetResponse aggregations = theClient.UsageAggregates.Get(ReportedStartTime,
+            UsageAggregationGetResponse aggregations = _theClient.UsageAggregates.Get(ReportedStartTime,
                 ReportedEndTime, AggregationGranularity, ShowDetails,
                 ContinuationToken);
 

src/ResourceManager/Commerce/Commands.UsageAggregates/TestTheCmdLet.ps1

@@ -1,6 +1,7 @@
-$subscription = <Add subscription ID>
+# sample script to help the developer verify changes using PowerShell and importing the dll.
+$subscription = "<Add subscription ID>"
 
-Import-Module C:\GitHub\MyGitHub\azure-powershell\src\Package\Debug\ResourceManager\AzureResourceManager\UsageAggregates\Microsoft.Azure.Commands.UsageAggregates.dll
+Import-Module "<path to>"Microsoft.Azure.Commands.UsageAggregates.dll
 
 Add-AzureAccount 
 

src/ResourceManager/Resources/Commands.Resources/AzureResourceManager.psd1

@@ -99,7 +99,7 @@ NestedModules = @(
     '.\ApiManagement\Microsoft.Azure.Commands.ApiManagement.dll',
     '.\StorageManagement\Microsoft.Azure.Commands.Management.Storage.dll',
     '.\OperationalInsights\Microsoft.Azure.Commands.OperationalInsights.dll',
-	'.\UsageAggregates\Microsoft.Azure.Commands.UsageAggregates.dll'
+    '.\UsageAggregates\Microsoft.Azure.Commands.UsageAggregates.dll'
 )
 
 # Functions to export from this module