test: add on_exit cleanup hook to cursor_streaming_large_test setup and fix formatting

Author: ocean

native/ecto_libsql/src/tests/error_handling_tests.rs

@@ -482,9 +482,10 @@ async fn test_prepared_statement_with_parameter_mismatch() {
 // DATABASE FILE ERROR TESTS
 // ============================================================================
 
+#[cfg(unix)]
 #[tokio::test]
 async fn test_create_db_invalid_permissions() {
-    // Test with path that's definitely invalid
+    // Test with path that's definitely invalid (Unix-specific: null bytes)
     let invalid_path = "\0invalid\0path.db"; // Null bytes in path
 
     // Creating DB with invalid path should error, not panic
@@ -495,6 +496,20 @@ async fn test_create_db_invalid_permissions() {
     let _ = result;
 }
 
+#[cfg(windows)]
+#[tokio::test]
+async fn test_create_db_invalid_permissions() {
+    // Test with path that's definitely invalid (Windows-specific: invalid characters)
+    let invalid_path = "COM1"; // Reserved device name on Windows
+
+    // Creating DB with invalid path should error, not panic
+    let result = Builder::new_local(invalid_path).build().await;
+
+    // This should error due to invalid path, or succeed silently
+    // The key is it doesn't panic
+    let _ = result;
+}
+
 #[tokio::test]
 async fn test_readonly_database_insert() {
     let db_path = setup_test_db();

test/connection_recovery_test.exs

@@ -7,6 +7,11 @@ defmodule EctoLibSql.ConnectionRecoveryTest do
 
   setup do
     {:ok, state} = EctoLibSql.connect(database: ":memory:")
+
+    on_exit(fn ->
+      EctoLibSql.disconnect([], state)
+    end)
+
     {:ok, state: state}
   end