Misc refactors and cleanup.

Author: sharpjs

Directory.Build.targets

@@ -22,8 +22,8 @@
 
   <!-- Visibility -->
   <ItemGroup>
-    <InternalsVisibleTo Include="PSql.Deploy.Tests" />
     <InternalsVisibleTo Include="PSql.Deploy.Engine.Tests" />
+    <InternalsVisibleTo Include="PSql.Deploy.Tests" />
     <InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
     <!-- Required for Moq to mock a class with an internal abstract method. -->
   </ItemGroup>

PSql.Deploy.Engine.Tests/Utilities/StringExtensionsTests.cs

@@ -18,7 +18,6 @@ public void IsNullOrEmpty(string? s, bool expected)
     [Test]
     [TestCase(null, null)]
     [TestCase("",   null)]
-    [TestCase(" ",  " " )]
     [TestCase("a",  "a" )]
     public void NullIfEmpty(string? s, string? expected)
     {

PSql.Deploy.Engine/Migrations/MigrationLoader.cs

@@ -138,8 +138,8 @@ private static void AppendFinalBatches(
         if (migration.IsPseudo)
             return;
 
-        var name = migration.Name.Replace("'", "''");
-        var hash = migration.Hash.Replace("'", "''");
+        var name = migration.Name.EscapeForSqlString();
+        var hash = migration.Hash.EscapeForSqlString();
 
         builder.StartNewBatch();
         builder.Append(

PSql.Deploy/Data/SqlTargetDatabase.cs

@@ -2,7 +2,6 @@
 // SPDX-License-Identifier: MIT
 
 using System.Diagnostics;
-using System.Net;
 
 using static System.Reflection.BindingFlags;
 

PSql.Deploy/PSql.Deploy.psd1.t

@@ -15,8 +15,8 @@
     # Requirements
     CompatiblePSEditions = 'Core'
     PowerShellVersion    = '7.2'
-    RequiredModules      = @() # Designed to work with PSql, but can work independently
-    RequiredAssemblies   = @()
+    #RequiredModules     = @(...) # Designed to work with PSql, but can work independently
+    #RequiredAssemblies  = @(...)
 
     # Initialization
     #ScriptsToProcess = @(...)