Separate and rename tests to match prose

Author: aclark4life

test/test_uri_parser.py

@@ -554,12 +554,15 @@ def test_port_with_whitespace(self):
         with self.assertRaisesRegex(ValueError, r"Port contains whitespace character: '\\n'"):
             parse_uri("mongodb://localhost:27\n017")
 
-    def test_allow_srv_hosts_with_fewer_than_three_dot_separated_parts(self):
+    # Initial DNS Seedlist Discovery prose tests
+    # https://github.com/mongodb/specifications/blob/0a7a8b5/source/initial-dns-seedlist-discovery/tests/README.md#prose-tests
+
+    def test_1_allow_srv_hosts_with_fewer_than_three_dot_separated_parts(self):
         with patch("dns.resolver.resolve"):
             parse_uri("mongodb+srv://localhost/")
             parse_uri("mongodb+srv://mongo.local/")
 
-    def test_error_when_return_address_does_not_end_with_srv_domain(self):
+    def test_2_throw_when_return_address_does_not_end_with_srv_domain(self):
         test_cases = [
             {
                 "query": "_mongodb._tcp.localhost",
@@ -576,6 +579,27 @@ def test_error_when_return_address_does_not_end_with_srv_domain(self):
                 "mock_target": "test_1.evil.com",
                 "expected_error": "Invalid SRV host",
             },
+        ]
+        for case in test_cases:
+            with patch("dns.resolver.resolve") as mock_resolver:
+
+                def mock_resolve(query, record_type, *args, **kwargs):
+                    mock_srv = MagicMock()
+                    mock_srv.target.to_text.return_value = case["mock_target"]
+                    return [mock_srv]
+
+                mock_resolver.side_effect = mock_resolve
+                domain = case["query"].split("._tcp.")[1]
+                connection_string = f"mongodb+srv://{domain}"
+                try:
+                    parse_uri(connection_string)
+                except ConfigurationError as e:
+                    self.assertIn(case["expected_error"], str(e))
+                else:
+                    self.fail(f"ConfigurationError was not raised for query: {case['query']}")
+
+    def test_3_throw_when_return_address_is_identical_to_srv_hostname(self):
+        test_cases = [
             {
                 "query": "_mongodb._tcp.localhost",
                 "mock_target": "localhost",
@@ -586,6 +610,28 @@ def test_error_when_return_address_does_not_end_with_srv_domain(self):
                 "mock_target": "mongo.local",
                 "expected_error": "Invalid SRV host",
             },
+        ]
+        for case in test_cases:
+            with patch("dns.resolver.resolve") as mock_resolver:
+
+                def mock_resolve(query, record_type, *args, **kwargs):
+                    mock_srv = MagicMock()
+                    mock_srv.target.to_text.return_value = case["mock_target"]
+                    return [mock_srv]
+
+                mock_resolver.side_effect = mock_resolve
+                domain = case["query"].split("._tcp.")[1]
+                connection_string = f"mongodb+srv://{domain}"
+                try:
+                    parse_uri(connection_string)
+                except ConfigurationError as e:
+                    self.assertIn(case["expected_error"], str(e))
+                else:
+                    self.fail(f"ConfigurationError was not raised for query: {case['query']}")
+
+    def test_4_throw_when_return_address_does_not_contain_dot_separating_shared_part_of_domain(self):
+        test_cases = [
+
             {
                 "query": "_mongodb._tcp.localhost",
                 "mock_target": "test_1.cluster_1localhost",