Responding to review comments

Author: markcowl

src/Common/Commands.Common.Storage/Adapters/ARM.Storage.3/ARMStorageService.cs

@@ -65,11 +65,20 @@ public List<string> AuthenticationKeys
             get { return _authenticationKeys; }
         }
 
-        public static string ParseResourceGroupFromId(string idFromServer)
+        /// <summary>
+        /// Get the resource group name from a storage account resource Id
+        /// </summary>
+        /// <param name="resourceId">The resource Id for the storage account</param>
+        /// <returns>The resource group containing the storage account</returns>
+        public static string ParseResourceGroupFromId(string resourceId)
         {
-            if (!string.IsNullOrEmpty(idFromServer))
+            if (!string.IsNullOrEmpty(resourceId))
             {
-                string[] tokens = idFromServer.Split(new[] { '/' }, StringSplitOptions.RemoveEmptyEntries);
+                string[] tokens = resourceId.Split(new[] { '/' }, StringSplitOptions.RemoveEmptyEntries);
+                if (tokens == null || tokens.Length < 4)
+                {
+                    throw new ArgumentOutOfRangeException("resourceId");
+                }
 
                 return tokens[3];
             }

src/Common/Commands.Common.Storage/IStorageService.cs

@@ -20,7 +20,7 @@ namespace Microsoft.WindowsAzure.Commands.Common.Storage
     public interface IStorageService
     {
         /// <summary>
-        /// The eblob service endpoint
+        /// The blob service endpoint
         /// </summary>
         Uri BlobEndpoint { get; }