fix: add / separator before Windows paths in inet:// protocol URLs; fix test assertion

Author: fdcastel

src/firebird/driver/core.py

@@ -2180,7 +2180,7 @@ def _connect_helper(dsn: str, host: str, port: str, database: str, protocol: Net
                     # Unix absolute path - use double slash so Firebird keeps the leading /
                     dsn += f'/{database}'  # Results in inet://host//absolute/path
                 elif ':' in database:  # Windows path (e.g., C:\...)
-                    dsn += database  # Concatenate directly without separator
+                    dsn += f'/{database}'  # Results in inet://host:port/D:\path
                 else:  # Relative/alias
                     dsn += f'/{database}'
             else:

tests/test_connection.py

@@ -162,21 +162,22 @@ def test_connect_config(fb_vars, db_file, driver_cfg):
 
     if host:
         # protocols
-        # For protocol URLs with absolute paths, we need double slash to preserve leading /
-        # inet://host//absolute/path so Firebird doesn't strip the leading /
+        # For protocol URLs the path always needs a / separator:
+        # - Unix: inet://host//absolute/path (double slash to keep the leading /)
+        # - Windows: inet://host:port/D:\path (single slash before drive letter)
         if str(db_file).startswith('/'):
-            dsn = f'{host}:{port}/{db_file}'  # Extra / for absolute paths
+            proto_path = f'{host}:{port}/{db_file}'  # Extra / for Unix absolute paths
         else:
-            dsn = f'{host}:{port}{db_file}'
+            proto_path = f'{host}:{port}/{db_file}'  # Single / for Windows drive-letter paths
         cfg = driver_cfg.get_database('test_db1')
         cfg.protocol.value = NetProtocol.INET
         with connect('test_db1') as con:
             assert con._att is not None
-            assert con.dsn == f'inet://{dsn}'
+            assert con.dsn == f'inet://{proto_path}'
         cfg.protocol.value = NetProtocol.INET4
         with connect('test_db1') as con:
             assert con._att is not None
-            assert con.dsn == f'inet4://{dsn}'
+            assert con.dsn == f'inet4://{proto_path}'
 
 def test_properties(db_connection):
     con = db_connection # Use the fixture