Incorporating the code review feedback

geekri · geekri · commit 090c8dc826c3 · 2015-03-23T20:01:21.000+05:30
diff --git a/src/ServiceManagement/RecoveryServices/Commands.RecoveryServices/Service/CreateAzureSiteRecoveryProtectionProfileObject.cs b/src/ServiceManagement/RecoveryServices/Commands.RecoveryServices/Service/CreateAzureSiteRecoveryProtectionProfileObject.cs
@@ -266,7 +266,6 @@ private void ProceedToCreateProtectionProfileObject()
                 {
                     RecoveryAzureSubscription = this.RecoveryAzureSubscription,
                     RecoveryAzureStorageAccountName = this.RecoveryAzureStorageAccount,
-                    //// Currently Data Encryption is not supported.
                     EncryptStoredData = this.EncryptStoredData,
                     ReplicationFrequencyInSeconds = replicationFrequencyInSeconds,
                     RecoveryPoints = this.RecoveryPoints,
diff --git a/src/ServiceManagement/RecoveryServices/Commands.RecoveryServices/lib/CertUtils.cs b/src/ServiceManagement/RecoveryServices/Commands.RecoveryServices/lib/CertUtils.cs
@@ -178,7 +178,7 @@ public static string GetCertInBase64EncodedForm(string certFileName)
             catch (Exception e)
             {
                 certInBase64EncodedForm = null;
-                throw new ArgumentException(e.Message, certFileName);
+                throw new ArgumentException(e.Message, certFileName, e);
             }
             finally
             {
diff --git a/src/ServiceManagement/RecoveryServices/Commands.RecoveryServices/lib/SiteRecovery.Tests.dll b/src/ServiceManagement/RecoveryServices/Commands.RecoveryServices/lib/SiteRecovery.Tests.dll

Original file line number	Diff line number	Diff line change
`@@ -266,7 +266,6 @@ private void ProceedToCreateProtectionProfileObject()`
`266`	`266`	`{`
`267`	`267`	`RecoveryAzureSubscription = this.RecoveryAzureSubscription,`
`268`	`268`	`RecoveryAzureStorageAccountName = this.RecoveryAzureStorageAccount,`
`269`		`- //// Currently Data Encryption is not supported.`
`270`	`269`	`EncryptStoredData = this.EncryptStoredData,`
`271`	`270`	`ReplicationFrequencyInSeconds = replicationFrequencyInSeconds,`
`272`	`271`	`RecoveryPoints = this.RecoveryPoints,`
Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ public static string GetCertInBase64EncodedForm(string certFileName)`
`178`	`178`	`catch (Exception e)`
`179`	`179`	`{`
`180`	`180`	`certInBase64EncodedForm = null;`
`181`		`- throw new ArgumentException(e.Message, certFileName);`
	`181`	`+ throw new ArgumentException(e.Message, certFileName, e);`
`182`	`182`	`}`
`183`	`183`	`finally`
`184`	`184`	`{`