fix

Azure · Nov 20, 2020 · 9a1ffe0 · 9a1ffe0
1 parent f3b5fc8
commit 9a1ffe0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/sdk/servicebus/Azure.Messaging.ServiceBus/src/Administration/QueueProperties.cs b/sdk/servicebus/Azure.Messaging.ServiceBus/src/Administration/QueueProperties.cs
@@ -347,7 +347,7 @@ public bool Equals(QueueProperties other)
                 && RequiresDuplicateDetection.Equals(otherDescription.RequiresDuplicateDetection)
                 && RequiresSession.Equals(otherDescription.RequiresSession)
                 && Status.Equals(otherDescription.Status)
-                && _internalSupportOrdering.Equals(other.SupportOrdering)
+                && SupportOrdering.Equals(other.SupportOrdering)
                 && EnableExpress == other.EnableExpress
                 && IsAnonymousAccessible == other.IsAnonymousAccessible
                 && string.Equals(_userMetadata, otherDescription._userMetadata, StringComparison.OrdinalIgnoreCase)

diff --git a/...us/Azure.Messaging.ServiceBus/tests/Administration/ServiceBusManagementClientLiveTests.cs b/...us/Azure.Messaging.ServiceBus/tests/Administration/ServiceBusManagementClientLiveTests.cs
@@ -94,10 +94,12 @@ public async Task BasicQueueCrudOperations()
             if (Mode == RecordedTestMode.Playback)
             {
                 Assert.AreEqual(queueOptions, new CreateQueueOptions(createdQueue) { AuthorizationRules = queueOptions.AuthorizationRules.Clone() });
+                Assert.AreEqual(createdQueue, new QueueProperties(queueOptions) { AuthorizationRules = createdQueue.AuthorizationRules });
             }
             else
             {
                 Assert.AreEqual(queueOptions, new CreateQueueOptions(createdQueue));
+                Assert.AreEqual(createdQueue, new QueueProperties(queueOptions));
             }
             Response<QueueProperties> getQueueResponse = await client.GetQueueAsync(queueOptions.Name);
             rawResponse = createdQueueResponse.GetRawResponse();
@@ -200,10 +202,13 @@ public async Task BasicTopicCrudOperations()
                 // these in our test recordings, so we skip the auth rule comparison
                 // when in playback mode.
                 Assert.AreEqual(options, new CreateTopicOptions(createdTopic) { AuthorizationRules = options.AuthorizationRules.Clone() });
+                Assert.AreEqual(createdTopic, new TopicProperties(options) { AuthorizationRules = createdTopic.AuthorizationRules.Clone() });
+
             }
             else
             {
                 Assert.AreEqual(options, new CreateTopicOptions(createdTopic));
+                Assert.AreEqual(createdTopic, new TopicProperties(options));
             }
 
             Response<TopicProperties> getTopicResponse = await client.GetTopicAsync(options.Name);