Co-authored-by: BenjaminMichaelis <[email protected]>

Author: Copilot

IntelliTect.Analyzer/IntelliTect.Analyzer/Analyzers/NamingMethodPascal.cs

@@ -107,7 +107,8 @@ private static void AnalyzeSymbol(SymbolAnalysisContext context)
 
         private static bool IsTestMethod(IMethodSymbol methodSymbol)
         {
-            // Test framework namespaces - checking namespace is more flexible than specific attribute names
+            // Test framework namespaces - any method decorated with an attribute from these namespaces
+            // is considered a test method and exempt from PascalCase validation
             string[] testFrameworkNamespaces = 
             [
                 "Xunit",                                    // xUnit (note: namespace is "Xunit", not "XUnit")
@@ -116,7 +117,8 @@ private static bool IsTestMethod(IMethodSymbol methodSymbol)
                 "TUnit.Core"                                // TUnit
             ];
 
-            // Fallback attribute names for test environments where namespace metadata may be incomplete
+            // Fallback attribute names - needed because our test infrastructure (DiagnosticVerifier)
+            // doesn't add references to test framework assemblies, so ContainingNamespace would be null
             string[] commonTestAttributeNames =
             [
                 "TestMethod", "TestMethodAttribute",        // MSTest
@@ -135,15 +137,15 @@ private static bool IsTestMethod(IMethodSymbol methodSymbol)
                     return false;
                 }
 
-                // Check namespace first (more robust for production code)
+                // Check namespace first (works in production with proper assembly references)
                 string containingNamespace = attribute.AttributeClass.ContainingNamespace?.ToDisplayString();
                 if (containingNamespace != null && 
                     testFrameworkNamespaces.Any(ns => containingNamespace.StartsWith(ns, StringComparison.Ordinal)))
                 {
                     return true;
                 }
 
-                // Fallback: check attribute name for common test attributes
+                // Fallback: check attribute name (needed for test environment)
                 string attributeName = attribute.AttributeClass.Name;
                 return commonTestAttributeNames.Contains(attributeName);
             });