Fixing comments from Manfred.

src/NUnitFramework/framework/Internal/Execution/EventPump.cs

@@ -1,8 +1,8 @@
 // Copyright (c) Charlie Poole, Rob Prouse and Contributors. MIT License - see LICENSE.txt
 
-using NUnit.Framework.Interfaces;
 using System;
 using System.Threading;
+using NUnit.Framework.Interfaces;
 
 namespace NUnit.Framework.Internal.Execution
 {
@@ -166,7 +166,7 @@ public void Stop()
         /// </summary>
         private void PumpThreadProc()
         {
-            Log.Debug($"Starting EventPump {Name}");
+            Log.Debug($"Starting {Name}");
 
             //ITestListener hostListeners = CoreExtensions.Host.Listeners;
             try

src/NUnitFramework/framework/Internal/Execution/EventQueue.cs

@@ -29,7 +29,7 @@ public interface IEvent<in TListener>
     /// or to queue them for forwarding on another thread or at
     /// a later time.
     /// </summary>
-    public abstract class Event : IEvent<ITestListener>, IDisposable
+    public abstract class Event : IEvent<ITestListener> , IDisposable
     {
         /// <summary>
         /// The Send method is implemented by derived classes to send the event to the specified listener.
@@ -40,7 +40,7 @@ public abstract class Event : IEvent<ITestListener>, IDisposable
         /// <summary>
         /// Not quite sure why this is needed, but the compiler complained.
         /// </summary>
-        public void Dispose()
+        public virtual void Dispose()
         {
         }
     }