(#10) Test for edges where the developer is being obtuse

Author: colinangusmackay

src/Stravaig.Configuration.SqlServer.Tests/SourceBuilderTests.cs

@@ -9,6 +9,44 @@ namespace Stravaig.Configuration.SqlServer.Tests;
 [TestFixture]
 public class SourceBuilderTests
 {
+    [Test]
+    public void DeveloperIsBeingObtuseByDeliberatelyNullingSchema_ThrowsException()
+    {
+        // Arrange
+        const string theConnectionString = "Server=MyServer;Database=MyDatabase";
+        var options = (SqlServerConfigurationOptions opts) =>
+        {
+            opts.ConnectionString = theConnectionString;
+            opts.SchemaName = null!;
+        };
+        var configBuilder = SetupConfig();
+
+        // Act & Assert
+        var ex = Should.Throw<SqlServerConfigurationProviderException>(
+            () => SourceBuilder.BuildSource(configBuilder, options));
+
+        ex.Message.ShouldBe("The schema name is required to use SQL Server Configuration.");
+    }
+
+    [Test]
+    public void DeveloperIsBeingObtuseByDeliberatelyNullingTable_ThrowsException()
+    {
+        // Arrange
+        const string theConnectionString = "Server=MyServer;Database=MyDatabase";
+        var options = (SqlServerConfigurationOptions opts) =>
+        {
+            opts.ConnectionString = theConnectionString;
+            opts.TableName = null!;
+        };
+        var configBuilder = SetupConfig();
+
+        // Act & Assert
+        var ex = Should.Throw<SqlServerConfigurationProviderException>(
+            () => SourceBuilder.BuildSource(configBuilder, options));
+
+        ex.Message.ShouldBe("The table name is required to use SQL Server Configuration.");
+    }
+
     [Test]
     public void SimplestOptions_HappyPath()
     {

src/Stravaig.Configuration.SqlServer.sln.DotSettings

@@ -1,2 +1,3 @@
-<wpf:ResourceDictionary xml:space="preserve" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:s="clr-namespace:System;assembly=mscorlib" xmlns:ss="urn:shemas-jetbrains-com:settings-storage-xaml" xmlns:wpf="http://schemas.microsoft.com/winfx/2006/xaml/presentation">
+<wpf:ResourceDictionary xml:space="preserve" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:s="clr-namespace:System;assembly=mscorlib" xmlns:ss="urn:shemas-jetbrains-com:settings-storage-xaml" xmlns:wpf="http://schemas.microsoft.com/winfx/2006/xaml/presentation">
+	<s:Boolean x:Key="/Default/UserDictionary/Words/=Nulling/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=Stravaig/@EntryIndexedValue">True</s:Boolean></wpf:ResourceDictionary>

src/Stravaig.Configuration.SqlServer/SourceBuilder.cs

@@ -23,8 +23,8 @@ public static SqlServerConfigurationSource BuildSource(IConfigurationBuilder bui
         return new SqlServerConfigurationSource(
             options.ConnectionString ?? throw new SqlServerConfigurationProviderException("Cannot build a SQL Server Configuration Provider without a connection string."),
             TimeSpan.FromSeconds(options.RefreshSeconds),
-            options.SchemaName,
-            options.TableName);
+            options.SchemaName ?? throw new SqlServerConfigurationProviderException("The schema name is required to use SQL Server Configuration."),
+            options.TableName ?? throw new SqlServerConfigurationProviderException("The table name is required to use SQL Server Configuration."));
     }
 
     private static void ApplyFromConnectionStringsSection(SqlServerConfigurationOptions options, IConfigurationRoot configRoot)