Merge branch 'release/6.0.0' into Caps

DbUp · Dec 17, 2024 · 279bb3f · 279bb3f
2 parents 26c1c55 + 73dd9a9
commit 279bb3f
Show file tree

Hide file tree

Showing 6 changed files with 32 additions and 6 deletions.
diff --git a/src/Sample/Program.cs b/src/Sample/Program.cs
@@ -1,4 +1,5 @@
 using System;
+using Microsoft.Data.Sqlite;
 
 namespace SqliteSampleApplication
 {
@@ -55,7 +56,7 @@ static void TemporaryFileDb()
 
         static void PermanentFileDb()
         {
-            Microsoft.Data.Sqlite.SqliteConnection connection = new("Data Source=dbup.db");
+            SqliteConnection connection = new("Data Source=dbup.db");
 
             using (var database = new DbUp.Sqlite.Helpers.SharedConnection(connection))
             {

diff --git a/src/Sample/Sample.csproj b/src/Sample/Sample.csproj
@@ -7,6 +7,7 @@
 
   <ItemGroup>
     <ProjectReference Include="..\dbup-sqlite\dbup-sqlite.csproj"/>
+    <PackageReference Include="Microsoft.Data.Sqlite" Version="8.0.7" />
   </ItemGroup>
 
   <ItemGroup>

diff --git a/src/Tests/SqliteSupportTests.cs b/src/Tests/SqliteSupportTests.cs
@@ -1,17 +1,16 @@
-using Microsoft.Data.Sqlite;
-using Shouldly;
+using Shouldly;
 using Xunit;
 
 namespace DbUp.Sqlite.Tests
 {
     public class SqliteSupportTests
     {
-        static readonly string dbFilePath = Path.Combine(Environment.CurrentDirectory, "test.db");
+        static readonly string DbFilePath = Path.Combine(Environment.CurrentDirectory, "test.db");
 
         [Fact]
         public void CanUseSQLite()
         {
-            var connectionString = $"Data Source={dbFilePath}";
+            var connectionString = $"Data Source={DbFilePath}";
 
             var upgrader = DeployChanges.To
                 .SqliteDatabase(connectionString)
@@ -23,5 +22,28 @@ public void CanUseSQLite()
             result.Error.ShouldBe(null);
             result.Successful.ShouldBe(true);
         }
+
+        /// <summary>
+        /// Test for https://github.com/DbUp/dbup-sqlite/issues/2
+        /// </summary>
+        [Fact]
+        public void DoesNotExhibitSafeHandleError()
+        {
+            var connectionString = "Data source=:memory:";
+
+            var upgrader =
+                DeployChanges.To
+                    .SQLiteDatabase(connectionString)
+                    .WithScript("Script001", @"
+create table test (
+    contact_id INTEGER PRIMARY KEY
+);
+")
+                    .LogScriptOutput()
+                    .LogToConsole()
+                    .Build();
+            var result = upgrader.PerformUpgrade();
+            result.Successful.ShouldBeTrue();
+        }
     }
 }
diff --git a/src/Tests/Tests.csproj b/src/Tests/Tests.csproj
@@ -10,6 +10,7 @@
 
   <ItemGroup>
     <ProjectReference Include="..\dbup-sqlite\dbup-sqlite.csproj"/>
+    <PackageReference Include="Microsoft.Data.Sqlite" Version="8.0.7" />
     <PackageReference Include="DbUp.Tests.Common" Version="6.0.0-beta.146"/>
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0"/>
     <PackageReference Include="xunit" Version="2.9.0" />

diff --git a/src/dbup-sqlite/Helpers/InMemorySqliteDatabase.cs b/src/dbup-sqlite/Helpers/InMemorySqliteDatabase.cs
@@ -23,6 +23,7 @@ public InMemorySqliteDatabase()
             {
                 DataSource = ":memory:",
                 DefaultTimeout = 5,
+                Mode = SqliteOpenMode.Memory
             };
             ConnectionString = connectionStringBuilder.ToString();
 

diff --git a/src/dbup-sqlite/dbup-sqlite.csproj b/src/dbup-sqlite/dbup-sqlite.csproj
@@ -24,7 +24,7 @@
 
   <ItemGroup>
     <PackageReference Include="dbup-core" Version="6.0.0-beta.146"/>
-    <PackageReference Include="Microsoft.Data.Sqlite" Version="8.0.7" />
+    <PackageReference Include="Microsoft.Data.Sqlite.Core" Version="8.0.7" />
   </ItemGroup>
 
   <ItemGroup>