Restores parallel settings, fixes filename test-args formatting

Author: benbierens

Tests/CodexReleaseTests/Parallelism.cs

@@ -1,6 +1,6 @@
 using NUnit.Framework;
 
-[assembly: LevelOfParallelism(10)]
+[assembly: LevelOfParallelism(1)]
 namespace CodexReleaseTests.DataTests
 {
 }

Tests/DistTestCore/Configuration.cs

@@ -14,7 +14,7 @@ public Configuration()
             kubeConfigFile = GetNullableEnvVarOrDefault("KUBECONFIG", null);
             logPath = GetEnvVarOrDefault("LOGPATH", "CodexTestLogs");
             dataFilesPath = GetEnvVarOrDefault("DATAFILEPATH", "TestDataFiles");
-            AlwaysDownloadContainerLogs = true; // !string.IsNullOrEmpty(GetEnvVarOrDefault("ALWAYS_LOGS", ""));
+            AlwaysDownloadContainerLogs = !string.IsNullOrEmpty(GetEnvVarOrDefault("ALWAYS_LOGS", ""));
         }
 
         public Configuration(string? kubeConfigFile, string logPath, string dataFilesPath)

Tests/DistTestCore/NameUtils.cs

@@ -55,12 +55,23 @@ private static string GetEnvVar(string name, string defaultValue)
         private static string FormatArguments(TestContext.TestAdapter test)
         {
             if (test.Arguments == null || !test.Arguments.Any()) return "";
-            return $"[{string.Join(',', test.Arguments)}]";
+            return $"[{string.Join(',', test.Arguments.Select(FormatArgument).ToArray())}]";
+        }
+        
+        private static string FormatArgument(object? obj)
+        {
+            if (obj == null) return "";
+            var str = obj.ToString();
+            if (string.IsNullOrEmpty(str)) return "";
+            return ReplaceInvalidCharacters(str);
         }
 
         private static string ReplaceInvalidCharacters(string name)
         {
-            return name.Replace(":", "_");
+            return name
+                .Replace(":", "_")
+                .Replace("/", "_")
+                .Replace("\\", "_");
         }
 
         private static string DetermineFolder(LogConfig config, DateTime start)