Merge pull request #5 from gaurangisaxena/gasaxena/powershell

mkadiri3 · web-flow · commit a21f588ce866 · 2019-07-25T12:28:39.000-07:00
use resource strings
diff --git a/src/DataShare/DataShare/Account/GetAzDataShareAccount.cs b/src/DataShare/DataShare/Account/GetAzDataShareAccount.cs
@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.Account
     using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -134,11 +135,7 @@ public override void ExecuteCmdlet()
                 }
                 catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
                 {
-                    throw new PSArgumentException(
-                        string.Format(
-                            "Account not found",
-                            this.Name,
-                            this.ResourceGroupName));
+                    throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.Name));
                 }
             }
         }
diff --git a/src/DataShare/DataShare/ConsumerInvitation/GetAzDataShareReceivedInvitation.cs b/src/DataShare/DataShare/ConsumerInvitation/GetAzDataShareReceivedInvitation.cs
@@ -25,6 +25,7 @@ namespace Microsoft.Azure.Commands.DataShare.ConsumerInvitation
     using Microsoft.Azure.Management.DataShare.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -70,8 +71,7 @@ public override void ExecuteCmdlet()
                 }
                 catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
                 {
-                    throw new PSArgumentException(
-                        $"ConsumerInvitation {this.InvitationId} not found");
+                    throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.InvitationId));
                 }
             }
             else
diff --git a/src/DataShare/DataShare/DataSet/GetAzDataShareDataSet.cs b/src/DataShare/DataShare/DataSet/GetAzDataShareDataSet.cs
@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.DataSet
     using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -117,8 +118,7 @@ public override  void ExecuteCmdlet()
                 }
                 catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
                 {
-                    throw new PSArgumentException(
-                        $"DataSet {this.Name} not found");
+                    throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.Name));
                 }
             }
             else
diff --git a/src/DataShare/DataShare/DataSetMapping/GetAzDataShareDataSetMapping.cs b/src/DataShare/DataShare/DataSetMapping/GetAzDataShareDataSetMapping.cs
@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.DataSetMapping
     using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -118,8 +119,7 @@ public override void ExecuteCmdlet()
                 }
                 catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
                 {
-                    throw new PSArgumentException(
-                        $"DataSetMapping {this.Name} not found");
+                    throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.Name));
                 }
             }
             else
diff --git a/src/DataShare/DataShare/Invitation/GetAzDataShareInvitation.cs b/src/DataShare/DataShare/Invitation/GetAzDataShareInvitation.cs
@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.Invitation
     using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -119,8 +120,7 @@ public override void ExecuteCmdlet()
                 }
                 catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
                 {
-                    throw new PSArgumentException(
-                        $"Invitation {this.Name} not found");
+                    throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.Name));
                 }
             }
             else
diff --git a/src/DataShare/DataShare/Properties/Resources.Designer.cs b/src/DataShare/DataShare/Properties/Resources.Designer.cs
diff --git a/src/DataShare/DataShare/Properties/Resources.resx b/src/DataShare/DataShare/Properties/Resources.resx
@@ -129,12 +129,18 @@
   <data name="ResourceCreateMessage" xml:space="preserve">
     <value>Creating '{0}'</value>
   </data>
+  <data name="ResourceNotFoundMessage" xml:space="preserve">
+    <value>Resource '{0}' does not exist</value>
+  </data>
   <data name="ResourceRemovalConfirmation" xml:space="preserve">
     <value>Are you sure you want to remove resource '{0}'? </value>
   </data>
   <data name="ResourceRemovedMessage" xml:space="preserve">
     <value>Removing resource '{0}'</value>
   </data>
+  <data name="ResourceUpdataConfirmation" xml:space="preserve">
+    <value>Are you sure you want to update this resource?</value>
+  </data>
   <data name="RevokeShareSubscriptionAccess" xml:space="preserve">
     <value>Revoking access of share subscription</value>
   </data>
diff --git a/src/DataShare/DataShare/Share/GetAzDataShare.cs b/src/DataShare/DataShare/Share/GetAzDataShare.cs
@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.Share
     using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -106,8 +107,7 @@ public override void ExecuteCmdlet()
                     }
                     catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
                     {
-                        throw new PSArgumentException(
-                            $"Share {this.Name} not found");
+                        throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.Name));
                     }
                 }
                 else
diff --git a/src/DataShare/DataShare/Share/GetAzDataShareProviderShareSubscription.cs b/src/DataShare/DataShare/Share/GetAzDataShareProviderShareSubscription.cs
@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.Share
     using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -122,8 +123,7 @@ public override void ExecuteCmdlet()
                 }
                 catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
                 {
-                    throw new PSArgumentException(
-                        $"ProviderShareSubscription {this.ShareSubscriptionId} not found");
+                    throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.ShareSubscriptionId));
                 }
             }
             else
diff --git a/src/DataShare/DataShare/Share/SetAzDataShare.cs b/src/DataShare/DataShare/Share/SetAzDataShare.cs
@@ -143,15 +143,13 @@ public override void ExecuteCmdlet()
             }
             catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
             {
-                throw new PSArgumentException(
-                    string.Format(
-                        $"Share {this.Name} not found"));
+                throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.Name));
             }
 
             if (this.Description != null || this.TermsOfUse != null)
             {
                 this.ConfirmAction(
-                    "Update resource",
+                    Resources.ResourceUpdataConfirmation,
                     this.Name,
                     () => this.UpdateShare(existingShare));
 
diff --git a/src/DataShare/DataShare/ShareSubscription/GetAzDataShareSubscription.cs b/src/DataShare/DataShare/ShareSubscription/GetAzDataShareSubscription.cs
@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.ShareSubscription
     using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -108,8 +109,7 @@ public override void ExecuteCmdlet()
                     }
                     catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
                     {
-                        throw new PSArgumentException(
-                            $"ShareSubscription {this.Name} not found");
+                        throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.Name));
                     }
                 }
                 else
diff --git a/src/DataShare/DataShare/ShareSynchronization/GetAzDataShareSynchronizationDetail.cs b/src/DataShare/DataShare/ShareSynchronization/GetAzDataShareSynchronizationDetail.cs
@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.Synchronization
     using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -141,8 +142,7 @@ public override void ExecuteCmdlet()
             }
             catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
             {
-                throw new PSArgumentException(
-                    $"ShareSynchronizationDetails {this.ShareName} not found");
+                throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.ShareName));
             }
         }
     }
diff --git a/src/DataShare/DataShare/SourceDataSet/GetAzDataShareSourceDataSet.cs b/src/DataShare/DataShare/SourceDataSet/GetAzDataShareSourceDataSet.cs
@@ -19,6 +19,7 @@ namespace Microsoft.Azure.Commands.DataShare.SourceDataSet
     using System.Linq;
     using System.Net;
     using System.Management.Automation;
+    using Hyak.Common.Properties;
     using Microsoft.Azure.Commands.DataShare.Common;
     using Microsoft.Azure.Commands.DataShare.Helpers;
     using Microsoft.Azure.Commands.ResourceManager.Common.ArgumentCompleters;
@@ -80,6 +81,7 @@ public class GetAzDataShareSourceDataSet : AzureDataShareCmdletBase
             ValueFromPipelineByPropertyName = true,
             HelpMessage = "Azure data share subscription resource id",
             ParameterSetName = ParameterSetNames.ResourceIdParameterSet)]
+        [ResourceIdCompleter(ResourceTypes.ShareSubscription)]
         [ValidateNotNullOrEmpty]
         public string ShareSubscriptionResourceId { get; set; }
 
@@ -98,32 +100,23 @@ public override void ExecuteCmdlet()
             string nextPageLink = null;
             var consumerSourceDataSetList = new List<ConsumerSourceDataSet>();
 
-            try
+            do
             {
-                do
-                {
-                    IPage<ConsumerSourceDataSet> sourceDataSets = string.IsNullOrEmpty(nextPageLink)
-                        ? this.DataShareManagementClient.ConsumerSourceDataSets.ListByShareSubscription(
-                            this.ResourceGroupName,
-                            this.AccountName,
-                            this.ShareSubscriptionName)
-                        : this.DataShareManagementClient.ConsumerSourceDataSets.ListByShareSubscriptionNext(
-                            nextPageLink);
+                IPage<ConsumerSourceDataSet> sourceDataSets = string.IsNullOrEmpty(nextPageLink)
+                    ? this.DataShareManagementClient.ConsumerSourceDataSets.ListByShareSubscription(
+                        this.ResourceGroupName,
+                        this.AccountName,
+                        this.ShareSubscriptionName)
+                    : this.DataShareManagementClient.ConsumerSourceDataSets.ListByShareSubscriptionNext(
+                        nextPageLink);
 
-                    consumerSourceDataSetList.AddRange(sourceDataSets.AsEnumerable());
-                    nextPageLink = sourceDataSets.NextPageLink;
-                } while (nextPageLink != null);
+                consumerSourceDataSetList.AddRange(sourceDataSets.AsEnumerable());
+                nextPageLink = sourceDataSets.NextPageLink;
+            } while (nextPageLink != null);
 
-                IEnumerable<PSDataShareSourceDataSet> dataSetsInShareSubscription =
-                    consumerSourceDataSetList.Select(dataSet => dataSet.ToPsObject());
-                this.WriteObject(dataSetsInShareSubscription, true);
-
-            }
-            catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
-            {
-                throw new PSArgumentException(
-                    $"Source DataSets not found for {this.ShareSubscriptionName}");
-            }
+            IEnumerable<PSDataShareSourceDataSet> dataSetsInShareSubscription =
+                consumerSourceDataSetList.Select(dataSet => dataSet.ToPsObject());
+            this.WriteObject(dataSetsInShareSubscription, true);
         }
     }
 }
diff --git a/src/DataShare/DataShare/Synchronization/GetAzDataShareSubscriptionSynchronization.cs b/src/DataShare/DataShare/Synchronization/GetAzDataShareSubscriptionSynchronization.cs
@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.Synchronization
     using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
+    using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;
     using Microsoft.Rest.Azure;
 
     /// <summary>
@@ -123,8 +124,7 @@ public override void ExecuteCmdlet()
             }
             catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
             {
-                throw new PSArgumentException(
-                    $"Synchronization not found for {this.ShareSubscriptionName}");
+                throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.ShareSubscriptionName));
             }
         }
     }
diff --git a/src/DataShare/DataShare/Synchronization/GetAzDataShareSubscriptionSynchronizationDetail.cs b/src/DataShare/DataShare/Synchronization/GetAzDataShareSubscriptionSynchronizationDetail.cs
@@ -19,6 +19,7 @@ namespace Microsoft.Azure.Commands.DataShare.Synchronization
     using System.Linq;
     using System.Management.Automation;
     using System.Net;
+    using Hyak.Common.Properties;
     using Microsoft.Azure.Commands.DataShare.Common;
     using Microsoft.Azure.Commands.DataShare.Helpers;
     using Microsoft.Azure.Commands.ResourceManager.Common.ArgumentCompleters;
@@ -28,6 +29,7 @@ namespace Microsoft.Azure.Commands.DataShare.Synchronization
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;
     using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;
     using Microsoft.Rest.Azure;
+    using Resources = Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties.Resources;
 
     /// <summary>
     /// Defines Get-AzDataShareSubscriptionSynchronizationDetail cmdlet.
@@ -143,8 +145,7 @@ public override void ExecuteCmdlet()
             }
             catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))
             {
-                throw new PSArgumentException(
-                    $"ShareSubscriptionSynchronizationDetails {this.ShareSubscriptionName} not found");
+                throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.ShareSubscriptionName));
             }
         }
     }
diff --git a/src/DataShare/DataShare/Synchronization/StartAzDataShareSynchronization.cs b/src/DataShare/DataShare/Synchronization/StartAzDataShareSynchronization.cs
@@ -124,20 +124,16 @@ public override void ExecuteCmdlet()
 
         private void StartSynchronization()
         {
-            var startFunc = (Func<string, string, string, Synchronize, ShareSubscriptionSynchronization>)this.DataShareManagementClient
-                    .ShareSubscriptions.SynchronizeMethod;
-            try
-            {
-                var synchronization = startFunc(
-                    this.ResourceGroupName,
-                    this.AccountName,
-                    this.ShareSubscriptionName,
-                    new Synchronize(this.SynchronizationMode));
-                this.WriteObject(synchronization.ToPsObject());
-            } catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.Conflict))
-            {
-                throw new PSArgumentException($"Synchronization already in progress.");
-            }
+            var startFunc = (Func<string, string, string, Synchronize, ShareSubscriptionSynchronization>)this
+                .DataShareManagementClient
+                .ShareSubscriptions.SynchronizeMethod;
+
+            var synchronization = startFunc(
+                this.ResourceGroupName,
+                this.AccountName,
+                this.ShareSubscriptionName,
+                new Synchronize(this.SynchronizationMode));
+            this.WriteObject(synchronization.ToPsObject());
         }
 
         private void SetParametersIfNeeded()
diff --git a/src/DataShare/DataShare/Synchronization/StopAzDataShareSubscriptionSynchronization.cs b/src/DataShare/DataShare/Synchronization/StopAzDataShareSubscriptionSynchronization.cs
diff --git a/src/DataShare/DataShare/SynchronizationSettings/GetAzDataShareSynchronizationSetting.cs b/src/DataShare/DataShare/SynchronizationSettings/GetAzDataShareSynchronizationSetting.cs
diff --git a/src/DataShare/DataShare/Trigger/GetAzDataShareTrigger.cs b/src/DataShare/DataShare/Trigger/GetAzDataShareTrigger.cs

Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,7 @@ namespace Microsoft.Azure.Commands.DataShare.Account`
`27`	`27`	`using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;`
`28`	`28`	`using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;`
`29`	`29`	`using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;`
	`30`	`+ using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;`
`30`	`31`	`using Microsoft.Rest.Azure;`
`31`	`32`
`32`	`33`	`/// <summary>`
`@@ -134,11 +135,7 @@ public override void ExecuteCmdlet()`
`134`	`135`	`}`
`135`	`136`	`catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))`
`136`	`137`	`{`
`137`		`- throw new PSArgumentException(`
`138`		`- string.Format(`
`139`		`- "Account not found",`
`140`		`- this.Name,`
`141`		`- this.ResourceGroupName));`
	`138`	`+ throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.Name));`
`142`	`139`	`}`
`143`	`140`	`}`
`144`	`141`	`}`
Original file line number	Diff line number	Diff line change
`@@ -25,6 +25,7 @@ namespace Microsoft.Azure.Commands.DataShare.ConsumerInvitation`
`25`	`25`	`using Microsoft.Azure.Management.DataShare.Models;`
`26`	`26`	`using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Extensions;`
`27`	`27`	`using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Models;`
	`28`	`+ using Microsoft.Azure.PowerShell.Cmdlets.DataShare.Properties;`
`28`	`29`	`using Microsoft.Rest.Azure;`
`29`	`30`
`30`	`31`	`/// <summary>`
`@@ -70,8 +71,7 @@ public override void ExecuteCmdlet()`
`70`	`71`	`}`
`71`	`72`	`catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))`
`72`	`73`	`{`
`73`		`- throw new PSArgumentException(`
`74`		`- $"ConsumerInvitation {this.InvitationId} not found");`
	`74`	`+ throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.InvitationId));`
`75`	`75`	`}`
`76`	`76`	`}`
`77`	`77`	`else`
Original file line number	Diff line number	Diff line change
`@@ -143,15 +143,13 @@ public override void ExecuteCmdlet()`
`143`	`143`	`}`
`144`	`144`	`catch (DataShareErrorException ex) when (ex.Response.StatusCode.Equals(HttpStatusCode.NotFound))`
`145`	`145`	`{`
`146`		`- throw new PSArgumentException(`
`147`		`- string.Format(`
`148`		`- $"Share {this.Name} not found"));`
	`146`	`+ throw new PSArgumentException(string.Format(Resources.ResourceNotFoundMessage, this.Name));`
`149`	`147`	`}`
`150`	`148`
`151`	`149`	`if (this.Description != null \|\| this.TermsOfUse != null)`
`152`	`150`	`{`
`153`	`151`	`this.ConfirmAction(`
`154`		`- "Update resource",`
	`152`	`+ Resources.ResourceUpdataConfirmation,`
`155`	`153`	`this.Name,`
`156`	`154`	`() => this.UpdateShare(existingShare));`
`157`	`155`