pr-fix: add null checks w/o guard

arcus-azure · Dec 13, 2024 · 751d7f8 · 751d7f8
1 parent 3dcc793
commit 751d7f8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/src/Arcus.Testing.Tests.Integration/Core/Fixture/TemporaryFile.cs b/src/Arcus.Testing.Tests.Integration/Core/Fixture/TemporaryFile.cs
@@ -15,6 +15,8 @@ internal class TemporaryFile : IDisposable
 
         private TemporaryFile(FileInfo file, byte[] fileContents)
         {
+            ArgumentNullException.ThrowIfNull(file);
+
             _file = file;
             Contents = fileContents;
         }
@@ -39,6 +41,8 @@ private TemporaryFile(FileInfo file, byte[] fileContents)
         /// </summary>
         public static TemporaryFile GenerateAt(DirectoryInfo directory)
         {
+            ArgumentNullException.ThrowIfNull(directory);
+
             string fileName = Bogus.System.FileName();
             byte[] fileContents = Bogus.Random.Bytes(Bogus.Random.Int(10, 20));
 
@@ -50,6 +54,8 @@ public static TemporaryFile GenerateAt(DirectoryInfo directory)
         /// </summary>
         public static TemporaryFile CreateAt(DirectoryInfo directory, string fileName, byte[] fileContents)
         {
+            ArgumentNullException.ThrowIfNull(directory);
+
             string filePath = Path.Combine(directory.FullName, fileName);
             File.WriteAllBytes(filePath, fileContents);
 

diff --git a/src/Arcus.Testing.Tests.Integration/Core/Fixture/TemporaryFileEdit.cs b/src/Arcus.Testing.Tests.Integration/Core/Fixture/TemporaryFileEdit.cs
@@ -13,6 +13,8 @@ internal class TemporaryFileEdit : IDisposable
 
         private TemporaryFileEdit(FileInfo file, string originalContents)
         {
+            ArgumentNullException.ThrowIfNull(file);
+
             _file = file;
             _originalContents = originalContents;
         }