test(hive): update mock to use _get_validated_metastore_connection for bot review

Author: mohitjeswani01

ingestion/tests/unit/topology/database/test_hive.py

@@ -1353,18 +1353,19 @@ def test_no_partition_section_returns_false(self):
 
     def test_metastore_connection_skips_detection(self):
         """When metastoreConnection is set, engine is never called."""
-        self.source.service_connection.metastoreConnection = Mock()
-
-        is_partitioned, partition = self.source.get_table_partition_details(
-            table_name="any_table",
-            schema_name="any_schema",
-            inspector=Mock(),
-        )
+        with patch.object(
+            self.source, "_get_validated_metastore_connection", return_value=Mock()
+        ):
+            is_partitioned, partition = self.source.get_table_partition_details(
+                table_name="any_table",
+                schema_name="any_schema",
+                inspector=Mock(),
+            )
 
-        self.assertFalse(is_partitioned)
-        self.assertIsNone(partition)
-        # Engine must never be touched
-        self.source.engine.connect.assert_not_called()
+            self.assertFalse(is_partitioned)
+            self.assertIsNone(partition)
+            # Engine must never be touched
+            self.source.engine.connect.assert_not_called()
 
     def test_engine_exception_returns_false(self):
         """Engine failure is caught gracefully, returns (False, None)."""