Skip to content

Commit 39e2c62

Browse files
committed
Fixed warnings in tests
1 parent 641d8ee commit 39e2c62

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

openapi-diff/src/modeler/AutoRest.Swagger.Tests/SwaggerModelerCompareTests.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ public void AddedReadOnlyRequiredProperty()
368368
ComparisonMessage[] messages = CompareSwagger("added_readonly_required_property.json").ToArray();
369369
List<ComparisonMessage> addedReqPropMessages = messages.Where(
370370
m => m.Severity == Category.Error && m.Id == ComparisonMessages.AddedRequiredProperty.Id).ToList();
371-
Assert.Equal(0, addedReqPropMessages.Count);
371+
Assert.Empty(addedReqPropMessages);
372372
}
373373

374374
/// <summary>
@@ -842,21 +842,21 @@ public void GlobalResponseTypeConstraintsChanged()
842842
public void RemovedPropertyTest()
843843
{
844844
var messages = CompareSwagger("removed_property.json").ToArray();
845-
Assert.True(messages.Where(m => m.Id == ComparisonMessages.RemovedProperty.Id).Any());
845+
Assert.NotEmpty(messages.Where(m => m.Id == ComparisonMessages.RemovedProperty.Id));
846846
}
847847

848848
[Fact]
849849
public void FormatChanged()
850850
{
851851
var messages = CompareSwagger("format_check_01.json").ToArray();
852-
Assert.True(messages.Where(m => m.Id == ComparisonMessages.TypeFormatChanged.Id).Any());
852+
Assert.NotEmpty(messages.Where(m => m.Id == ComparisonMessages.TypeFormatChanged.Id));
853853
}
854854

855855
[Fact]
856856
public void FormatRemoved()
857857
{
858858
var messages = CompareSwagger("format_check_02.json").ToArray();
859-
Assert.True(messages.Where(m => m.Id == ComparisonMessages.TypeFormatChanged.Id).Any());
859+
Assert.NotEmpty(messages.Where(m => m.Id == ComparisonMessages.TypeFormatChanged.Id));
860860
}
861861

862862
[Fact]
@@ -952,7 +952,7 @@ public void DidNotChangeGlobalParameterOrder()
952952
// of Implicit and Client doesn't matter.
953953
//
954954
// Reference: https://github.com/Azure/autorest/blob/main/docs/extensions/readme.md#x-ms-parameter-location
955-
Assert.Equal(0, messages.Where(m => m.Id == ComparisonMessages.ChangedParameterOrder.Id).Count());
955+
Assert.Empty(messages.Where(m => m.Id == ComparisonMessages.ChangedParameterOrder.Id));
956956
}
957957

958958
[Fact]
@@ -1047,11 +1047,11 @@ public void ParameterRemoved()
10471047

10481048
foreach (var reqParamName in reqParamNames)
10491049
{
1050-
Assert.True(reqParamMessageStrings.Any(str => str.Contains(reqParamName)));
1050+
Assert.Contains(reqParamMessageStrings, str => str.Contains(reqParamName));
10511051
}
10521052
foreach (var optParamName in optParamNames)
10531053
{
1054-
Assert.True(optParamMessageStrings.Any(str => str.Contains(optParamName)));
1054+
Assert.Contains(optParamMessageStrings, str => str.Contains(optParamName));
10551055
}
10561056
}
10571057
}

0 commit comments

Comments
 (0)