Skip to content
This repository was archived by the owner on Oct 12, 2023. It is now read-only.

Commit 1d5df37

Browse files
ryanspletzernemakam
authored andcommitted
Changing topicName param to topicPath in CreateRuleAsync, fixing variable typo in CreateRuleAsync (#546)
* Changing topicName to topicPath in CreateRuleAsync CreateRuleAsync uses topicName as a parameter whereas the rest of the methods use topicPath. Changing for consistency. * Fixing var typo in CreateRuleAsync * Updating ApiApprovals.ApproveAzureServiceBus.approved.txt * Adding back in trimmed whitespace to approved txt file to resolve unit test issue * Fixed topicPath description in comments
1 parent 1a742c5 commit 1d5df37

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

src/Microsoft.Azure.ServiceBus/Management/ManagementClient.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -634,7 +634,7 @@ public virtual async Task<SubscriptionDescription> CreateSubscriptionAsync(Subsc
634634
/// <summary>
635635
/// Adds a new rule to the subscription under given topic.
636636
/// </summary>
637-
/// <param name="topicName">The topic name relative to the service namespace base address.</param>
637+
/// <param name="topicPath">The topic path relative to the service namespace base address.</param>
638638
/// <param name="subscriptionName">The name of the subscription.</param>
639639
/// <param name="ruleDescription">A <see cref="RuleDescription"/> object describing the attributes with which the messages are matched and acted upon.</param>
640640
/// <param name="cancellationToken"></param>
@@ -646,16 +646,16 @@ public virtual async Task<SubscriptionDescription> CreateSubscriptionAsync(Subsc
646646
/// <exception cref="ServerBusyException">The server is busy. You should wait before you retry the operation.</exception>
647647
/// <exception cref="ServiceBusException">An internal error or unexpected exception occurs.</exception>
648648
/// <returns><see cref="RuleDescription"/> of the recently created rule.</returns>
649-
public virtual async Task<RuleDescription> CreateRuleAsync(string topicName, string subscriptionName, RuleDescription ruleDescription, CancellationToken cancellationToken = default)
649+
public virtual async Task<RuleDescription> CreateRuleAsync(string topicPath, string subscriptionName, RuleDescription ruleDescription, CancellationToken cancellationToken = default)
650650
{
651-
EntityNameHelper.CheckValidTopicName(topicName);
652-
EntityNameHelper.CheckValidSubscriptionName(topicName);
651+
EntityNameHelper.CheckValidTopicName(topicPath);
652+
EntityNameHelper.CheckValidSubscriptionName(subscriptionName);
653653
ruleDescription = ruleDescription ?? throw new ArgumentNullException(nameof(ruleDescription));
654654

655655
var atomRequest = ruleDescription.Serialize().ToString();
656656

657657
var content = await PutEntity(
658-
EntityNameHelper.FormatRulePath(topicName, subscriptionName, ruleDescription.Name),
658+
EntityNameHelper.FormatRulePath(topicPath, subscriptionName, ruleDescription.Name),
659659
atomRequest,
660660
false,
661661
null,

test/Microsoft.Azure.ServiceBus.UnitTests/API/ApiApprovals.ApproveAzureServiceBus.approved.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -657,7 +657,7 @@ namespace Microsoft.Azure.ServiceBus.Management
657657
public System.Threading.Tasks.Task CloseAsync() { }
658658
public virtual System.Threading.Tasks.Task<Microsoft.Azure.ServiceBus.Management.QueueDescription> CreateQueueAsync(string queuePath, System.Threading.CancellationToken cancellationToken = null) { }
659659
public virtual System.Threading.Tasks.Task<Microsoft.Azure.ServiceBus.Management.QueueDescription> CreateQueueAsync(Microsoft.Azure.ServiceBus.Management.QueueDescription queueDescription, System.Threading.CancellationToken cancellationToken = null) { }
660-
public virtual System.Threading.Tasks.Task<Microsoft.Azure.ServiceBus.RuleDescription> CreateRuleAsync(string topicName, string subscriptionName, Microsoft.Azure.ServiceBus.RuleDescription ruleDescription, System.Threading.CancellationToken cancellationToken = null) { }
660+
public virtual System.Threading.Tasks.Task<Microsoft.Azure.ServiceBus.RuleDescription> CreateRuleAsync(string topicPath, string subscriptionName, Microsoft.Azure.ServiceBus.RuleDescription ruleDescription, System.Threading.CancellationToken cancellationToken = null) { }
661661
public virtual System.Threading.Tasks.Task<Microsoft.Azure.ServiceBus.Management.SubscriptionDescription> CreateSubscriptionAsync(string topicPath, string subscriptionName, System.Threading.CancellationToken cancellationToken = null) { }
662662
public virtual System.Threading.Tasks.Task<Microsoft.Azure.ServiceBus.Management.SubscriptionDescription> CreateSubscriptionAsync(Microsoft.Azure.ServiceBus.Management.SubscriptionDescription subscriptionDescription, System.Threading.CancellationToken cancellationToken = null) { }
663663
public virtual System.Threading.Tasks.Task<Microsoft.Azure.ServiceBus.Management.SubscriptionDescription> CreateSubscriptionAsync(Microsoft.Azure.ServiceBus.Management.SubscriptionDescription subscriptionDescription, Microsoft.Azure.ServiceBus.RuleDescription defaultRule, System.Threading.CancellationToken cancellationToken = null) { }

0 commit comments

Comments
 (0)