Skip to content

Commit b1b70b6

Browse files
authored
Fix RS1024 warnings (#169)
1 parent 4c033c0 commit b1b70b6

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

src/FluentAssertions.Analyzers/Tips/MsTest/MsTestBase.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ protected ExpressionSyntax GetNewExpressionForAreNotEqualOrAreEqualStrings(Expre
4848
ignoreCase = possibleIgnoreCaseArg.Token.IsKind(SyntaxKind.TrueKeyword);
4949
}
5050

51-
if (newArguments.Count >= 2 && semanticModel.GetCultureInfoType().Equals(semanticModel.GetTypeInfo(rename.Arguments[3].Expression).Type))
51+
if (newArguments.Count >= 2 && semanticModel.GetCultureInfoType().Equals(semanticModel.GetTypeInfo(rename.Arguments[3].Expression).Type, SymbolEqualityComparer.Default))
5252
{
5353
newArguments = newArguments.Remove(newArguments[1]);
5454
}

src/FluentAssertions.Analyzers/Tips/ShouldEquals.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ protected override async Task<bool> CanRewriteAssertion(ExpressionSyntax express
5353
var info = model.GetTypeInfo(member.Expression);
5454

5555
var taskCompletionSourceInfo = model.Compilation.GetTypeByMetadataName(typeof(TaskCompletionSource<>).FullName);
56-
if (info.Type.Equals(taskCompletionSourceInfo)) return false;
56+
if (info.Type.Equals(taskCompletionSourceInfo, SymbolEqualityComparer.Default)) return false;
5757

5858
var streamInfo = model.Compilation.GetTypeByMetadataName(typeof(Stream).FullName);
5959
if (info.Type.AllInterfaces.Contains(streamInfo)) return false;
@@ -73,7 +73,7 @@ protected override async Task<ExpressionSyntax> GetNewExpressionAsync(Expression
7373

7474
var stringInfo = model.Compilation.GetTypeByMetadataName(typeof(string).FullName);
7575

76-
if (info.Type.Equals(stringInfo))
76+
if (info.Type.Equals(stringInfo, SymbolEqualityComparer.Default))
7777
{
7878
return GetNewExpression(expression, NodeReplacement.Rename("Equals", "Be"));
7979
}

src/FluentAssertions.Analyzers/Utilities/MemberValidator.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ public static ArgumentPredicate IsType(Func<SemanticModel, INamedTypeSymbol> typ
112112
{
113113
return (argument, semanticModel) =>
114114
{
115-
return semanticModel.GetTypeInfo(argument.Expression).Type.Equals(typeSelector(semanticModel));
115+
return semanticModel.GetTypeInfo(argument.Expression).Type.Equals(typeSelector(semanticModel), SymbolEqualityComparer.Default);
116116
};
117117
}
118118
}

src/FluentAssertions.Analyzers/Utilities/TypesExtensions.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ public static bool IsTypeOrConstructedFromTypeOrImplementsType(this INamedTypeSy
1515
public static bool IsTypeOrConstructedFromTypeOrImplementsType(this INamedTypeSymbol type, INamedTypeSymbol other)
1616
{
1717
var abstractType = type.OriginalDefinition;
18-
return abstractType.Equals(other)
19-
|| abstractType.AllInterfaces.Any(@interface => @interface.OriginalDefinition.Equals(other));
18+
return abstractType.Equals(other, SymbolEqualityComparer.Default)
19+
|| abstractType.AllInterfaces.Any(@interface => @interface.OriginalDefinition.Equals(other, SymbolEqualityComparer.Default));
2020
}
2121
}
2222
}

0 commit comments

Comments
 (0)