Revert "Fix get_rows_fieldnames_from_select() to return dictionary-style mappings"

This reverts commit 445092e.

Author: martinburchell

cardinal_pythonlib/sqlalchemy/core_query.py

@@ -147,7 +147,7 @@ def get_rows_fieldnames_from_select(
                 f"{select_query.column_descriptions}"
             )
 
-    result = session.execute(select_query).mappings()
+    result = session.execute(select_query)
 
     fieldnames_rmkview = result.keys()
     # ... of type RMKeyView, e.g. RMKeyView(['a', 'b'])

cardinal_pythonlib/sqlalchemy/tests/core_query_tests.py

@@ -117,8 +117,8 @@ def test_get_rows_fieldnames_from_select(self) -> None:
         rows, fieldnames = get_rows_fieldnames_from_select(self.session, query)
         self.assertEqual(fieldnames, [self.a, self.b])
         self.assertEqual(len(rows), 2)
-        self.assertEqual(rows[0], {"a": self.a_val1, "b": self.b_val1})
-        self.assertEqual(rows[1], {"a": self.a_val2, "b": self.b_val2})
+        self.assertEqual(rows[0], (self.a_val1, self.b_val1))
+        self.assertEqual(rows[1], (self.a_val2, self.b_val2))
 
     def test_count_star_and_max(self) -> None:
         count, maximum = count_star_and_max(

cardinal_pythonlib/sqlalchemy/tests/orm_query_tests.py

@@ -119,7 +119,7 @@ def test_get_rows_fieldnames_select_works(self) -> None:
         query = select(Pet.id, Pet.name).select_from(Pet.__table__)
         rows, fieldnames = get_rows_fieldnames_from_select(self.session, query)
         self.assertEqual(fieldnames, ["id", "name"])
-        self.assertEqual(rows, [{"id": 1, "name": self._pet_1_name}])
+        self.assertEqual(rows, [(1, self._pet_1_name)])
 
     def test_get_rows_fieldnames_whole_object_q_fails(self) -> None:
         # We want to disallow querying