Skip to content
New issue

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

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove comments regarding 1449 workaround #1601

Merged
merged 1 commit into from Oct 20, 2016
Merged

Remove comments regarding 1449 workaround #1601

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/System.ServiceModel.Http/tests/ServiceModel/BasicHttpBindingTest.cs
View file
Open in desktop
Expand Up @@ -167,7 +167,7 @@ public static void Name_Property_Sets(string value)
}

[WcfTheory]
[InlineData(new object[] { null } )] // Work-around issue #1449 with this syntax
[InlineData(new object[] { null } )]
[InlineData("")]
public static void Name_Property_Set_Invalid_Value_Throws(string value)
{
Expand All @@ -187,7 +187,7 @@ public static void Namespace_Property_Sets(string value)
}

[WcfTheory]
[InlineData(new object[] { null } )] // Work-around issue #1449 with this syntax
[InlineData(new object[] { null } )]
public static void Namespace_Property_Set_Invalid_Value_Throws(string value)
{
var binding = new BasicHttpBinding();
Expand Down
2 changes: 1 addition & 1 deletion src/System.ServiceModel.Primitives/tests/Channels/CustomBindingTest.cs
View file
Open in desktop
Expand Up @@ -49,7 +49,7 @@ public static void CustomBinding_Name_Property(string bindingName)

[WcfTheory]
[InlineData("")]
[InlineData(new object[] { null } )] // Work-around issue #1449 with this syntax
[InlineData(new object[] { null } )]
public static void CustomBinding_Name_Property_Set_Throws(string bindingName)
{
CustomBinding customBinding = new CustomBinding();
Expand Down
4 changes: 2 additions & 2 deletions src/System.ServiceModel.Primitives/tests/ServiceModel/MessageContractTest.cs
View file
Open in desktop
Expand Up @@ -53,7 +53,7 @@ public static void WrapperName_Property_Sets_Throws_Argument(string wrapperName)
}

[WcfTheory]
[InlineData(new object[] { null } )] // Work-around issue #1449 with this syntax
[InlineData(new object[] { null } )]
public static void WrapperName_Property_Sets_Throws_ArgumentNull(string wrapperName)
{
MessageContractAttribute messageCA = new MessageContractAttribute();
Expand All @@ -64,7 +64,7 @@ public static void WrapperName_Property_Sets_Throws_ArgumentNull(string wrapperN
[InlineData("http://www.contoso.com")]
[InlineData("testNamespace")]
[InlineData("")]
[InlineData(new object[] { null } )] // Work-around issue #1449 with this syntax
[InlineData(new object[] { null } )]
public static void WrapperNamespace_Property_Sets(string wrapperNamespace)
{
MessageContractAttribute messageCA = new MessageContractAttribute();
Expand Down