refactor: exclude obsolete methods from code coverage

Source/Testably.Abstractions.Interface/Helpers/PathSystemBase.cs

@@ -15,6 +15,7 @@ namespace Testably.Abstractions.Helpers;
 #else
 [Obsolete("Will be removed in a future version!")]
 #endif
+[ExcludeFromCodeCoverage]
 public abstract class PathSystemBase : IPath
 {
 	/// <summary>

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

@@ -23,6 +23,7 @@ public IFileSystem FileSystem
 
 	/// <inheritdoc cref="IDirectoryInfoFactory.FromDirectoryName(string)" />
 	[Obsolete("Use `IDirectoryInfoFactory.New(string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IDirectoryInfo FromDirectoryName(string directoryName)
 	{
 		using IDisposable registration = RegisterMethod(nameof(FromDirectoryName),

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

@@ -24,6 +24,7 @@ public IFileSystem FileSystem
 
 	/// <inheritdoc cref="IDriveInfoFactory.FromDriveName(string)" />
 	[Obsolete("Use `IDriveInfoFactory.New(string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IDriveInfo FromDriveName(string driveName)
 	{
 		using IDisposable registration = RegisterMethod(nameof(FromDriveName),

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

@@ -23,6 +23,7 @@ public IFileSystem FileSystem
 
 	/// <inheritdoc cref="IFileInfoFactory.FromFileName(string)" />
 	[Obsolete("Use `IFileInfoFactory.New(string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IFileInfo FromFileName(string fileName)
 	{
 		using IDisposable registration = RegisterMethod(nameof(FromFileName),

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

@@ -1,11 +1,9 @@
 using Microsoft.Win32.SafeHandles;
 using System;
+using System.Diagnostics.CodeAnalysis;
 using System.IO;
 using Testably.Abstractions.Testing.Helpers;
 using Testably.Abstractions.Testing.Statistics;
-#if NET6_0_OR_GREATER
-using System.Diagnostics.CodeAnalysis;
-#endif
 
 namespace Testably.Abstractions.Testing.FileSystem;
 
@@ -29,28 +27,33 @@ public IFileSystem FileSystem
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode)" />
 	[Obsolete("Use `IFileStreamFactory.New(string, FileMode)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(string path, FileMode mode)
 		=> New(path, mode);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess)" />
 	[Obsolete("Use `IFileStreamFactory.New(string, FileMode, FileAccess)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(string path, FileMode mode, FileAccess access)
 		=> New(path, mode, access);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess, FileShare)" />
 	[Obsolete("Use `IFileStreamFactory.New(string, FileMode, FileAccess, FileShare)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(string path, FileMode mode, FileAccess access, FileShare share)
 		=> New(path, mode, access, share);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess, FileShare, int)" />
 	[Obsolete("Use `IFileStreamFactory.New(string, FileMode, FileAccess, FileShare, int)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(string path, FileMode mode, FileAccess access, FileShare share,
 		int bufferSize)
 		=> New(path, mode, access, share, bufferSize);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess, FileShare, int, FileOptions)" />
 	[Obsolete(
 		"Use `IFileStreamFactory.New(string, FileMode, FileAccess, FileShare, int, FileOptions)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(string path, FileMode mode, FileAccess access, FileShare share,
 		int bufferSize,
 		FileOptions options)
@@ -59,47 +62,55 @@ public Stream Create(string path, FileMode mode, FileAccess access, FileShare sh
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess, FileShare, int, bool)" />
 	[Obsolete(
 		"Use `IFileStreamFactory.New(string, FileMode, FileAccess, FileShare, int, bool)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(string path, FileMode mode, FileAccess access, FileShare share,
 		int bufferSize,
 		bool useAsync)
 		=> New(path, mode, access, share, bufferSize, useAsync);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(SafeFileHandle, FileAccess)" />
 	[Obsolete("Use `IFileStreamFactory.New(SafeFileHandle, FileAccess)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(SafeFileHandle handle, FileAccess access)
 		=> New(handle, access);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(SafeFileHandle, FileAccess, int)" />
 	[Obsolete("Use `IFileStreamFactory.New(SafeFileHandle, FileAccess, int)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(SafeFileHandle handle, FileAccess access, int bufferSize)
 		=> New(handle, access, bufferSize);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(SafeFileHandle, FileAccess, int, bool)" />
 	[Obsolete("Use `IFileStreamFactory.New(SafeFileHandle, FileAccess, int, bool)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(SafeFileHandle handle, FileAccess access, int bufferSize, bool isAsync)
 		=> New(handle, access, bufferSize, isAsync);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(IntPtr, FileAccess)" />
 	[Obsolete(
 		"This method has been deprecated. Please use New(SafeFileHandle, FileAccess) instead. http://go.microsoft.com/fwlink/?linkid=14202")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(IntPtr handle, FileAccess access)
 		=> throw new NotImplementedException();
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(IntPtr, FileAccess, bool)" />
 	[Obsolete(
 		"This method has been deprecated. Please use New(SafeFileHandle, FileAccess) instead. http://go.microsoft.com/fwlink/?linkid=14202")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(IntPtr handle, FileAccess access, bool ownsHandle)
 		=> throw new NotImplementedException();
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(IntPtr, FileAccess, bool, int)" />
 	[Obsolete(
 		"This method has been deprecated. Please use New(SafeFileHandle, FileAccess, int) instead. http://go.microsoft.com/fwlink/?linkid=14202")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(IntPtr handle, FileAccess access, bool ownsHandle, int bufferSize)
 		=> throw new NotImplementedException();
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(IntPtr, FileAccess, bool, int, bool)" />
 	[Obsolete(
 		"This method has been deprecated. Please use New(SafeFileHandle, FileAccess, int, bool) instead. http://go.microsoft.com/fwlink/?linkid=14202")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(IntPtr handle, FileAccess access, bool ownsHandle, int bufferSize,
 		bool isAsync)
 		=> throw new NotImplementedException();

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

@@ -24,16 +24,19 @@ public IFileSystem FileSystem
 
 	/// <inheritdoc cref="IFileSystemWatcherFactory.CreateNew()" />
 	[Obsolete("Use `IFileSystemWatcherFactory.New()` instead")]
+	[ExcludeFromCodeCoverage]
 	public IFileSystemWatcher CreateNew()
 		=> New();
 
 	/// <inheritdoc cref="IFileSystemWatcherFactory.CreateNew(string)" />
 	[Obsolete("Use `IFileSystemWatcherFactory.New(string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IFileSystemWatcher CreateNew(string path)
 		=> New(path);
 
 	/// <inheritdoc cref="IFileSystemWatcherFactory.CreateNew(string, string)" />
 	[Obsolete("Use `IFileSystemWatcherFactory.New(string, string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IFileSystemWatcher CreateNew(string path, string filter)
 		=> New(path, filter);
 

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

@@ -304,6 +304,7 @@ public string GetRelativePath(string relativeTo, string path)
 #if !NETSTANDARD2_0
 	[Obsolete(
 		"Insecure temporary file creation methods should not be used. Use `Path.Combine(Path.GetTempPath(), Path.GetRandomFileName())` instead.")]
+	[ExcludeFromCodeCoverage]
 #endif
 	public string GetTempFileName()
 	{

Source/Testably.Abstractions/FileSystem/DirectoryInfoFactory.cs

@@ -18,6 +18,7 @@ internal DirectoryInfoFactory(RealFileSystem fileSystem)
 
 	/// <inheritdoc cref="IDirectoryInfoFactory.FromDirectoryName(string)" />
 	[Obsolete("Use `IDirectoryInfoFactory.New(string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IDirectoryInfo FromDirectoryName(string directoryName)
 		=> New(directoryName);
 

Source/Testably.Abstractions/FileSystem/DriveInfoFactory.cs

@@ -19,6 +19,7 @@ internal DriveInfoFactory(RealFileSystem fileSystem)
 
 	/// <inheritdoc cref="IDriveInfoFactory.FromDriveName(string)" />
 	[Obsolete("Use `IDriveInfoFactory.New(string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IDriveInfo FromDriveName(string driveName)
 		=> New(driveName);
 

Source/Testably.Abstractions/FileSystem/FileInfoFactory.cs

@@ -18,6 +18,7 @@ internal FileInfoFactory(RealFileSystem fileSystem)
 
 	/// <inheritdoc cref="IFileInfoFactory.FromFileName(string)" />
 	[Obsolete("Use `IFileInfoFactory.New(string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IFileInfo FromFileName(string fileName)
 		=> New(fileName);
 

Source/Testably.Abstractions/FileSystem/FileStreamFactory.cs

@@ -1,9 +1,7 @@
 using Microsoft.Win32.SafeHandles;
 using System;
-using System.IO;
-#if NET6_0_OR_GREATER
 using System.Diagnostics.CodeAnalysis;
-#endif
+using System.IO;
 
 namespace Testably.Abstractions.FileSystem;
 
@@ -21,21 +19,25 @@ internal FileStreamFactory(RealFileSystem fileSystem)
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode)" />
 	[Obsolete("Use `IFileStreamFactory.New(string, FileMode)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(string path, FileMode mode)
 		=> New(path, mode);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess)" />
 	[Obsolete("Use `IFileStreamFactory.New(string, FileMode, FileAccess)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(string path, FileMode mode, FileAccess access)
 		=> New(path, mode, access);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess, FileShare)" />
 	[Obsolete("Use `IFileStreamFactory.New(string, FileMode, FileAccess, FileShare)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(string path, FileMode mode, FileAccess access, FileShare share)
 		=> New(path, mode, access, share);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess, FileShare, int)" />
 	[Obsolete("Use `IFileStreamFactory.New(string, FileMode, FileAccess, FileShare, int)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(
 		string path,
 		FileMode mode,
@@ -47,6 +49,7 @@ public Stream Create(
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess, FileShare, int, FileOptions)" />
 	[Obsolete(
 		"Use `IFileStreamFactory.New(string, FileMode, FileAccess, FileShare, int, FileOptions)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(
 		string path,
 		FileMode mode,
@@ -59,6 +62,7 @@ public Stream Create(
 	/// <inheritdoc cref="IFileStreamFactory.Create(string, FileMode, FileAccess, FileShare, int, bool)" />
 	[Obsolete(
 		"Use `IFileStreamFactory.New(string, FileMode, FileAccess, FileShare, int, bool)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(
 		string path,
 		FileMode mode,
@@ -70,40 +74,47 @@ public Stream Create(
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(SafeFileHandle, FileAccess)" />
 	[Obsolete("Use `IFileStreamFactory.New(SafeFileHandle, FileAccess)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(SafeFileHandle handle, FileAccess access)
 		=> New(handle, access);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(SafeFileHandle, FileAccess, int)" />
 	[Obsolete("Use `IFileStreamFactory.New(SafeFileHandle, FileAccess, int)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(SafeFileHandle handle, FileAccess access, int bufferSize)
 		=> New(handle, access, bufferSize);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(SafeFileHandle, FileAccess, int, bool)" />
 	[Obsolete("Use `IFileStreamFactory.New(SafeFileHandle, FileAccess, int, bool)` instead")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(SafeFileHandle handle, FileAccess access, int bufferSize, bool isAsync)
 		=> New(handle, access, bufferSize, isAsync);
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(IntPtr, FileAccess)" />
 	[Obsolete(
 		"This method has been deprecated. Please use New(SafeFileHandle, FileAccess) instead. http://go.microsoft.com/fwlink/?linkid=14202")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(IntPtr handle, FileAccess access)
 		=> throw new NotImplementedException();
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(IntPtr, FileAccess, bool)" />
 	[Obsolete(
 		"This method has been deprecated. Please use New(SafeFileHandle, FileAccess) instead. http://go.microsoft.com/fwlink/?linkid=14202")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(IntPtr handle, FileAccess access, bool ownsHandle)
 		=> throw new NotImplementedException();
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(IntPtr, FileAccess, bool, int)" />
 	[Obsolete(
 		"This method has been deprecated. Please use New(SafeFileHandle, FileAccess, int) instead. http://go.microsoft.com/fwlink/?linkid=14202")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(IntPtr handle, FileAccess access, bool ownsHandle, int bufferSize)
 		=> throw new NotImplementedException();
 
 	/// <inheritdoc cref="IFileStreamFactory.Create(IntPtr, FileAccess, bool, int, bool)" />
 	[Obsolete(
 		"This method has been deprecated. Please use New(SafeFileHandle, FileAccess, int, bool) instead. http://go.microsoft.com/fwlink/?linkid=14202")]
+	[ExcludeFromCodeCoverage]
 	public Stream Create(
 		IntPtr handle,
 		FileAccess access,

Source/Testably.Abstractions/FileSystem/FileSystemWatcherFactory.cs

@@ -18,16 +18,19 @@ internal FileSystemWatcherFactory(RealFileSystem fileSystem)
 
 	/// <inheritdoc cref="IFileSystemWatcherFactory.CreateNew()" />
 	[Obsolete("Use `IFileSystemWatcherFactory.New()` instead")]
+	[ExcludeFromCodeCoverage]
 	public IFileSystemWatcher CreateNew()
 		=> New();
 
 	/// <inheritdoc cref="IFileSystemWatcherFactory.CreateNew(string)" />
 	[Obsolete("Use `IFileSystemWatcherFactory.New(string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IFileSystemWatcher CreateNew(string path)
 		=> New(path);
 
 	/// <inheritdoc cref="IFileSystemWatcherFactory.CreateNew(string, string)" />
 	[Obsolete("Use `IFileSystemWatcherFactory.New(string, string)` instead")]
+	[ExcludeFromCodeCoverage]
 	public IFileSystemWatcher CreateNew(string path, string filter)
 		=> New(path, filter);
 

Source/Testably.Abstractions/FileSystem/PathWrapper.cs

@@ -160,6 +160,7 @@ public string GetRelativePath(string relativeTo, string path)
 #if !NETSTANDARD2_0
 	[Obsolete(
 		"Insecure temporary file creation methods should not be used. Use `Path.Combine(Path.GetTempPath(), Path.GetRandomFileName())` instead.")]
+	[ExcludeFromCodeCoverage]
 #endif
 	public string GetTempFileName()
 		=> Path.GetTempFileName();

Tests/Api/Testably.Abstractions.Api.Tests/Expected/Testably.Abstractions.Interface_net6.0.txt

@@ -24,6 +24,7 @@ namespace Testably.Abstractions.Helpers
         void StoreMetadata<T>(string key, T? value);
         bool TryGetWrappedInstance<T>([System.Diagnostics.CodeAnalysis.NotNullWhen(true)] out T? wrappedInstance);
     }
+    [System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
     [System.Obsolete("Will be removed in a future version!")]
     public abstract class PathSystemBase : System.IO.Abstractions.IFileSystemEntity, System.IO.Abstractions.IPath
     {

Tests/Api/Testably.Abstractions.Api.Tests/Expected/Testably.Abstractions.Interface_net7.0.txt

@@ -28,6 +28,7 @@ namespace Testably.Abstractions.Helpers
         void StoreMetadata<T>(string key, T? value);
         bool TryGetWrappedInstance<T>([System.Diagnostics.CodeAnalysis.NotNullWhen(true)] out T? wrappedInstance);
     }
+    [System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
     [System.Obsolete("Will be removed in a future version!")]
     public abstract class PathSystemBase : System.IO.Abstractions.IFileSystemEntity, System.IO.Abstractions.IPath
     {

Tests/Api/Testably.Abstractions.Api.Tests/Expected/Testably.Abstractions.Interface_net8.0.txt

@@ -28,6 +28,7 @@ namespace Testably.Abstractions.Helpers
         void StoreMetadata<T>(string key, T? value);
         bool TryGetWrappedInstance<T>([System.Diagnostics.CodeAnalysis.NotNullWhen(true)] out T? wrappedInstance);
     }
+    [System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
     [System.Obsolete("Will be removed in a future version!")]
     public abstract class PathSystemBase : System.IO.Abstractions.IFileSystemEntity, System.IO.Abstractions.IPath
     {

Tests/Api/Testably.Abstractions.Api.Tests/Expected/Testably.Abstractions.Interface_netstandard2.0.txt

@@ -16,6 +16,7 @@ namespace Testably.Abstractions.Helpers
         void StoreMetadata<T>(string key, T? value);
         bool TryGetWrappedInstance<T>([System.Diagnostics.CodeAnalysis.NotNullWhen(true)] out T? wrappedInstance);
     }
+    [System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
     [System.Obsolete]
     public abstract class PathSystemBase : System.IO.Abstractions.IFileSystemEntity, System.IO.Abstractions.IPath
     {

Tests/Api/Testably.Abstractions.Api.Tests/Expected/Testably.Abstractions.Interface_netstandard2.1.txt

@@ -24,6 +24,7 @@ namespace Testably.Abstractions.Helpers
         void StoreMetadata<T>(string key, T? value);
         bool TryGetWrappedInstance<T>([System.Diagnostics.CodeAnalysis.NotNullWhen(true)] out T? wrappedInstance);
     }
+    [System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
     [System.Obsolete("Will be removed in a future version!")]
     public abstract class PathSystemBase : System.IO.Abstractions.IFileSystemEntity, System.IO.Abstractions.IPath
     {