A bit of testing

src/ntcore/NetworkTableInstance.cs

@@ -41,7 +41,7 @@ namespace NetworkTables;
  * kept to the NetworkTableInstance returned by this function to keep it from being garbage
  * collected.
  */
-public sealed partial class NetworkTableInstance : IEquatable<NetworkTableInstance?>, IEqualityOperators<NetworkTableInstance?, NetworkTableInstance?, bool>
+public sealed partial class NetworkTableInstance : IDisposable, IEquatable<NetworkTableInstance?>, IEqualityOperators<NetworkTableInstance?, NetworkTableInstance?, bool>
 {
     private static readonly ConcurrentDictionary<NtInst, NetworkTableInstance> s_instances = new();
 

test/ntcore.test/LoggerTest.cs

@@ -0,0 +1,33 @@
+using System.Runtime.InteropServices;
+using Xunit;
+
+namespace NetworkTables.Test;
+
+public class LoggerTest
+{
+    [Fact]
+    public void AddMessageTest()
+    {
+        using var instance = NetworkTableInstance.Create();
+        List<NetworkTableEvent> msgs = [];
+        instance.AddLogger((int)NtLogLevel.Info, 100, msgs.Add);
+
+        instance.StartClient4("client");
+        instance.SetServer("127.0.0.1", 10000);
+
+        // Wait for client to report it's started, then wait another 0.1 sec
+        int count = 0;
+        while (!instance.GetNetworkMode().HasFlag(NetworkMode.Client4))
+        {
+            Thread.Sleep(100);
+            count++;
+            if (count > 30)
+            {
+                Assert.Fail("Failed to converge");
+            }
+        }
+        Thread.Sleep(100);
+
+        Assert.NotEmpty(msgs);
+    }
+}