Merge pull request #1 from gaurangisaxena/makadiri/comments

mkadiri3 · web-flow · commit bdc340312db6 · 2019-07-16T12:20:15.000-07:00
removed confirm action and resolved comments
diff --git a/src/DataShare/DataShare/Account/NewAzDataShareAccount.cs b/src/DataShare/DataShare/Account/NewAzDataShareAccount.cs
@@ -50,7 +50,6 @@ public class NewAzDataShareAccount : AzureDataShareCmdletBase
         /// </summary>
         [Parameter(Mandatory = true, HelpMessage = "Azure data share account name.")]
         [ValidateNotNullOrEmpty]
-        [ResourceNameCompleter(ResourceTypes.Account, "ResourceGroupName")]
         public string Name { get; set; }
 
         /// <summary>
@@ -74,22 +73,15 @@ public class NewAzDataShareAccount : AzureDataShareCmdletBase
 
         public override void ExecuteCmdlet()
         {
-            this.ConfirmAction(
-                string.Format(Resources.ResourceCreateConfirmation, this.Name),
-                this.Name,
-                this.NewAccount);
-        }
-
-        private void NewAccount()
-        {
-            if (this.ShouldProcess(this.Name, "Create"))
+            if (this.ShouldProcess(this.Name, "Creating data share account"))
             {
                 Account dataShareAccount = this.DataShareManagementClient.Accounts.Create(
                     this.ResourceGroupName,
                     this.Name,
                     new Account()
                     {
-                        Location = this.Location, Tags = this.Tag?.ToDictionaryTags(),
+                        Location = this.Location,
+                        Tags = this.Tag?.ToDictionaryTags(),
                         Identity = new Identity
                         {
                             Type = "SystemAssigned"
diff --git a/src/DataShare/DataShare/DataSet/NewAzDataShareDataSet.cs b/src/DataShare/DataShare/DataSet/NewAzDataShareDataSet.cs
@@ -119,7 +119,6 @@ public class NewAzDataShareDataSet : AzureDataShareCmdletBase
             HelpMessage = "Azure data set name",
             ParameterSetName = ParameterSetNames.AdlsGen1DataSetParameterSet)]
         [ValidateNotNullOrEmpty]
-        [ResourceNameCompleter(ResourceTypes.DataSet, "ResourceGroupName", "AccountName", "ShareName")]
         public string Name { get; set; }
 
         /// <summary>
@@ -141,6 +140,7 @@ public class NewAzDataShareDataSet : AzureDataShareCmdletBase
             HelpMessage = "Azure storage account resourceId",
             ParameterSetName = ParameterSetNames.AdlsGen1DataSetParameterSet)]
         [ValidateNotNullOrEmpty]
+        [ResourceNameCompleter(ResourceTypes.StorageAccount, "ResourceGroupName")]
         public string StorageAccountResourceId { get; set; }
 
         /// <summary>
@@ -229,7 +229,7 @@ public override void ExecuteCmdlet()
                 ParameterSetNames.BlobDataSetParameterSet,
                 StringComparison.OrdinalIgnoreCase))
             {
-                if (this.ShouldProcess(this.Name, "Create"))
+                if (this.ShouldProcess(this.Name, "Creating data set"))
                 {
                     if (this.FilePath != null)
                     {
@@ -297,7 +297,7 @@ public override void ExecuteCmdlet()
                 StringComparison.OrdinalIgnoreCase))
             {
 
-                if (this.ShouldProcess(this.Name, "Create"))
+                if (this.ShouldProcess(this.Name, "Creating data set"))
                 {
                     if (this.FilePath != null)
                     {
@@ -365,7 +365,7 @@ public override void ExecuteCmdlet()
                 StringComparison.OrdinalIgnoreCase))
             {
                 storageAccountName = parsedStorageResourceId.GetAccountName();
-                if (this.ShouldProcess(this.Name, "Create"))
+                if (this.ShouldProcess(this.Name, "Creating data set"))
                 {
                     if (this.FileName != null)
                     {
diff --git a/src/DataShare/DataShare/DataSetMapping/NewAzDataShareDataSetMapping.cs b/src/DataShare/DataShare/DataSetMapping/NewAzDataShareDataSetMapping.cs
@@ -99,7 +99,6 @@ public class NewAzDataShareDataSetMapping : AzureDataShareCmdletBase
             HelpMessage = "Azure data set mapping name",
             ParameterSetName = ParameterSetNames.AdlsGen2DataSetParameterSet)]
         [ValidateNotNullOrEmpty]
-        [ResourceNameCompleter(ResourceTypes.DataSetMapping, "ResourceGroupName", "AccountName", "ShareSubscriptionName")]
         public string Name { get; set; }
 
         /// <summary>
@@ -116,6 +115,7 @@ public class NewAzDataShareDataSetMapping : AzureDataShareCmdletBase
             HelpMessage = "Azure Storage Account ResourceId",
             ParameterSetName = ParameterSetNames.AdlsGen2DataSetParameterSet)]
         [ValidateNotNullOrEmpty]
+        [ResourceNameCompleter(ResourceTypes.StorageAccount, "ResourceGroupName")]
         public string StorageAccountResourceId { get; set; }
 
         /// <summary>
@@ -198,7 +198,7 @@ public override void ExecuteCmdlet()
                 ParameterSetNames.BlobDataSetParameterSet,
                 StringComparison.OrdinalIgnoreCase))
             {
-                if (this.ShouldProcess(this.Name, "Create"))
+                if (this.ShouldProcess(this.Name, "Creating data set mapping"))
                 {
                     if (this.FilePath != null)
                     {
@@ -269,7 +269,7 @@ public override void ExecuteCmdlet()
                 StringComparison.OrdinalIgnoreCase))
             {
 
-                if (this.ShouldProcess(this.Name, "Create"))
+                if (this.ShouldProcess(this.Name, "Creating data set mapping"))
                 {
                     if (this.FilePath != null)
                     {
diff --git a/src/DataShare/DataShare/DataShare.csproj b/src/DataShare/DataShare/DataShare.csproj
@@ -25,8 +25,4 @@
     </EmbeddedResource>
   </ItemGroup>
 
-  <ItemGroup>
-    <PackageReference Update="Microsoft.Rest.ClientRuntime" Version="2.3.20" />
-  </ItemGroup>
-
 </Project>
diff --git a/src/DataShare/DataShare/Extensions/ResourceTypes.cs b/src/DataShare/DataShare/Extensions/ResourceTypes.cs
@@ -10,5 +10,6 @@ public static class ResourceTypes
         public const string ShareSubscription = "Microsoft.DataShare/sharesubscriptions";
         public const string SynchronizationSetting = "Microsoft.DataShare/synchronizationSettings";
         public const string Trigger = "Microsoft.DataShare/triggers";
+        public const string StorageAccount = "Microsoft.Storage/storageAccounts";
     }
 }
diff --git a/src/DataShare/DataShare/Invitation/GetAzDataShareInvitation.cs b/src/DataShare/DataShare/Invitation/GetAzDataShareInvitation.cs
@@ -83,6 +83,7 @@ public class GetAzDataShareInvitation : AzureDataShareCmdletBase
             HelpMessage = "Azure data share invitation name",
             ParameterSetName = ParameterSetNames.FieldsParameterSet)]
         [ValidateNotNullOrEmpty]
+        [ResourceNameCompleter(ResourceTypes.Invitation, "ResourceGroupName", "AccountName","ShareName")]
         public string Name { get; set; }
 
         /// <summary>
diff --git a/src/DataShare/DataShare/Invitation/NewAzDataShareInvitation.cs b/src/DataShare/DataShare/Invitation/NewAzDataShareInvitation.cs
@@ -127,55 +127,39 @@ public override void ExecuteCmdlet()
         {
             if (this.ParameterSetName.Equals(ParameterSetNames.InvitationEmailParameterSet, StringComparison.OrdinalIgnoreCase))
             {
-                this.ConfirmAction(
-                    string.Format(Resources.ResourceCreateConfirmation, this.Name),
-                    this.Name,
-                    this.NewEmailInvitation);
-            }
-
-            if (this.ParameterSetName.Equals(ParameterSetNames.InvitationTenantParameterSet, StringComparison.OrdinalIgnoreCase))
-            {
-                this.ConfirmAction(
-                    string.Format(Resources.ResourceCreateConfirmation, this.Name),
-                    this.Name,
-                    this.NewObjectIdInvitation);
-            }
-        }
+                if (this.ShouldProcess(this.Name, "Creating invitation"))
+                {
+                    Invitation newInvitation = this.DataShareManagementClient.Invitations.Create(
+                        this.ResourceGroupName,
+                        this.AccountName,
+                        this.ShareName,
+                        this.Name,
+                        new Invitation()
+                        {
+                            TargetEmail = this.TargetEmail
+                        });
 
-        private void NewEmailInvitation()
-        {
-            if (this.ShouldProcess(this.Name, "Create"))
-            {
-                Invitation newInvitation = this.DataShareManagementClient.Invitations.Create(
-                    this.ResourceGroupName,
-                    this.AccountName,
-                    this.ShareName,
-                    this.Name,
-                    new Invitation()
-                    {
-                        TargetEmail = this.TargetEmail
-                    });
-
-                this.WriteObject(newInvitation.ToPsObject());
+                    this.WriteObject(newInvitation.ToPsObject());
+                }
             }
-        }
 
-        private void NewObjectIdInvitation()
-        {
-            if (this.ShouldProcess(this.Name, "Create"))
+            if (this.ParameterSetName.Equals(ParameterSetNames.InvitationTenantParameterSet, StringComparison.OrdinalIgnoreCase))
             {
-                Invitation newInvitation = this.DataShareManagementClient.Invitations.Create(
-                    this.ResourceGroupName,
-                    this.AccountName,
-                    this.ShareName,
-                    this.Name,
-                    new Invitation()
-                    {
-                        TargetObjectId = this.TargetObjectId,
-                        TargetActiveDirectoryId = this.TargetTenantId
-                    });
+                if (this.ShouldProcess(this.Name, "Creating invitation"))
+                {
+                    Invitation newInvitation = this.DataShareManagementClient.Invitations.Create(
+                        this.ResourceGroupName,
+                        this.AccountName,
+                        this.ShareName,
+                        this.Name,
+                        new Invitation()
+                        {
+                            TargetObjectId = this.TargetObjectId,
+                            TargetActiveDirectoryId = this.TargetTenantId
+                        });
 
-                this.WriteObject(newInvitation.ToPsObject());
+                    this.WriteObject(newInvitation.ToPsObject());
+                }
             }
         }
     }
diff --git a/src/DataShare/DataShare/Invitation/RemoveAzDataShareInvitation.cs b/src/DataShare/DataShare/Invitation/RemoveAzDataShareInvitation.cs
@@ -80,6 +80,7 @@ public class RemoveAzDataShareInvitation : AzureDataShareCmdletBase
             ValueFromPipelineByPropertyName = true,
             ParameterSetName = ParameterSetNames.FieldsParameterSet,
             HelpMessage = "Azure data share invitation name")]
+        [ResourceNameCompleter(ResourceTypes.Invitation, "ResourceGroupName", "AccountName", "ShareName")]
         public string Name { get; set; }
 
 
diff --git a/src/DataShare/DataShare/ParameterSetNames.cs b/src/DataShare/DataShare/ParameterSetNames.cs
@@ -10,5 +10,6 @@ class ParameterSetNames
         public const string BlobDataSetParameterSet = "ByBlobDataSetParamaterSet";
         public const string AdlsGen2DataSetParameterSet = "ByAdlsGen2DataSetParameterSet";
         public const string AdlsGen1DataSetParameterSet = "ByAdlsGen1DataSetParameterSet";
+        public const string ProviderShareSubscriptionIdParameterSet = "ByProviderShareSubscriptionIdParameterSet";
     }
 }
diff --git a/src/DataShare/DataShare/Properties/AssemblyInfo.cs b/src/DataShare/DataShare/Properties/AssemblyInfo.cs
@@ -25,5 +25,5 @@
 [assembly: ComVisible(false)]
 [assembly: CLSCompliant(false)]
 [assembly: Guid("d90791a2-8102-47fc-8150-de25ae796eb1")]
-[assembly: AssemblyVersion("1.1.0")]
-[assembly: AssemblyFileVersion("1.1.0")]
+[assembly: AssemblyVersion("0.1.0")]
+[assembly: AssemblyFileVersion("0.1.0")]
diff --git a/src/DataShare/DataShare/ProviderShareSubscriptions/GrantAzDataShareSubscriptionAccess.cs b/src/DataShare/DataShare/ProviderShareSubscriptions/GrantAzDataShareSubscriptionAccess.cs
@@ -80,8 +80,7 @@ public class GrantAzDataShareSubscriptionAccess : AzureDataShareCmdletBase
             Mandatory = true,
             ValueFromPipelineByPropertyName = true,
             HelpMessage = "The share subscription id of the provider share subscription",
-            ParameterSetName = ParameterSetNames.ResourceIdParameterSet)]
-        [ResourceIdCompleter(ResourceTypes.ShareSubscription)]
+            ParameterSetName = ParameterSetNames.ProviderShareSubscriptionIdParameterSet)]
         [ValidateNotNullOrEmpty]
         public string ShareSubscriptionId { get; set; }
 
@@ -92,7 +91,7 @@ public class GrantAzDataShareSubscriptionAccess : AzureDataShareCmdletBase
             Mandatory = true,
             ValueFromPipelineByPropertyName = true,
             HelpMessage = "The resource id of the azure data share",
-            ParameterSetName = ParameterSetNames.ResourceIdParameterSet)]
+            ParameterSetName = ParameterSetNames.ProviderShareSubscriptionIdParameterSet)]
         [ResourceIdCompleter(ResourceTypes.Share)]
         [ValidateNotNullOrEmpty]
         public string ResourceId { get; set; }
diff --git a/src/DataShare/DataShare/ProviderShareSubscriptions/RevokeAzDataShareSubscriptionAccess.cs b/src/DataShare/DataShare/ProviderShareSubscriptions/RevokeAzDataShareSubscriptionAccess.cs
@@ -81,7 +81,7 @@ public class RevokeAzDataShareSubscriptionAccess : AzureDataShareCmdletBase
             Mandatory = true,
             ValueFromPipelineByPropertyName = true,
             HelpMessage = "The share subscription id of the provider share subscription",
-            ParameterSetName = ParameterSetNames.ResourceIdParameterSet)]
+            ParameterSetName = ParameterSetNames.ProviderShareSubscriptionIdParameterSet)]
         [ValidateNotNullOrEmpty]
         public string ShareSubscriptionId { get; set; }
 
@@ -92,7 +92,7 @@ public class RevokeAzDataShareSubscriptionAccess : AzureDataShareCmdletBase
             Mandatory = true,
             ValueFromPipelineByPropertyName = true,
             HelpMessage = "The resource id of the azure data share",
-            ParameterSetName = ParameterSetNames.ResourceIdParameterSet)]
+            ParameterSetName = ParameterSetNames.ProviderShareSubscriptionIdParameterSet)]
         [ResourceIdCompleter(ResourceTypes.Share)]
         [ValidateNotNullOrEmpty]
         public string ResourceId { get; set; }
diff --git a/src/DataShare/DataShare/Share/GetAzDataShareProviderShareSubscription.cs b/src/DataShare/DataShare/Share/GetAzDataShareProviderShareSubscription.cs
@@ -86,7 +86,7 @@ public class GetAzDataShareProviderShareSubscription : AzureDataShareCmdletBase
             Mandatory = false,
             ValueFromPipelineByPropertyName = true,
             HelpMessage = "The share subscription id of the provider share subscription",
-            ParameterSetName = ParameterSetNames.ResourceIdParameterSet)]
+            ParameterSetName = ParameterSetNames.ProviderShareSubscriptionIdParameterSet)]
         [ValidateNotNullOrEmpty]
         public string ShareSubscriptionId { get; set; }
 
@@ -97,7 +97,7 @@ public class GetAzDataShareProviderShareSubscription : AzureDataShareCmdletBase
             Mandatory = true,
             ValueFromPipelineByPropertyName = true,
             HelpMessage = "The resource id of the share",
-            ParameterSetName = ParameterSetNames.ResourceIdParameterSet)]
+            ParameterSetName = ParameterSetNames.ProviderShareSubscriptionIdParameterSet)]
         [ResourceIdCompleter(ResourceTypes.Share)]
         [ValidateNotNullOrEmpty]
         public string ResourceId { get; set; }
diff --git a/src/DataShare/DataShare/Share/NewAzDataShare.cs b/src/DataShare/DataShare/Share/NewAzDataShare.cs
@@ -60,7 +60,6 @@ public class NewAzDataShare : AzureDataShareCmdletBase
             ParameterSetName = ParameterSetNames.FieldsParameterSet,
             HelpMessage = "Azure data share name")]
         [ValidateNotNullOrEmpty]
-        [ResourceNameCompleter(ResourceTypes.Share, "ResourceGroupName", "AccountName")]
         public string Name { get; set; }
 
         /// <summary>
@@ -83,15 +82,7 @@ public class NewAzDataShare : AzureDataShareCmdletBase
 
         public override void ExecuteCmdlet()
         {
-            this.ConfirmAction(
-                string.Format(Resources.ResourceCreateConfirmation, this.Name),
-                this.Name,
-                this.NewShare);
-        }
-
-        private void NewShare()
-        {
-            if (this.ShouldProcess(this.Name, "Create"))
+            if (this.ShouldProcess(this.Name, "Creating data share"))
             {
                 Share dataShare = this.DataShareManagementClient.Shares.Create(
                     this.ResourceGroupName,
diff --git a/src/DataShare/DataShare/ShareSubscription/NewAzDataShareSubscription.cs b/src/DataShare/DataShare/ShareSubscription/NewAzDataShareSubscription.cs
@@ -60,7 +60,6 @@ public class NewAzDataShareSubscription : AzureDataShareCmdletBase
             ParameterSetName = ParameterSetNames.FieldsParameterSet, 
             HelpMessage = "Azure data share subscription name")]
         [ValidateNotNullOrEmpty]
-        [ResourceNameCompleter(ResourceTypes.ShareSubscription, "ResourceGroupName", "AccountName")]
         public string Name { get; set; }
 
         /// <summary>
@@ -75,15 +74,7 @@ public class NewAzDataShareSubscription : AzureDataShareCmdletBase
 
         public override void ExecuteCmdlet()
         {
-            this.ConfirmAction(
-                string.Format(Resources.ResourceCreateConfirmation, this.Name),
-                this.Name,
-                this.NewShareSubscription);
-        }
-
-        private void NewShareSubscription()
-        {
-            if (this.ShouldProcess(this.Name, "Create"))
+            if (this.ShouldProcess(this.Name, "Creating share subscription"))
             {
                 ShareSubscription shareSubscription = this.DataShareManagementClient.ShareSubscriptions.Create(
                     this.ResourceGroupName,
diff --git a/src/DataShare/DataShare/SynchronizationSettings/NewAzDataShareSynchronizationSetting.cs b/src/DataShare/DataShare/SynchronizationSettings/NewAzDataShareSynchronizationSetting.cs
@@ -81,7 +81,6 @@ public class NewAzDataShareSynchronizationSetting : AzureDataShareCmdletBase
             HelpMessage = "Synchronization setting name",
             ParameterSetName = ParameterSetNames.FieldsParameterSet)]
         [ValidateNotNullOrEmpty]
-        [ResourceNameCompleter(ResourceTypes.SynchronizationSetting, "ResourceGroupName", "AccountName", "ShareName")]
         public string Name { get; set; }
 
         /// <summary>
@@ -109,7 +108,7 @@ public class NewAzDataShareSynchronizationSetting : AzureDataShareCmdletBase
 
         public override void ExecuteCmdlet()
         {
-            if (this.ShouldProcess(this.Name, "Create"))
+            if (this.ShouldProcess(this.Name, "Creating share synchronization"))
             {
                 var setting = new ScheduledSynchronizationSetting(
                     recurrenceInterval: this.RecurrenceInterval,
diff --git a/src/DataShare/DataShare/Trigger/NewAzDataShareTrigger.cs b/src/DataShare/DataShare/Trigger/NewAzDataShareTrigger.cs
@@ -76,7 +76,6 @@ public class NewAzDataShareTrigger : AzureDataShareCmdletBase
             HelpMessage = "Azure data share trigger name",
             ParameterSetName = ParameterSetNames.FieldsParameterSet)]
         [ValidateNotNullOrEmpty]
-        [ResourceNameCompleter(ResourceTypes.Trigger, "ResourceGroupName", "AccountName", "ShareSubscriptionName")]
         public string Name { get; set; }
 
 

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,6 @@ public class NewAzDataShareDataSet : AzureDataShareCmdletBase`
`119`	`119`	`HelpMessage = "Azure data set name",`
`120`	`120`	`ParameterSetName = ParameterSetNames.AdlsGen1DataSetParameterSet)]`
`121`	`121`	`[ValidateNotNullOrEmpty]`
`122`		`- [ResourceNameCompleter(ResourceTypes.DataSet, "ResourceGroupName", "AccountName", "ShareName")]`
`123`	`122`	`public string Name { get; set; }`
`124`	`123`
`125`	`124`	`/// <summary>`
`@@ -141,6 +140,7 @@ public class NewAzDataShareDataSet : AzureDataShareCmdletBase`
`141`	`140`	`HelpMessage = "Azure storage account resourceId",`
`142`	`141`	`ParameterSetName = ParameterSetNames.AdlsGen1DataSetParameterSet)]`
`143`	`142`	`[ValidateNotNullOrEmpty]`
	`143`	`+ [ResourceNameCompleter(ResourceTypes.StorageAccount, "ResourceGroupName")]`
`144`	`144`	`public string StorageAccountResourceId { get; set; }`
`145`	`145`
`146`	`146`	`/// <summary>`
`@@ -229,7 +229,7 @@ public override void ExecuteCmdlet()`
`229`	`229`	`ParameterSetNames.BlobDataSetParameterSet,`
`230`	`230`	`StringComparison.OrdinalIgnoreCase))`
`231`	`231`	`{`
`232`		`- if (this.ShouldProcess(this.Name, "Create"))`
	`232`	`+ if (this.ShouldProcess(this.Name, "Creating data set"))`
`233`	`233`	`{`
`234`	`234`	`if (this.FilePath != null)`
`235`	`235`	`{`
`@@ -297,7 +297,7 @@ public override void ExecuteCmdlet()`
`297`	`297`	`StringComparison.OrdinalIgnoreCase))`
`298`	`298`	`{`
`299`	`299`
`300`		`- if (this.ShouldProcess(this.Name, "Create"))`
	`300`	`+ if (this.ShouldProcess(this.Name, "Creating data set"))`
`301`	`301`	`{`
`302`	`302`	`if (this.FilePath != null)`
`303`	`303`	`{`
`@@ -365,7 +365,7 @@ public override void ExecuteCmdlet()`
`365`	`365`	`StringComparison.OrdinalIgnoreCase))`
`366`	`366`	`{`
`367`	`367`	`storageAccountName = parsedStorageResourceId.GetAccountName();`
`368`		`- if (this.ShouldProcess(this.Name, "Create"))`
	`368`	`+ if (this.ShouldProcess(this.Name, "Creating data set"))`
`369`	`369`	`{`
`370`	`370`	`if (this.FileName != null)`
`371`	`371`	`{`
Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,6 @@ public class NewAzDataShareDataSetMapping : AzureDataShareCmdletBase`
`99`	`99`	`HelpMessage = "Azure data set mapping name",`
`100`	`100`	`ParameterSetName = ParameterSetNames.AdlsGen2DataSetParameterSet)]`
`101`	`101`	`[ValidateNotNullOrEmpty]`
`102`		`- [ResourceNameCompleter(ResourceTypes.DataSetMapping, "ResourceGroupName", "AccountName", "ShareSubscriptionName")]`
`103`	`102`	`public string Name { get; set; }`
`104`	`103`
`105`	`104`	`/// <summary>`
`@@ -116,6 +115,7 @@ public class NewAzDataShareDataSetMapping : AzureDataShareCmdletBase`
`116`	`115`	`HelpMessage = "Azure Storage Account ResourceId",`
`117`	`116`	`ParameterSetName = ParameterSetNames.AdlsGen2DataSetParameterSet)]`
`118`	`117`	`[ValidateNotNullOrEmpty]`
	`118`	`+ [ResourceNameCompleter(ResourceTypes.StorageAccount, "ResourceGroupName")]`
`119`	`119`	`public string StorageAccountResourceId { get; set; }`
`120`	`120`
`121`	`121`	`/// <summary>`
`@@ -198,7 +198,7 @@ public override void ExecuteCmdlet()`
`198`	`198`	`ParameterSetNames.BlobDataSetParameterSet,`
`199`	`199`	`StringComparison.OrdinalIgnoreCase))`
`200`	`200`	`{`
`201`		`- if (this.ShouldProcess(this.Name, "Create"))`
	`201`	`+ if (this.ShouldProcess(this.Name, "Creating data set mapping"))`
`202`	`202`	`{`
`203`	`203`	`if (this.FilePath != null)`
`204`	`204`	`{`
`@@ -269,7 +269,7 @@ public override void ExecuteCmdlet()`
`269`	`269`	`StringComparison.OrdinalIgnoreCase))`
`270`	`270`	`{`
`271`	`271`
`272`		`- if (this.ShouldProcess(this.Name, "Create"))`
	`272`	`+ if (this.ShouldProcess(this.Name, "Creating data set mapping"))`
`273`	`273`	`{`
`274`	`274`	`if (this.FilePath != null)`
`275`	`275`	`{`
Original file line number	Diff line number	Diff line change
`@@ -10,5 +10,6 @@ public static class ResourceTypes`
`10`	`10`	`public const string ShareSubscription = "Microsoft.DataShare/sharesubscriptions";`
`11`	`11`	`public const string SynchronizationSetting = "Microsoft.DataShare/synchronizationSettings";`
`12`	`12`	`public const string Trigger = "Microsoft.DataShare/triggers";`
	`13`	`+ public const string StorageAccount = "Microsoft.Storage/storageAccounts";`
`13`	`14`	`}`
`14`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -10,5 +10,6 @@ class ParameterSetNames`
`10`	`10`	`public const string BlobDataSetParameterSet = "ByBlobDataSetParamaterSet";`
`11`	`11`	`public const string AdlsGen2DataSetParameterSet = "ByAdlsGen2DataSetParameterSet";`
`12`	`12`	`public const string AdlsGen1DataSetParameterSet = "ByAdlsGen1DataSetParameterSet";`
	`13`	`+ public const string ProviderShareSubscriptionIdParameterSet = "ByProviderShareSubscriptionIdParameterSet";`
`13`	`14`	`}`
`14`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,6 @@ public class NewAzDataShareSynchronizationSetting : AzureDataShareCmdletBase`
`81`	`81`	`HelpMessage = "Synchronization setting name",`
`82`	`82`	`ParameterSetName = ParameterSetNames.FieldsParameterSet)]`
`83`	`83`	`[ValidateNotNullOrEmpty]`
`84`		`- [ResourceNameCompleter(ResourceTypes.SynchronizationSetting, "ResourceGroupName", "AccountName", "ShareName")]`
`85`	`84`	`public string Name { get; set; }`
`86`	`85`
`87`	`86`	`/// <summary>`
`@@ -109,7 +108,7 @@ public class NewAzDataShareSynchronizationSetting : AzureDataShareCmdletBase`
`109`	`108`
`110`	`109`	`public override void ExecuteCmdlet()`
`111`	`110`	`{`
`112`		`- if (this.ShouldProcess(this.Name, "Create"))`
	`111`	`+ if (this.ShouldProcess(this.Name, "Creating share synchronization"))`
`113`	`112`	`{`
`114`	`113`	`var setting = new ScheduledSynchronizationSetting(`
`115`	`114`	`recurrenceInterval: this.RecurrenceInterval,`