Address PR review: unused using, fixture namespace, getter visibility

Author: paulmedynski

src/Microsoft.Data.SqlClient/tests/UnitTests/SimulatedServerTests/Fixtures/TdsServerFixture.cs

@@ -5,6 +5,8 @@
 using System;
 using Microsoft.SqlServer.TDS.Servers;
 
+namespace Microsoft.Data.SqlClient.UnitTests.SimulatedServerTests;
+
 /// <summary>
 /// An xunit test fixture that manages the lifecycle of a TdsServer.
 /// </summary>

src/Microsoft.Data.SqlClient/tests/UnitTests/SimulatedServerTests/TdsTokenBoundsTests.cs

@@ -10,7 +10,6 @@
 using Microsoft.SqlServer.TDS.Done;
 using Microsoft.SqlServer.TDS.FeatureExtAck;
 using Microsoft.SqlServer.TDS.Servers;
-using Microsoft.SqlServer.TDS.SessionState;
 using Xunit;
 
 namespace Microsoft.Data.SqlClient.UnitTests.SimulatedServerTests;

src/Microsoft.Data.SqlClient/tests/tools/TDS/TDS.Servers/GenericTdsServer.cs

@@ -138,7 +138,7 @@ public GenericTdsServer(T arguments, QueryEngine queryEngine)
         /// sent to the client. Tests can use this to inject or replace tokens in the
         /// response message.
         /// </summary>
-        public Action<TDSMessage> OnSQLBatchCompleted { get; set; }
+        public Action<TDSMessage> OnSQLBatchCompleted { private get; set; }
 
         public OnLogin7ValidatedDelegate OnLogin7Validated { private get; set; }