Use conn_name_attr for SqliteHook connection (#10156)

The DbApiHook allows for a conn_name_attr to be changed in subclasses,
however SqliteHook's `get_conn` method is always calling the main class
attribute. Find the correct attribute and use this to establish the
connection.

Allow attr setting outside init for test case

Closes #10147

airflow/providers/sqlite/hooks/sqlite.py

@@ -28,12 +28,12 @@ class SqliteHook(DbApiHook):
 
     conn_name_attr = 'sqlite_conn_id'
     default_conn_name = 'sqlite_default'
-    supports_autocommit = False
 
     def get_conn(self):
         """
         Returns a sqlite connection object
         """
-        conn = self.get_connection(self.sqlite_conn_id)  # pylint: disable=no-member
-        conn = sqlite3.connect(conn.host)
+        conn_id = getattr(self, self.conn_name_attr)
+        airflow_conn = self.get_connection(conn_id)
+        conn = sqlite3.connect(airflow_conn.host)
         return conn

tests/providers/sqlite/hooks/test_sqlite.py

@@ -32,7 +32,7 @@ def setUp(self):
         self.connection = Connection(host='host')
 
         class UnitTestSqliteHook(SqliteHook):
-            conn_name_attr = 'sqlite_conn_id'
+            conn_name_attr = 'test_conn_id'
 
         self.db_hook = UnitTestSqliteHook()
         self.db_hook.get_connection = mock.Mock()
@@ -43,6 +43,13 @@ def test_get_conn(self, mock_connect):
         self.db_hook.get_conn()
         mock_connect.assert_called_once_with('host')
 
+    @patch('airflow.providers.sqlite.hooks.sqlite.sqlite3.connect')
+    def test_get_conn_non_default_id(self, mock_connect):
+        self.db_hook.test_conn_id = 'non_default'  # pylint: disable=attribute-defined-outside-init
+        self.db_hook.get_conn()
+        mock_connect.assert_called_once_with('host')
+        self.db_hook.get_connection.assert_called_once_with('non_default')
+
 
 class TestSqliteHook(unittest.TestCase):