chore: remove log spamming and skip flaky test

Author: olavloite

google/cloud/spanner_v1/database_sessions_manager.py

@@ -155,8 +155,6 @@ def _build_multiplexed_session(self) -> Session:
         )
         session.create()
 
-        self._database.logger.info("Created multiplexed session.")
-
         return session
 
     def _build_maintenance_thread(self) -> Thread:

tests/unit/test_database_session_manager.py

@@ -98,9 +98,8 @@ def test_read_only_multiplexed(self):
         pool.get.assert_not_called()
         pool.put.assert_not_called()
 
-        # Verify logger calls.
-        info = manager._database.logger.info
-        info.assert_called_once_with("Created multiplexed session.")
+        # Verify create_session was called.
+        manager._database.spanner_api.create_session.assert_called_once()
 
     def test_partitioned_pooled(self):
         manager = self._manager
@@ -137,9 +136,8 @@ def test_partitioned_multiplexed(self):
         pool.get.assert_not_called()
         pool.put.assert_not_called()
 
-        # Verify logger calls.
-        info = manager._database.logger.info
-        info.assert_called_once_with("Created multiplexed session.")
+        # Verify create_session was called.
+        manager._database.spanner_api.create_session.assert_called_once()
 
     def test_read_write_pooled(self):
         manager = self._manager
@@ -176,9 +174,8 @@ def test_read_write_multiplexed(self):
         pool.get.assert_not_called()
         pool.put.assert_not_called()
 
-        # Verify logger calls.
-        info = manager._database.logger.info
-        info.assert_called_once_with("Created multiplexed session.")
+        # Verify create_session was called.
+        manager._database.spanner_api.create_session.assert_called_once()
 
     def test_multiplexed_maintenance(self):
         manager = self._manager
@@ -199,10 +196,6 @@ def test_multiplexed_maintenance(self):
         self.assertTrue(session_2.is_multiplexed)
         self.assertNotEqual(session_1, session_2)
 
-        # Verify logger calls.
-        info = manager._database.logger.info
-        info.assert_called_with("Created multiplexed session.")
-
     def test_exception_bad_request(self):
         manager = self._manager
         api = manager._database.spanner_api

tests/unit/test_spanner.py

@@ -12,7 +12,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-
+import sys
+import pytest
 import threading
 from google.protobuf.struct_pb2 import Struct
 from google.cloud.spanner_v1 import (
@@ -1097,6 +1098,10 @@ def test_transaction_for_concurrent_statement_should_begin_one_transaction_with_
         )
         self.assertEqual(actual_id_suffixes, expected_id_suffixes)
 
+    @pytest.mark.skipif(
+        sys.version_info >= (3, 14),
+        reason="Flaky on Python 3.14 due to thread scheduling.",
+    )
     def test_transaction_for_concurrent_statement_should_begin_one_transaction_with_read(
         self,
     ):