Fix issue with table clean up in unit tests

Author: stuartpa

MssqlMcp/MssqlMcp.Tests/UnitTests.cs

@@ -6,7 +6,7 @@
 
 namespace MssqlMcp.Tests
 {
-    public sealed class MssqlMcpTests
+    public sealed class MssqlMcpTests : IDisposable
     {
         private readonly string _tableName;
         private readonly Tools _tools;
@@ -18,7 +18,7 @@ public MssqlMcpTests()
             _tools = new Tools(connectionFactory, loggerMock.Object);
         }
 
-        internal void Dispose()
+        public void Dispose()
         {
             // Cleanup: Drop the table after each test
             var _ = _tools.DropTable($"DROP TABLE IF EXISTS {_tableName}").GetAwaiter().GetResult();

MssqlMcp/MssqlMcp/Tools/ReadData.cs

@@ -23,7 +23,7 @@ public async Task<DbOperationResult> ReadData(
                 while (await reader.ReadAsync())
                 {
                     var row = new Dictionary<string, object?>();
-                    for (int i = 0; i < reader.FieldCount; i++)
+                    for (var i = 0; i < reader.FieldCount; i++)
                     {
                         row[reader.GetName(i)] = reader.IsDBNull(i) ? null : reader.GetValue(i);
                     }