在Task.Run中调用EventHandler是否合理？异步方法单元测试代码优化咨询

直接把ProcessNotification改成同步方法是更合理的选择，原因如下：

• 没必要的异步包装：原代码里的Task.Run纯粹是把事件调用扔到线程池执行，属于无意义的“伪异步”——事件触发本身是同步操作，这种包装只会增加线程切换的额外开销，没有任何实际收益。
• 单元测试更简洁：同步方法不需要处理异步等待、上下文切换这些测试环节的额外复杂度，调用后就能直接验证事件是否触发、参数是否正确，大幅简化测试代码。
• 符合事件触发的常规设计：事件发布者的职责就是同步通知订阅者，至于订阅者的处理逻辑是否需要异步，应该由订阅方自己决定（比如在事件处理方法里用async/await），而不是发布者强行用Task.Run包装。
• 规避潜在风险：用Task.Run包装事件调用，可能导致事件处理代码在非预期线程执行（比如订阅者依赖UI线程上下文时会引发异常）；另外，异步包装后若事件处理抛出异常，还需要额外的捕获逻辑，同步版本能让异常直接传递给调用方，更符合常规错误处理流程。

原被测代码：

public class Foo
{
  public event EventHandler<bool> Disconnect;

  { ... }

  public async Task ProcessNotification(bool e)
  {
    await Task.Run(() => Disconnect?.Invoke(this, e)).ConfigureAwait(false);
  }
}

修改后的代码：

public class Foo
{
  public event EventHandler<bool> Disconnect;

  { ... }

  public void ProcessNotification(bool e)
  {
    Disconnect?.Invoke(this, e);
  }
}

内容的提问来源于stack exchange，提问作者Alberto Martín