fix: [cp25]db_name overwritten by empty string (#2678) (#2702)

issue: #2670
pr: #2678

Added a unit test for it as well.

Signed-off-by: yangxuan <xuan.yang@zilliz.com>
Co-authored-by: Mihail Yanchev <34907806+mihailyanchev@users.noreply.github.com>
Co-authored-by: Mihail Yanchev <yanchev.mihail@gmail.com>

Author: XuanYang-cn

pymilvus/orm/connections.py

@@ -451,7 +451,7 @@ def with_config(config: Tuple) -> bool:
                 user = parsed_uri.username or user
                 password = parsed_uri.password or password
 
-                group = parsed_uri.path.split("/")
+                group = [segment for segment in parsed_uri.path.split("/") if segment]
                 db_name = group[1] if len(group) > 1 else db_name
 
                 # Set secure=True if https scheme

tests/test_connections.py

@@ -372,3 +372,28 @@ def test_issue_1196(self):
 
                 config = connections.get_connection_addr("default")
                 assert config == {"address": 'localhost:19531', "user": 'root', "secure": True}
+
+    def test_issue_2670(self):
+        """
+        Test for db_name being overwritten with empty string, when the uri
+        ends in a slash - e.g. http://localhost:19530/
+
+        See: https://github.com/milvus-io/pymilvus/issues/2670
+
+        Actual behaviour before fix: if a uri is passed ending with a slash,
+            it will overwrite the db_name with an empty string.
+        Expected and current behaviour: if db_name is passed explicitly,
+            it should be used in the initialization of the GrpcHandler.
+        
+        """
+        db_name = "default"
+        alias = self.test_issue_2670.__name__
+
+        with mock.patch(f"{mock_prefix}.__init__", return_value=None) as mock_init, mock.patch(
+            f"{mock_prefix}._wait_for_channel_ready", return_value=None):
+            config = {"alias": alias, "uri": "http://localhost:19530/", "db_name": db_name}
+            connections.connect(**config)
+            
+            mock_init.assert_called_with(
+                **{'address': 'localhost:19530', 'user': '', 'password': '', 'token': '', 'db_name': db_name}
+                )