Adds additional integration/unit tests (#34)

Author: samgibsonmoj

tests/Api.Tests/AggregateServiceTests.cs

@@ -304,11 +304,38 @@ private string[] CreateRecordWithNoAddresses()
         return contents;
     }
 
+    [Fact]
+    public async Task WriteAsync_UsesCrlfLineEndings()
+    {
+        // Arrange
+        var context = new AddressFieldsContext([]);
+        var writer = new AddressWriter(_testDirectory, context);
+        var contents = CreateFullRecordArray();
+
+        // Act
+        await writer.WriteAsync("NOMS001", contents);
+        await writer.WriteAsync("NOMS002", contents);
+        writer.Dispose();
+
+        // Assert
+        var outputFile = Path.Combine(_testDirectory, "Addresses.txt");
+        var fileContent = await File.ReadAllTextAsync(outputFile);
+        Assert.Contains("\r\n", fileContent);
+        Assert.DoesNotContain("\n", fileContent.Replace("\r\n", string.Empty));
+    }
+
     public void Dispose()
     {
         if (Directory.Exists(_testDirectory))
         {
-            Directory.Delete(_testDirectory, recursive: true);
+            try
+            {
+                Directory.Delete(_testDirectory, recursive: true);
+            }
+            catch
+            {
+                // Ignore cleanup errors
+            }
         }
     }
 }

tests/Api.Tests/SearchEndpointsTests.cs

@@ -90,12 +90,40 @@ public async Task WriteAsync_WithDuplicateAgencyCodes_WritesNonDuplicates()
         Assert.Contains(lines, l => l.Contains("AGY002"));
         Assert.DoesNotContain(lines, l => l.Contains("Duplicate"));
     }
+
+    [Fact]
+    public async Task WriteAsync_UsesCrlfLineEndings()
+    {
+        // Arrange
+        var writer = new AgenciesWriter(_testDirectory, []);
+        
+        var record1 = new[]{ string.Empty, "Agency One", "AGY001" };
+        var record2 = new[]{ string.Empty, "Agency Two", "AGY002" };
+
+        // Act
+        await writer.WriteAsync("NOMS001", record1);
+        await writer.WriteAsync("NOMS002", record2);
+        writer.Dispose();
+
+        // Assert
+        var outputFile = Path.Combine(_testDirectory, "Agencies.txt");
+        var fileContent = await File.ReadAllTextAsync(outputFile);
+        Assert.Contains("\r\n", fileContent);
+        Assert.DoesNotContain("\n", fileContent.Replace("\r\n", string.Empty));
+    }
 
     public void Dispose()
     {
         if (Directory.Exists(_testDirectory))
         {
-            Directory.Delete(_testDirectory, recursive: true);
+            try
+            {
+                Directory.Delete(_testDirectory, recursive: true);
+            }
+            catch
+            {
+                // Ignore cleanup errors
+            }
         }
     }
 }

tests/Api.Tests/SentenceInformationServiceTests.cs

@@ -75,7 +75,14 @@ public void Dispose()
     {
         if (Directory.Exists(_testDirectory))
         {
-            Directory.Delete(_testDirectory, recursive: true);
+            try
+            {
+                Directory.Delete(_testDirectory, recursive: true);
+            }
+            catch
+            {
+                // Ignore cleanup errors
+            }
         }
     }
 }