update test coverage

coenm · Sep 25, 2024 · 02c0fa1 · 02c0fa1
1 parent 19415db
commit 02c0fa1
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/src/RepoM.App/Services/ConfigBasedAppDataPathProviderFactory.cs b/src/RepoM.App/Services/ConfigBasedAppDataPathProviderFactory.cs
@@ -22,7 +22,7 @@ public ConfigBasedAppDataPathProviderFactory(string[] args, IFileSystem fileSyst
     public ConfigBasedAppDataPathProviderFactory(string[] args, IFileSystem fileSystem, IFileProvider fileProvider)
         :this(args, fileSystem)
     {
-        _fileProvider = fileProvider;
+        _fileProvider = fileProvider ?? throw new ArgumentNullException(nameof(fileProvider));
     }
 
     public AppDataPathProvider Create()

diff --git a/tests/RepoM.App.Tests/ConfigBasedAppDataPathProviderFactoryTest.cs b/tests/RepoM.App.Tests/ConfigBasedAppDataPathProviderFactoryTest.cs
@@ -41,6 +41,26 @@ public ConfigBasedAppDataPathProviderFactoryTest()
         SetupGetFullPath();
     }
 
+    [Fact]
+    public void Ctor_ShouldThrow_WhenArgumentNull()
+    {
+        // arrange
+
+        // act
+        Func<ConfigBasedAppDataPathProviderFactory> act1 = () => new ConfigBasedAppDataPathProviderFactory([], null!);
+        Func<ConfigBasedAppDataPathProviderFactory> act2 = () => new ConfigBasedAppDataPathProviderFactory(null!, A.Dummy<IFileSystem>());
+        Func<ConfigBasedAppDataPathProviderFactory> act3 = () => new ConfigBasedAppDataPathProviderFactory([], A.Dummy<IFileSystem>(), null!);
+        Func<ConfigBasedAppDataPathProviderFactory> act4 = () => new ConfigBasedAppDataPathProviderFactory([], null!, A.Dummy<IFileProvider>());
+        Func<ConfigBasedAppDataPathProviderFactory> act5 = () => new ConfigBasedAppDataPathProviderFactory(null!, A.Dummy<IFileSystem>(), A.Dummy<IFileProvider>());
+
+        // assert
+        act1.Should().Throw<ArgumentNullException>();
+        act2.Should().Throw<ArgumentNullException>();
+        act3.Should().Throw<ArgumentNullException>();
+        act4.Should().Throw<ArgumentNullException>();
+        act5.Should().Throw<ArgumentNullException>();
+    }
+
     [Fact]
     public void Create_ShouldSetPathToRelativeFolder_WhenAppSettingsJsonIsSet()
     {