Extend exception mapping to all connection methods

Address reviewer feedback: wrap remaining connection methods that call C++
to ensure complete SQLSTATE error mapping coverage.

Additional methods now wrapped with try-catch + _raise_connection_error:
- autocommit property getter (get_autocommit)
- set_attr() for connection attributes
- getinfo() for connection information
- close() internal rollback during cleanup

This ensures users never see raw RuntimeError with SQLSTATE prefixes
from any connection operation. All C++ checkError() calls now properly
map to correct DB-API 2.0 exception types.

Addresses review comment on PR #562

Author: gargsaumya

mssql_python/connection.py

@@ -496,7 +496,10 @@ def autocommit(self) -> bool:
         Returns:
             bool: True if autocommit is enabled, False otherwise.
         """
-        return self._conn.get_autocommit()
+        try:
+            return self._conn.get_autocommit()
+        except RuntimeError as e:
+            _raise_connection_error(e)
 
     @autocommit.setter
     def autocommit(self, value: bool) -> None:
@@ -928,6 +931,9 @@ def set_attr(self, attribute: int, value: Union[int, str, bytes, bytearray]) ->
             self._conn.set_attr(attribute, value)
             logger.info(f"Connection attribute {sanitized_attr} set successfully")
 
+        except RuntimeError as e:
+            # Handle C++ layer RuntimeError with proper DB-API exception mapping
+            _raise_connection_error(e)
         except Exception as e:
             error_msg = f"Failed to set connection attribute {sanitized_attr}: {str(e)}"
 
@@ -1308,6 +1314,9 @@ def getinfo(self, info_type: int) -> Union[str, int, bool, None]:
         # Get the raw result from the C++ layer
         try:
             raw_result = self._conn.get_info(info_type)
+        except RuntimeError as e:
+            # Handle C++ layer RuntimeError with proper DB-API exception mapping
+            _raise_connection_error(e)
         except Exception as e:  # pylint: disable=broad-exception-caught
             # Log the error and return None for invalid info types
             logger.warning(f"getinfo({info_type}) failed: {e}")
@@ -1619,7 +1628,11 @@ def close(self) -> None:
                     # For autocommit True, this is not necessary as each statement is
                     # committed immediately
                     logger.debug("Rolling back uncommitted changes before closing connection.")
-                    self._conn.rollback()
+                    try:
+                        self._conn.rollback()
+                    except RuntimeError as e:
+                        # Handle C++ layer RuntimeError with proper DB-API exception mapping
+                        _raise_connection_error(e)
                 # TODO: Check potential race conditions in case of multithreaded scenarios
                 # Close the connection
                 self._conn.close()