refactor: analyze brittle tests (#348)

On MacOS the `GetCurrentDirectory` sometimes throws an exception. Catch this exception in the constructor of the MockFileSystem to avoid brittle tests. --------- Co-authored-by: Valentin Breuß <[email protected]>
Testably · Aug 5, 2023 · e5a16b7 · e5a16b7
1 parent af033f8
commit e5a16b7
Showing 1 changed file with 13 additions and 4 deletions.
diff --git a/Source/Testably.Abstractions.Testing/MockFileSystem.cs b/Source/Testably.Abstractions.Testing/MockFileSystem.cs
@@ -1,6 +1,7 @@
 using Microsoft.Win32.SafeHandles;
 using System;
 using System.Collections.Generic;
+using System.IO;
 using Testably.Abstractions.Testing.FileSystem;
 using Testably.Abstractions.Testing.Storage;
 
@@ -165,11 +166,19 @@ public MockFileSystem WithSafeFileHandleStrategy(
 
 	private void AddDriveFromCurrentDirectory()
 	{
-		string? root = Path.GetPathRoot(System.IO.Directory.GetCurrentDirectory());
-		if (root != null &&
-		    root[0] != _storage.MainDrive.Name[0])
+		try
 		{
-			Storage.GetOrAddDrive(root);
+			string? root = Path.GetPathRoot(System.IO.Directory.GetCurrentDirectory());
+			if (root != null &&
+			    root[0] != _storage.MainDrive.Name[0])
+			{
+				Storage.GetOrAddDrive(root);
+			}
+		}
+		catch (IOException)
+		{
+			// Ignore any IOException, when trying to read the current directory
+			// due to brittle tests on MacOS
 		}
 	}
 }