Exclude events for MA0147

meziantou · Jan 13, 2024 · 1ddf530 · 1ddf530
1 parent 2807d2e
commit 1ddf530
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/src/Meziantou.Analyzer/Rules/DoNotUseAsyncDelegateForSyncDelegateAnalyzer.cs b/src/Meziantou.Analyzer/Rules/DoNotUseAsyncDelegateForSyncDelegateAnalyzer.cs
@@ -31,6 +31,9 @@ public override void Initialize(AnalysisContext context)
     private static void AnalyzerDelegateCreationOperation(OperationAnalysisContext context)
     {
         var operation = (IDelegateCreationOperation)context.Operation;
+        if (operation.Parent is IEventAssignmentOperation)
+            return;
+
         if (operation.Type is INamedTypeSymbol { DelegateInvokeMethod: IMethodSymbol delegateInvokeMethod })
         {
             if (!delegateInvokeMethod.ReturnsVoid)

diff --git a/tests/Meziantou.Analyzer.Test/Rules/DoNotUseAsyncDelegateForSyncDelegateAnalyzerTests.cs b/tests/Meziantou.Analyzer.Test/Rules/DoNotUseAsyncDelegateForSyncDelegateAnalyzerTests.cs
@@ -120,4 +120,20 @@ await CreateProjectBuilder()
                   """)
               .ValidateAsync();
     }
+
+    [Fact]
+    public async Task Event_AsyncVoid()
+    {
+        await CreateProjectBuilder()
+              .WithSourceCode("""
+                  Sample.A += async (sender, e) => { };
+                  Sample.A -= async (sender, e) => { };
+
+                  class Sample
+                  {
+                      public static event System.EventHandler A;
+                  }
+                  """)
+              .ValidateAsync();
+    }
 }