Made IDatabaseClient implement the IDisposable interface (#30)

Author: Sem1712

Demo/Program.cs

@@ -1,7 +1,7 @@
 using Libsql.Client;
 
 // Create a database client using the static factory method
-var dbClient = await DatabaseClient.Create(opts => {
+using var dbClient = await DatabaseClient.Create(opts => {
     opts.Url = ":memory:";
 });
 

Libsql.Client.Tests/EmbeddedReplicaTests.cs

@@ -2,7 +2,7 @@
 
 namespace Libsql.Client.Tests;
 
-public class EmbeddedReplicaTests : IClassFixture<DatabaseContainer>
+public class EmbeddedReplicaTests : IClassFixture<DatabaseContainer>, IDisposable
 {
     public IDatabaseClient DatabaseClient { get; }
 
@@ -38,4 +38,10 @@ public async Task CanCallSync()
     {
         await DatabaseClient.Sync();
     }
+
+    public void Dispose()
+    {
+        DatabaseClient.Dispose();
+        GC.SuppressFinalize(this);
+    }
 }

Libsql.Client.Tests/PositionalArgumentTests.cs

@@ -1,6 +1,6 @@
 namespace Libsql.Client.Tests;
 
-public class PositionalArgumentTests
+public class PositionalArgumentTests : IDisposable
 {
     private readonly IDatabaseClient _db = DatabaseClient.Create().Result;
 
@@ -86,4 +86,10 @@ public async Task BindBlobParameter()
 
         Assert.Equal(new byte[] { 1, 2, 3 }, blob.Value);
     }
+
+    public void Dispose()
+    {
+        _db.Dispose();
+        GC.SuppressFinalize(this);
+    }
 }

Libsql.Client.Tests/RemoteTests.cs

@@ -2,7 +2,7 @@
 
 namespace Libsql.Client.Tests;
 
-public class RemoteTests : IClassFixture<DatabaseContainer>
+public class RemoteTests : IClassFixture<DatabaseContainer>, IDisposable
 {
     public IDatabaseClient DatabaseClient { get; }
 
@@ -28,4 +28,10 @@ public async Task CanConnectAndQueryRemoteDatabase()
         var value = Assert.IsType<Integer>(count);
         Assert.Equal(3503, value.Value);
     }
+
+    public void Dispose()
+    {
+        DatabaseClient.Dispose();
+        GC.SuppressFinalize(this);
+    }
 }

Libsql.Client.Tests/ResultSetTests.cs

@@ -1,6 +1,6 @@
 namespace Libsql.Client.Tests;
 
-public class ResultSetTests
+public class ResultSetTests : IDisposable
 {
     private readonly IDatabaseClient _db = DatabaseClient.Create().Result;
 
@@ -90,4 +90,10 @@ public async Task Changes_ReturnsExectedValue_WhenMultipleUpdates()
 
         Assert.Equal(10ul, rs2.RowsAffected);
     }
+
+    public void Dispose()
+    {
+        _db.Dispose();
+        GC.SuppressFinalize(this);
+    }
 }

Libsql.Client.Tests/RowsTests.cs

@@ -1,6 +1,6 @@
 namespace Libsql.Client.Tests;
 
-public class RowsTests
+public class RowsTests : IDisposable
 {
     private readonly IDatabaseClient _db = DatabaseClient.Create().Result;
 
@@ -38,4 +38,10 @@ public async Task Rows_CanPartiallyIterateTwice()
         var expected = new []{new Integer(1), new Integer(2), new Integer(3)};
         Assert.Equal(expected, secondArray.Select(x => x.First() as Integer).ToArray());
     }
+
+    public void Dispose()
+    {
+        _db.Dispose();
+        GC.SuppressFinalize(this);
+    }
 }

Libsql.Client.Tests/SelectTests.cs

@@ -1,6 +1,6 @@
 namespace Libsql.Client.Tests;
 
-public class SelectTests
+public class SelectTests : IDisposable
 {
     private readonly IDatabaseClient _db = DatabaseClient.Create().Result;
 
@@ -118,4 +118,10 @@ public async Task SelectNullType()
         var value = row.First();
         Assert.IsType<Null>(value);
     }
+
+    public void Dispose()
+    {
+        _db.Dispose();
+        GC.SuppressFinalize(this);
+    }
 }

Libsql.Client/DatabaseWrapper.cs

@@ -6,7 +6,7 @@
 
 namespace Libsql.Client
 {
-    internal class DatabaseWrapper : IDatabaseClient, IDisposable
+    internal class DatabaseWrapper : IDatabaseClient
     {
         private libsql_database_t _db;
         private libsql_connection_t _connection;

Libsql.Client/IDatabaseClient.cs

@@ -1,11 +1,12 @@
+using System;
 using System.Threading.Tasks;
 
 namespace Libsql.Client
 {
     /// <summary>
     /// Interface for a Libsql database client.
     /// </summary>
-    public interface IDatabaseClient
+    public interface IDatabaseClient : IDisposable
     {
         /// <summary>
         /// Executes the given SQL query and returns the result set.

README.md

@@ -31,7 +31,7 @@ For an example, see the Demo project in the repository.
 
 ```csharp
 // Create an in-memory database.
-var dbClient = DatabaseClient.Create(opts => {
+var dbClient = await DatabaseClient.Create(opts => {
     opts.Url = ":memory:";
 });
 ```
@@ -80,7 +80,7 @@ dbClient.Dispose();
 or with a `using` statement:
 
 ```csharp
-using (var dbClient = DatabaseClient.Create(opts => {
+using (var dbClient = await DatabaseClient.Create(opts => {
     opts.Url = ":memory:";
 }))
 {