From 751d7f8c88ccaabd1f473b48d88c35c4d18033ed Mon Sep 17 00:00:00 2001 From: stijnmoreels <9039753+stijnmoreels@users.noreply.github.com> Date: Fri, 13 Dec 2024 09:46:27 +0100 Subject: [PATCH] pr-fix: add null checks w/o guard --- .../Core/Fixture/TemporaryFile.cs | 6 ++++++ .../Core/Fixture/TemporaryFileEdit.cs | 2 ++ 2 files changed, 8 insertions(+) diff --git a/src/Arcus.Testing.Tests.Integration/Core/Fixture/TemporaryFile.cs b/src/Arcus.Testing.Tests.Integration/Core/Fixture/TemporaryFile.cs index 443f72ab..ea1d5d4f 100644 --- 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) /// 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) /// 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 index 1d991bad..145ccf1e 100644 --- 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; }