refactor: fix sonar issues due to updated analyzers (#443)

- Make `TestingException` only serializable when using an older
Framework than .NET 8
- Fix
[CA1859](https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1859)
Use concrete types when possible for improved performance
- Fix
[CA1866](https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1865-ca1867)
The char overload is a better performing overload than a string with a
single char.
- Fix [xUnit1042](https://xunit.net/xunit.analyzers/rules/xUnit1042) The
member referenced by the MemberData attribute returns untyped data rows
- Disable the following rules:
-
[CA1510](https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1510)
Use ArgumentNullException throw helper
-
[CA1512](https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1512)
Use ArgumentOutOfRangeException throw helper
-
[CA1513](https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1513)
Use ObjectDisposedException throw helper
-
[CA1860](https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1860)
Avoid using 'Enumerable.Any()' extension method
-
[CA1861](https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1861)
Avoid constant arrays as arguments

Author: vbreuss

.editorconfig

@@ -229,3 +229,10 @@ dotnet_naming_style.begins_with_i.capitalization = pascal_case
 
 # ReSharper inspection severities
 resharper_arrange_method_or_operator_body_highlighting = hint
+
+# Diagnostics
+dotnet_diagnostic.CA1510.severity = none
+dotnet_diagnostic.CA1512.severity = none
+dotnet_diagnostic.CA1513.severity = none
+dotnet_diagnostic.CA1860.severity = none
+dotnet_diagnostic.CA1861.severity = none

Source/Testably.Abstractions.Compression/Internal/ZipUtilities.cs

@@ -208,7 +208,7 @@ internal static void ExtractRelativeToDirectory(this IZipArchiveEntry source,
 			source.FileSystem.Directory.CreateDirectory(directoryPath);
 		}
 
-		if (source.FullName.EndsWith("/"))
+		if (source.FullName.EndsWith('/'))
 		{
 			if (source.Length != 0)
 			{

Source/Testably.Abstractions.Compression/Polyfills/StringExtensionMethods.cs

@@ -0,0 +1,22 @@
+#if NETSTANDARD2_0
+using System.Diagnostics.CodeAnalysis;
+
+namespace Testably.Abstractions.Polyfills;
+
+/// <summary>
+///     Provides extension methods to simplify writing platform independent tests.
+/// </summary>
+[ExcludeFromCodeCoverage]
+internal static class StringExtensionMethods
+{
+	/// <summary>
+	///     Determines whether the end of this string instance matches the specified character.
+	/// </summary>
+	internal static bool EndsWith(
+		this string @this,
+		char value)
+	{
+		return @this.EndsWith($"{value}");
+	}
+}
+#endif

Source/Testably.Abstractions.Compression/Usings.cs

@@ -1 +1,4 @@
-global using System.IO.Abstractions;
+#if NETSTANDARD2_0 || NETSTANDARD2_1
+global using Testably.Abstractions.Polyfills;
+# endif
+global using System.IO.Abstractions;

Source/Testably.Abstractions.Testing/FileSystem/FileSystemInfoMock.cs

@@ -114,7 +114,7 @@ public string Extension
 	{
 		get
 		{
-			if (Location.FullPath.EndsWith(".") &&
+			if (Location.FullPath.EndsWith('.') &&
 			    !Execute.IsWindows)
 			{
 				return ".";

Source/Testably.Abstractions.Testing/FileSystemInitializer/TestingException.cs

@@ -6,7 +6,9 @@ namespace Testably.Abstractions.Testing.FileSystemInitializer;
 /// <summary>
 ///     Custom <see cref="TestingException" /> when using the test system incorrectly.
 /// </summary>
+#if !NET8_0_OR_GREATER
 [Serializable]
+#endif
 public class TestingException : Exception
 {
 	/// <summary>

Source/Testably.Abstractions.Testing/Helpers/RandomFactory.cs

@@ -20,7 +20,7 @@ public static IRandom Shared
 	/// <summary>
 	///     <see href="https://andrewlock.net/building-a-thread-safe-random-implementation-for-dotnet-framework/" />
 	/// </summary>
-	private static IRandom CreateThreadSafeRandomWrapper()
+	private static RandomWrapper CreateThreadSafeRandomWrapper()
 	{
 		int seed;
 		lock (Global)

Source/Testably.Abstractions.Testing/Polyfills/StringExtensionMethods.cs

@@ -18,5 +18,15 @@ internal static bool EndsWith(
 	{
 		return @this.EndsWith($"{value}");
 	}
+	
+	/// <summary>
+	///     Determines whether this string instance starts with the specified character.
+	/// </summary>
+	internal static bool StartsWith(
+		this string @this,
+		char value)
+	{
+		return @this.StartsWith($"{value}");
+	}
 }
 #endif

Source/Testably.Abstractions.Testing/RandomSystem/RandomProviderMock.cs

@@ -6,7 +6,7 @@ namespace Testably.Abstractions.Testing.RandomSystem;
 
 internal sealed class RandomProviderMock : IRandomProvider
 {
-	[ThreadStatic] private static IRandom? _shared;
+	[ThreadStatic] private static RandomMock? _shared;
 
 	private static Generator<Guid> DefaultGuidGenerator
 		=> Generator<Guid>.FromCallback(Guid.NewGuid);
@@ -34,7 +34,7 @@ public IRandom GetRandom(int seed = SharedSeed)
 
 	#endregion
 
-	private static IRandom DefaultRandomGenerator(int seed)
+	private static RandomMock DefaultRandomGenerator(int seed)
 	{
 		if (seed == SharedSeed)
 		{

Source/Testably.Abstractions.Testing/Storage/InMemoryContainer.cs

@@ -250,7 +250,7 @@ public static IStorageContainer NewFile(IStorageLocation location,
 
 	internal FileAttributes AdjustAttributes(FileAttributes attributes)
 	{
-		if (Path.GetFileName(_location.FullPath).StartsWith("."))
+		if (Path.GetFileName(_location.FullPath).StartsWith('.'))
 		{
 			FileAttributes attr = attributes;
 			attributes = Execute.OnLinux(