Refactor all test classes to implement IDisposable and ensure proper resource cleanup. (#124)

Author: nik-solcast

tests/Solcast.Tests/AggregationClientTests.cs

@@ -1,39 +1,50 @@
-using NUnit.Framework;
-using Solcast.Clients;
-using System.Threading.Tasks;
-
-namespace Solcast.Tests
-{
-    [TestFixture]
-    public class AggregationClientTests
-    {
-        private AggregationClient _aggregationClient;
-
-        [SetUp]
-        public void Setup()
-        {
-            _aggregationClient = new AggregationClient();
-        }
-
-        [Test]
-        public async Task GetLiveAggregation_ShouldReturnValidData()
-        {
-            var response = await _aggregationClient.GetLiveAggregations(
-                collectionId: "country_total",
-                aggregationId: "it_total"
-            );
-            Assert.IsNotNull(response);
-        }
-
-        [Test]
-        public async Task GetForecastAggregation_ShouldReturnValidData()
-        {
-            var response = await _aggregationClient.GetForecastAggregations(
-                collectionId: "country_total",
-                aggregationId: "it_total",
-                outputParameters: ["percentage", "pv_estimate"]
-            );
-            Assert.IsNotNull(response);
-        }
-    }
-}
+using NUnit.Framework;
+using Solcast.Clients;
+using System;
+using System.Threading.Tasks;
+
+namespace Solcast.Tests
+{
+    [TestFixture]
+    public class AggregationClientTests : IDisposable
+    {
+        private AggregationClient _aggregationClient;
+        private bool _disposed = false;
+
+        [SetUp]
+        public void Setup()
+        {
+            _aggregationClient = new AggregationClient();
+        }
+
+        [Test]
+        public async Task GetLiveAggregation_ShouldReturnValidData()
+        {
+            var response = await _aggregationClient.GetLiveAggregations(
+                collectionId: "country_total",
+                aggregationId: "it_total"
+            );
+            Assert.IsNotNull(response);
+        }
+
+        [Test]
+        public async Task GetForecastAggregation_ShouldReturnValidData()
+        {
+            var response = await _aggregationClient.GetForecastAggregations(
+                collectionId: "country_total",
+                aggregationId: "it_total",
+                outputParameters: ["percentage", "pv_estimate"]
+            );
+            Assert.IsNotNull(response);
+        }
+
+        public void Dispose()
+        {
+            if (!_disposed)
+            {
+                _aggregationClient?.Dispose();
+                _disposed = true;
+            }
+        }
+    }
+}

tests/Solcast.Tests/ForecastClientTests.cs

@@ -7,10 +7,11 @@
 namespace Solcast.Tests
 {
     [TestFixture]
-    public class ForecastClientTests
+    public class ForecastClientTests : IDisposable
     {
         private ForecastClient _forecastClient;
         private string _originalApiKey;
+        private bool _disposed = false;
 
         [OneTimeSetUp]
         public void OneTimeSetup()
@@ -100,5 +101,14 @@ public void OneTimeTearDown()
             // Restore the original API key after all tests
             Environment.SetEnvironmentVariable("SOLCAST_API_KEY", _originalApiKey);
         }
+
+        public void Dispose()
+        {
+            if (!_disposed)
+            {
+                _forecastClient?.Dispose();
+                _disposed = true;
+            }
+        }
     }
 }