Cleanup + stability

Author: NickCraver

tests/StackExchange.Redis.Tests/AsyncTests.cs

@@ -9,8 +9,6 @@ namespace StackExchange.Redis.Tests;
 [Collection(NonParallelCollection.Name)]
 public class AsyncTests(ITestOutputHelper output) : TestBase(output)
 {
-    protected override string GetConfiguration() => TestConfig.Current.PrimaryServerAndPort;
-
     [Fact]
     public async Task AsyncTasksReportFailureIfServerUnavailable()
     {

tests/StackExchange.Redis.Tests/ClientKillTests.cs

@@ -10,8 +10,6 @@ namespace StackExchange.Redis.Tests;
 
 public class ClientKillTests(ITestOutputHelper output) : TestBase(output)
 {
-    protected override string GetConfiguration() => TestConfig.Current.PrimaryServerAndPort;
-
     [Fact]
     public async Task ClientKill()
     {

tests/StackExchange.Redis.Tests/ConnectionShutdownTests.cs

@@ -7,8 +7,6 @@ namespace StackExchange.Redis.Tests;
 
 public class ConnectionShutdownTests(ITestOutputHelper output) : TestBase(output)
 {
-    protected override string GetConfiguration() => TestConfig.Current.PrimaryServerAndPort;
-
     [Fact(Skip = "Unfriendly")]
     public async Task ShutdownRaisesConnectionFailedAndRestore()
     {

tests/StackExchange.Redis.Tests/Issues/SO25567566Tests.cs

@@ -6,8 +6,6 @@ namespace StackExchange.Redis.Tests.Issues;
 
 public class SO25567566Tests(ITestOutputHelper output) : TestBase(output)
 {
-    protected override string GetConfiguration() => TestConfig.Current.PrimaryServerAndPort;
-
     [Fact]
     public async Task Execute()
     {

tests/StackExchange.Redis.Tests/LockingTests.cs

@@ -9,8 +9,6 @@ namespace StackExchange.Redis.Tests;
 [Collection(NonParallelCollection.Name)]
 public class LockingTests(ITestOutputHelper output) : TestBase(output)
 {
-    protected override string GetConfiguration() => TestConfig.Current.PrimaryServerAndPort;
-
     public enum TestMode
     {
         MultiExec,

tests/StackExchange.Redis.Tests/LoggerTests.cs

@@ -12,8 +12,6 @@ namespace StackExchange.Redis.Tests;
 [Collection(NonParallelCollection.Name)]
 public class LoggerTests(ITestOutputHelper output) : TestBase(output)
 {
-    protected override string GetConfiguration() => TestConfig.Current.PrimaryServerAndPort;
-
     [Fact]
     public async Task BasicLoggerConfig()
     {

tests/StackExchange.Redis.Tests/ScriptingTests.cs

@@ -435,6 +435,7 @@ async Task<bool> EvaluateScript()
     [Fact]
     public async Task CompareScriptToDirect()
     {
+        Skip.UnlessLongRunning();
         await using var conn = Create(allowAdmin: true, require: RedisFeatures.v2_6_0);
 
         const string Script = "return redis.call('incr', KEYS[1])";
@@ -448,7 +449,7 @@ public async Task CompareScriptToDirect()
         // we're using a pipeline here, so send 1000 messages, but for timing: only care about the last
         const int Loop = 5000;
         RedisKey key = Me();
-        RedisKey[] keys = new[] { key }; // script takes an array
+        RedisKey[] keys = [key]; // script takes an array
 
         // run via script
         db.KeyDelete(key, CommandFlags.FireAndForget);

tests/StackExchange.Redis.Tests/SocketTests.cs

@@ -6,8 +6,6 @@ namespace StackExchange.Redis.Tests;
 
 public class SocketTests(ITestOutputHelper output) : TestBase(output)
 {
-    protected override string GetConfiguration() => TestConfig.Current.PrimaryServerAndPort;
-
     [Fact]
     public async Task CheckForSocketLeaks()
     {