Apply CR

andrerod · Oct 17, 2013 · 43d3d1b · 43d3d1b
1 parent c74e434
commit 43d3d1b
Show file tree

Hide file tree

Showing 6 changed files with 17 additions and 17 deletions.
diff --git a/WindowsAzurePowershell/src/Commands.Test/Common/ProfileStorageTests.cs b/WindowsAzurePowershell/src/Commands.Test/Common/ProfileStorageTests.cs
@@ -142,8 +142,8 @@ public void SavingWritesProfileDataToStore()
                 Name = sourceEnv.Name,
                 PublishSettingsFileUrl = sourceEnv.PublishSettingsFileUrl,
                 ServiceEndpoint = sourceEnv.ServiceEndpoint,
-                AdTenantUrl = sourceEnv.AdTenantUrl,
+                ActiveDirectoryEndpoint = sourceEnv.AdTenantUrl,
-                CommonTenantId = sourceEnv.CommonTenantId,
+                ActiveDirectoryCommonTenantId = sourceEnv.CommonTenantId,
                 StorageEndpointSuffix = sourceEnv.StorageEndpointSuffix
             });
 

diff --git a/WindowsAzurePowershell/src/Commands.Utilities/Common/Authentication/AdalConfiguration.cs b/WindowsAzurePowershell/src/Commands.Utilities/Common/Authentication/AdalConfiguration.cs
@@ -76,8 +76,8 @@ public AdalConfiguration()
         public AdalConfiguration(WindowsAzureEnvironment environment)
             : this()
         {
-            AdEndpoint = environment.AdTenantUrl;
+            AdEndpoint = environment.ActiveDirectoryEndpoint;
-            AdDomain = environment.CommonTenantId;
+            AdDomain = environment.ActiveDirectoryCommonTenantId;
         }
 
         public AdalConfiguration(WindowsAzureSubscription subscription)

diff --git a/WindowsAzurePowershell/src/Commands.Utilities/Common/ProfileData.cs b/WindowsAzurePowershell/src/Commands.Utilities/Common/ProfileData.cs
@@ -66,8 +66,8 @@ public AzureEnvironmentData(WindowsAzureEnvironment inMemoryEnvironment)
             ServiceEndpoint = inMemoryEnvironment.ServiceEndpoint;
             ManagementPortalUrl = inMemoryEnvironment.ManagementPortalUrl;
             StorageEndpointSuffix = inMemoryEnvironment.StorageEndpointSuffix;
-            AdTenantUrl = inMemoryEnvironment.AdTenantUrl;
+            AdTenantUrl = inMemoryEnvironment.ActiveDirectoryEndpoint;
-            CommonTenantId = inMemoryEnvironment.CommonTenantId;
+            CommonTenantId = inMemoryEnvironment.ActiveDirectoryCommonTenantId;
         }
 
         /// <summary>
@@ -82,8 +82,8 @@ public WindowsAzureEnvironment ToAzureEnvironment()
                 ServiceEndpoint = this.ServiceEndpoint,
                 ManagementPortalUrl = this.ManagementPortalUrl,
                 StorageEndpointSuffix = this.StorageEndpointSuffix,
-                AdTenantUrl = this.AdTenantUrl,
+                ActiveDirectoryEndpoint = this.AdTenantUrl,
-                CommonTenantId = this.CommonTenantId
+                ActiveDirectoryCommonTenantId = this.CommonTenantId
             };
         }
 

diff --git a/WindowsAzurePowershell/src/Commands.Utilities/Common/WindowsAzureEnvironment.cs b/WindowsAzurePowershell/src/Commands.Utilities/Common/WindowsAzureEnvironment.cs
@@ -47,14 +47,14 @@ public class WindowsAzureEnvironment
         /// Url for the Active Directory tenant for this environment
         /// </summary>
         /// <remarks>If null, this environment does not support AD authentication</remarks>
-        public string AdTenantUrl { get; set; }
+        public string ActiveDirectoryEndpoint { get; set; }
 
         /// <summary>
         /// Name for the common tenant used as the first step
         /// in the AD authentication process for this environment.
         /// </summary>
         /// <remarks>If null, this environment does not support AD authentication</remarks>
-        public string CommonTenantId { get; set; }
+        public string ActiveDirectoryCommonTenantId { get; set; }
 
         private string storageEndpointSuffix;
 
@@ -169,7 +169,7 @@ private string AddRealm(string baseUrl, string realm)
 
         public IEnumerable<WindowsAzureSubscription> AddAccount(ITokenProvider tokenProvider)
         {
-            if (AdTenantUrl == null)
+            if (ActiveDirectoryEndpoint == null)
             {
                 throw new Exception(string.Format(Resources.EnvironmentDoesNotSupportActiveDirectory, Name));
             }
@@ -185,7 +185,7 @@ public IEnumerable<WindowsAzureSubscription> AddAccount(ITokenProvider tokenProv
                 {
                     var azureSubscription = new WindowsAzureSubscription
                     {
-                        ActiveDirectoryEndpoint = AdTenantUrl,
+                        ActiveDirectoryEndpoint = ActiveDirectoryEndpoint,
                         ActiveDirectoryTenantId = subscription.ActiveDirectoryTenantId,
                         ActiveDirectoryUserId = mainToken.UserId,
                         SubscriptionId = subscription.SubscriptionId,
@@ -227,8 +227,8 @@ public IEnumerable<WindowsAzureSubscription> AddAccount(ITokenProvider tokenProv
                     ServiceEndpoint = WindowsAzureEnvironmentConstants.AzureServiceEndpoint,
                     ManagementPortalUrl = WindowsAzureEnvironmentConstants.AzureManagementPortalUrl,
                     // TODO: Get real endpoint for prod
-                    AdTenantUrl = "https://login.windows.net/",
+                    ActiveDirectoryEndpoint = "https://login.windows.net/",
-                    CommonTenantId = "common",
+                    ActiveDirectoryCommonTenantId = "common",
                     StorageEndpointSuffix = WindowsAzureEnvironmentConstants.AzureStorageEndpointSuffix
                 }
             },

diff --git a/WindowsAzurePowershell/src/Commands/Environment/AddAzureEnvironment.cs b/WindowsAzurePowershell/src/Commands/Environment/AddAzureEnvironment.cs
@@ -53,8 +53,8 @@ public override void ExecuteCmdlet()
                 ServiceEndpoint = ServiceEndpoint,
                 ManagementPortalUrl = ManagementPortalUrl,
                 StorageEndpointSuffix = StorageEndpoint,
-                AdTenantUrl = ActiveDirectoryEndpoint,
+                ActiveDirectoryEndpoint = ActiveDirectoryEndpoint,
-                CommonTenantId = "Common"
+                ActiveDirectoryCommonTenantId = "Common"
             };
 
             WindowsAzureProfile.Instance.AddEnvironment(newEnvironment);

diff --git a/WindowsAzurePowershell/src/Commands/Environment/SetAzureEnvironment.cs b/WindowsAzurePowershell/src/Commands/Environment/SetAzureEnvironment.cs
@@ -53,7 +53,7 @@ public override void ExecuteCmdlet()
                 env.ServiceEndpoint = Value(ServiceEndpoint, env.ServiceEndpoint);
                 env.ManagementPortalUrl = Value(ManagementPortalUrl, env.ManagementPortalUrl);
                 env.StorageEndpointSuffix = Value(StorageEndpoint, env.StorageEndpointSuffix);
-                env.AdTenantUrl = Value(AdEndpointUrl, env.AdTenantUrl);
+                env.ActiveDirectoryEndpoint = Value(AdEndpointUrl, env.ActiveDirectoryEndpoint);
 
                 WindowsAzureProfile.Instance.UpdateEnvironment(env);