throw if argument is null

Author: smdn

src/Smdn.Reflection.ReverseGenerating.ListApi.Core/Smdn.Reflection.ReverseGenerating.ListApi/AssemblyLoader.cs

@@ -32,6 +32,9 @@ public static TResult UsingAssembly<TArg, TResult>(
     ILogger? logger = null
   )
   {
+    if (assemblyFile is null)
+       throw new ArgumentNullException(nameof(assemblyFile));
+
     return UsingAssemblyCore(
       new(
         ComponentAssemblyPath: assemblyFile.FullName,
@@ -61,7 +64,7 @@ public static TResult UsingAssembly<TArg, TResult>(
     return UsingAssemblyCore(
       new(
         ComponentAssemblyPath: componentAssemblyPath ?? throw new ArgumentNullException(nameof(componentAssemblyPath)),
-        Stream: assemblyStream
+        Stream: assemblyStream ?? throw new ArgumentNullException(nameof(assemblyStream))
       ),
       loadIntoReflectionOnlyContext,
       arg,

tests/Smdn.Reflection.ReverseGenerating.ListApi.Core/Smdn.Reflection.ReverseGenerating.ListApi/AssemblyLoader.cs

@@ -38,6 +38,59 @@ public void Init()
     logger = services.BuildServiceProvider().GetService<ILoggerFactory>()?.CreateLogger("test");
   }
 
+  [Test]
+  public void UsingAssembly_ArgumentNull_AssemblyFile(
+    [Values(true, false)] bool loadIntoReflectionOnlyContext
+  )
+  {
+    Assert.Throws<ArgumentNullException>(() => {
+      AssemblyLoader.UsingAssembly(
+        assemblyFile: null!,
+        loadIntoReflectionOnlyContext: loadIntoReflectionOnlyContext,
+        arg: 0,
+        (assm, arg) => arg,
+        context: out _,
+        logger: null
+      );
+    });
+  }
+
+  [Test]
+  public void UsingAssembly_ArgumentNull_AssemblyStream(
+    [Values(true, false)] bool loadIntoReflectionOnlyContext
+  )
+  {
+    Assert.Throws<ArgumentNullException>(() => {
+      AssemblyLoader.UsingAssembly(
+        assemblyStream: null!,
+        componentAssemblyPath: string.Empty,
+        loadIntoReflectionOnlyContext: loadIntoReflectionOnlyContext,
+        arg: 0,
+        (assm, arg) => arg,
+        context: out _,
+        logger: null
+      );
+    });
+  }
+
+  [Test]
+  public void UsingAssembly_ArgumentNull_ComponentAssemblyPath(
+    [Values(true, false)] bool loadIntoReflectionOnlyContext
+  )
+  {
+    Assert.Throws<ArgumentNullException>(() => {
+      AssemblyLoader.UsingAssembly(
+        assemblyStream: Stream.Null,
+        componentAssemblyPath: null!,
+        loadIntoReflectionOnlyContext: loadIntoReflectionOnlyContext,
+        arg: 0,
+        (assm, arg) => arg,
+        context: out _,
+        logger: null
+      );
+    });
+  }
+
 #if NETCOREAPP3_1_OR_GREATER || NET6_0_OR_GREATER
   [TestCase(true, "netstandard2.1")]
   [TestCase(false, "netstandard2.1")]