Fix RCS1241 (#1197)

ChangeLog.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 
 - Fix [RCS1164](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1164) ([#1196](https://github.com/JosefPihrt/Roslynator/pull/1196)).
+- Fix [RCS1241](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1241) ([#1197](https://github.com/JosefPihrt/Roslynator/pull/1197)).
 
 ## [4.5.0] - 2023-08-27
 

src/Analyzers.CodeFixes/CSharp/CodeFixes/ImplementNonGenericCounterpartCodeFixProvider.cs

@@ -290,6 +290,18 @@ private static async Task<Document> RefactorAsync(
 
             newTypeDeclaration = classDeclaration.WithBaseList(baseList.WithTypes(baseTypes));
         }
+        else if (kind == SyntaxKind.RecordDeclaration)
+        {
+            var recordDeclaration = (RecordDeclarationSyntax)newTypeDeclaration;
+
+            BaseListSyntax baseList = recordDeclaration.BaseList;
+
+            SeparatedSyntaxList<BaseTypeSyntax> baseTypes = baseList.Types;
+
+            baseTypes = AddBaseType(baseTypes, baseType);
+
+            newTypeDeclaration = recordDeclaration.WithBaseList(baseList.WithTypes(baseTypes));
+        }
         else if (kind == SyntaxKind.StructDeclaration
             || kind == SyntaxKind.RecordStructDeclaration)
         {

src/Analyzers/CSharp/Analysis/NamedTypeSymbolAnalyzer.cs

@@ -1,9 +1,11 @@
 // Copyright (c) Josef Pihrt and Contributors. Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
+using System;
 using System.Collections.Generic;
 using System.Collections.Immutable;
 using System.Linq;
 using Microsoft.CodeAnalysis;
+using Microsoft.CodeAnalysis.CSharp;
 using Microsoft.CodeAnalysis.CSharp.Syntax;
 using Microsoft.CodeAnalysis.Diagnostics;
 
@@ -130,19 +132,19 @@ private static void AnalyzeNamedType(SymbolAnalysisContext context)
 
     private static void ReportDiagnostic(SymbolAnalysisContext context, INamedTypeSymbol symbol, string interfaceName, string genericInterfaceName)
     {
-        SyntaxToken identifier = default;
+        SyntaxNode node = symbol.GetSyntax(context.CancellationToken);
 
-        if (symbol.TypeKind == TypeKind.Class)
-        {
-            var classDeclaration = (ClassDeclarationSyntax)symbol.GetSyntax(context.CancellationToken);
+        SyntaxToken identifier = GetIdentifier();
 
-            identifier = classDeclaration.Identifier;
-        }
-        else if (symbol.TypeKind == TypeKind.Struct)
+        SyntaxToken GetIdentifier()
         {
-            var structDeclaration = (StructDeclarationSyntax)symbol.GetSyntax(context.CancellationToken);
-
-            identifier = structDeclaration.Identifier;
+            return node switch
+            {
+                ClassDeclarationSyntax classDeclaration => classDeclaration.Identifier,
+                StructDeclarationSyntax structDeclaration => structDeclaration.Identifier,
+                RecordDeclarationSyntax recordDeclaration => recordDeclaration.Identifier,
+                _ => throw new InvalidOperationException($"Unknown syntax node kind '{node.Kind()}'."),
+            };
         }
 
         DiagnosticHelpers.ReportDiagnostic(

src/Tests/Analyzers.Tests/RCS1241ImplementNonGenericCounterpartTests.cs

@@ -64,6 +64,51 @@ public int CompareTo(object obj)
 ", equivalenceKey: EquivalenceKey.Create(Descriptor.Id));
     }
 
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ImplementNonGenericCounterpart)]
+    public async Task Test_Record_IComparable()
+    {
+        await VerifyDiagnosticAndFixAsync(@"
+using System;
+using System.Collections.Generic;
+
+public class C
+{
+}
+
+public abstract record class [|Comparable|] : IComparable<C>
+{
+    public abstract int CompareTo(C other);
+}
+", @"
+using System;
+using System.Collections.Generic;
+
+public class C
+{
+}
+
+public abstract record class Comparable : IComparable<C>, IComparable
+{
+    public abstract int CompareTo(C other);
+
+    public int CompareTo(object obj)
+    {
+        if (obj == null)
+        {
+            return 1;
+        }
+
+        if (obj is C x)
+        {
+            return CompareTo(x);
+        }
+
+        throw new ArgumentException("""", nameof(obj));
+    }
+}
+", equivalenceKey: EquivalenceKey.Create(Descriptor.Id));
+    }
+
     [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ImplementNonGenericCounterpart)]
     public async Task Test_IComparable_Explicit()
     {