Fix the PosInfoMoq2005 rule to check only the class types.

PosInformatique · Jul 13, 2024 · 33de2d5 · 33de2d5
1 parent 64faf38
commit 33de2d5
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 6 deletions.
diff --git a/src/Moq.Analyzers/Analyzers/ConstructorArgumentsMustMatchAnalyzer.cs b/src/Moq.Analyzers/Analyzers/ConstructorArgumentsMustMatchAnalyzer.cs
@@ -72,6 +72,12 @@ private static void Analyze(SyntaxNodeAnalysisContext context)
                 return;
             }
 
+            // Check the type is a class (other type are ignored)
+            if (mockedType.TypeKind != TypeKind.Class)
+            {
+                return;
+            }
+
             // Check the type is a named type
             if (mockedType is not INamedTypeSymbol namedTypeSymbol)
             {

diff --git a/tests/Moq.Analyzers.Tests/Analyzers/ConstructorArgumentsMustMatchAnalyzerTest.cs b/tests/Moq.Analyzers.Tests/Analyzers/ConstructorArgumentsMustMatchAnalyzerTest.cs
@@ -208,8 +208,12 @@ private C(int a, object b, int c, System.IDisposable d)
             await Verifier.VerifyAnalyzerAsync(source);
         }
 
-        [Fact]
-        public async Task Arguments_Match_WithNoConstructor()
+
+        [Theory]
+        [InlineData("class")]
+        [InlineData("abstract class")]
+        [InlineData("interface")]
+        public async Task Arguments_Match_WithNoConstructor(string type)
         {
             var source = @"
                 namespace ConsoleApplication1
@@ -224,7 +228,7 @@ public void TestMethod()
                         }
                     }
 
-                    public class ClassWithNoConstructor { }
+                    public " + type + @" ClassWithNoConstructor { }
                 }";
 
             await Verifier.VerifyAnalyzerAsync(source);
@@ -272,8 +276,11 @@ public C(int a, object b, int c, System.IDisposable d)
             await Verifier.VerifyAnalyzerAsync(source);
         }
 
-        [Fact]
-        public async Task Arguments_Match_WithMockBehavior_WithNoConstructor()
+        [Theory]
+        [InlineData("class")]
+        [InlineData("abstract class")]
+        [InlineData("interface")]
+        public async Task Arguments_Match_WithMockBehavior_WithNoConstructor(string type)
         {
             var source = @"
                 namespace ConsoleApplication1
@@ -288,7 +295,7 @@ public void TestMethod()
                        }
                     }
 
-                    public class ClassWithNoConstructor { }
+                    public " + type + @" ClassWithNoConstructor { }
                 }";
 
             await Verifier.VerifyAnalyzerAsync(source);