Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Minor issues #53

Merged
merged 2 commits into from Apr 19, 2012

Conversation

Projects
None yet
2 participants

aliakb commented Apr 19, 2012

A couple of minor fixes.

@ogail ogail commented on the diff Apr 19, 2012

...r.UnitTests.MsTest/ServiceBusTests/ManagementTests.cs
@@ -950,5 +950,14 @@ public void InvalidArgsInCreateService()
Assert.ThrowsException<ArgumentException>(() => new ServiceBusClient(" ", validHandler));
Assert.ThrowsException<ArgumentNullException>(() => new ServiceBusClient("namespace", null));
}
+
+ /// <summary>
+ /// Tests passing invalid arguments to RuleSettings' constructor.
@ogail

ogail Apr 19, 2012

Contributor

Extra '

@ogail ogail commented on the diff Apr 19, 2012

...re.ServiceLayer/ServiceBus/BrokeredMessageSettings.cs
@@ -28,6 +28,12 @@ namespace Microsoft.WindowsAzure.ServiceLayer.ServiceBus
/// <summary>
/// Settings of a new brokered message.
/// </summary>
+ /// <remarks>Brokered messages support custom properties (see Properties).
+ /// A custom property can be a string, a number, a date/time, or a Boolea
@ogail

ogail Apr 19, 2012

Contributor

Boolean the n is missing

aliakb pushed a commit that referenced this pull request Apr 19, 2012

@aliakb aliakb merged commit 56b9ed1 into Azure:dev Apr 19, 2012

chuanboz pushed a commit to chuanboz/HDInsight that referenced this pull request Mar 7, 2015

Merge pull request #53 from hdinsight/dev
Bringing up Humboldt changes for GA

huangpf added a commit that referenced this pull request Jul 23, 2015

yugangw-msft pushed a commit that referenced this pull request Aug 18, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment