chore: remove duplicate idn-email test and fix ambiguous test descriptions (#881)

AcEKaycgR · web-flow · commit aac8a1bd4d46 · 2026-03-26T15:06:36.000-07:00
diff --git a/tests/draft2019-09/optional/format/idn-email.json b/tests/draft2019-09/optional/format/idn-email.json
@@ -42,19 +42,14 @@
                 "valid": true
             },
             {
-                "description": "an invalid idn e-mail address",
+                "description": "an invalid e-mail/idn e-mail address",
                 "data": "2962",
                 "valid": false
             },
             {
                 "description": "a valid e-mail address",
                 "data": "joe.bloggs@example.com",
                 "valid": true
-            },
-            {
-                "description": "an invalid e-mail address",
-                "data": "2962",
-                "valid": false
             }
         ]
     }
diff --git a/tests/draft2019-09/optional/format/ipv6.json b/tests/draft2019-09/optional/format/ipv6.json
@@ -42,7 +42,7 @@
                 "valid": true
             },
             {
-                "description": "an IPv6 address with out-of-range values",
+                "description": "a group with 5 hex digits is invalid",
                 "data": "12345::",
                 "valid": false
             },
diff --git a/tests/draft2019-09/optional/format/iri.json b/tests/draft2019-09/optional/format/iri.json
@@ -62,7 +62,7 @@
                 "valid": true
             },
             {
-                "description": "an invalid IRI based on IPv6",
+                "description": "an IPv6 address without enclosing brackets is invalid",
                 "data": "http://2001:0db8:85a3:0000:0000:8a2e:0370:7334",
                 "valid": false
             },
diff --git a/tests/draft2019-09/optional/format/uri.json b/tests/draft2019-09/optional/format/uri.json
@@ -77,7 +77,7 @@
                 "valid": true
             },
             {
-                "description": "a valid URL ",
+                "description": "a valid URL",
                 "data": "ldap://[2001:db8::7]/c=GB?objectClass?one",
                 "valid": true
             },
diff --git a/tests/draft2020-12/optional/format/idn-email.json b/tests/draft2020-12/optional/format/idn-email.json
@@ -42,19 +42,14 @@
                 "valid": true
             },
             {
-                "description": "an invalid idn e-mail address",
+                "description": "an invalid e-mail/idn e-mail address",
                 "data": "2962",
                 "valid": false
             },
             {
                 "description": "a valid e-mail address",
                 "data": "joe.bloggs@example.com",
                 "valid": true
-            },
-            {
-                "description": "an invalid e-mail address",
-                "data": "2962",
-                "valid": false
             }
         ]
     }
diff --git a/tests/draft2020-12/optional/format/ipv6.json b/tests/draft2020-12/optional/format/ipv6.json
@@ -42,7 +42,7 @@
                 "valid": true
             },
             {
-                "description": "an IPv6 address with out-of-range values",
+                "description": "a group with 5 hex digits is invalid",
                 "data": "12345::",
                 "valid": false
             },
diff --git a/tests/draft2020-12/optional/format/iri.json b/tests/draft2020-12/optional/format/iri.json
@@ -62,7 +62,7 @@
                 "valid": true
             },
             {
-                "description": "an invalid IRI based on IPv6",
+                "description": "an IPv6 address without enclosing brackets is invalid",
                 "data": "http://2001:0db8:85a3:0000:0000:8a2e:0370:7334",
                 "valid": false
             },
diff --git a/tests/draft2020-12/optional/format/uri.json b/tests/draft2020-12/optional/format/uri.json
@@ -77,7 +77,7 @@
                 "valid": true
             },
             {
-                "description": "a valid URL ",
+                "description": "a valid URL",
                 "data": "ldap://[2001:db8::7]/c=GB?objectClass?one",
                 "valid": true
             },
diff --git a/tests/draft3/optional/format/ipv6.json b/tests/draft3/optional/format/ipv6.json
@@ -9,7 +9,7 @@
                 "valid": true
             },
             {
-                "description": "an IPv6 address with out-of-range values",
+                "description": "a group with 5 hex digits is invalid",
                 "data": "12345::",
                 "valid": false
             },
diff --git a/tests/draft4/optional/format/ipv6.json b/tests/draft4/optional/format/ipv6.json
@@ -39,7 +39,7 @@
                 "valid": true
             },
             {
-                "description": "an IPv6 address with out-of-range values",
+                "description": "a group with 5 hex digits is invalid",
                 "data": "12345::",
                 "valid": false
             },
diff --git a/tests/draft4/optional/format/uri.json b/tests/draft4/optional/format/uri.json
@@ -74,7 +74,7 @@
                 "valid": true
             },
             {
-                "description": "a valid URL ",
+                "description": "a valid URL",
                 "data": "ldap://[2001:db8::7]/c=GB?objectClass?one",
                 "valid": true
             },
diff --git a/tests/draft6/optional/format/ipv6.json b/tests/draft6/optional/format/ipv6.json
@@ -39,7 +39,7 @@
                 "valid": true
             },
             {
-                "description": "an IPv6 address with out-of-range values",
+                "description": "a group with 5 hex digits is invalid",
                 "data": "12345::",
                 "valid": false
             },
diff --git a/tests/draft6/optional/format/uri.json b/tests/draft6/optional/format/uri.json
@@ -74,7 +74,7 @@
                 "valid": true
             },
             {
-                "description": "a valid URL ",
+                "description": "a valid URL",
                 "data": "ldap://[2001:db8::7]/c=GB?objectClass?one",
                 "valid": true
             },
diff --git a/tests/draft7/optional/format/idn-email.json b/tests/draft7/optional/format/idn-email.json
@@ -39,19 +39,14 @@
                 "valid": true
             },
             {
-                "description": "an invalid idn e-mail address",
+                "description": "an invalid e-mail/idn e-mail address",
                 "data": "2962",
                 "valid": false
             },
             {
                 "description": "a valid e-mail address",
                 "data": "joe.bloggs@example.com",
                 "valid": true
-            },
-            {
-                "description": "an invalid e-mail address",
-                "data": "2962",
-                "valid": false
             }
         ]
     }
diff --git a/tests/draft7/optional/format/ipv6.json b/tests/draft7/optional/format/ipv6.json
@@ -39,7 +39,7 @@
                 "valid": true
             },
             {
-                "description": "an IPv6 address with out-of-range values",
+                "description": "a group with 5 hex digits is invalid",
                 "data": "12345::",
                 "valid": false
             },
diff --git a/tests/draft7/optional/format/iri.json b/tests/draft7/optional/format/iri.json
@@ -59,7 +59,7 @@
                 "valid": true
             },
             {
-                "description": "an invalid IRI based on IPv6",
+                "description": "an IPv6 address without enclosing brackets is invalid",
                 "data": "http://2001:0db8:85a3:0000:0000:8a2e:0370:7334",
                 "valid": false
             },
diff --git a/tests/draft7/optional/format/uri.json b/tests/draft7/optional/format/uri.json
@@ -74,7 +74,7 @@
                 "valid": true
             },
             {
-                "description": "a valid URL ",
+                "description": "a valid URL",
                 "data": "ldap://[2001:db8::7]/c=GB?objectClass?one",
                 "valid": true
             },

Original file line number	Diff line number	Diff line change
`@@ -42,19 +42,14 @@`
`42`	`42`	`"valid": true`
`43`	`43`	`},`
`44`	`44`	`{`
`45`		`- "description": "an invalid idn e-mail address",`
	`45`	`+ "description": "an invalid e-mail/idn e-mail address",`
`46`	`46`	`"data": "2962",`
`47`	`47`	`"valid": false`
`48`	`48`	`},`
`49`	`49`	`{`
`50`	`50`	`"description": "a valid e-mail address",`
`51`	`51`	`"data": "joe.bloggs@example.com",`
`52`	`52`	`"valid": true`
`53`		`- },`
`54`		`- {`
`55`		`- "description": "an invalid e-mail address",`
`56`		`- "data": "2962",`
`57`		`- "valid": false`
`58`	`53`	`}`
`59`	`54`	`]`
`60`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@`
`42`	`42`	`"valid": true`
`43`	`43`	`},`
`44`	`44`	`{`
`45`		`- "description": "an IPv6 address with out-of-range values",`
	`45`	`+ "description": "a group with 5 hex digits is invalid",`
`46`	`46`	`"data": "12345::",`
`47`	`47`	`"valid": false`
`48`	`48`	`},`
Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@`
`62`	`62`	`"valid": true`
`63`	`63`	`},`
`64`	`64`	`{`
`65`		`- "description": "an invalid IRI based on IPv6",`
	`65`	`+ "description": "an IPv6 address without enclosing brackets is invalid",`
`66`	`66`	`"data": "http://2001:0db8:85a3:0000:0000:8a2e:0370:7334",`
`67`	`67`	`"valid": false`
`68`	`68`	`},`
Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,7 @@`
`77`	`77`	`"valid": true`
`78`	`78`	`},`
`79`	`79`	`{`
`80`		`- "description": "a valid URL ",`
	`80`	`+ "description": "a valid URL",`
`81`	`81`	`"data": "ldap://[2001:db8::7]/c=GB?objectClass?one",`
`82`	`82`	`"valid": true`
`83`	`83`	`},`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`"valid": true`
`10`	`10`	`},`
`11`	`11`	`{`
`12`		`- "description": "an IPv6 address with out-of-range values",`
	`12`	`+ "description": "a group with 5 hex digits is invalid",`
`13`	`13`	`"data": "12345::",`
`14`	`14`	`"valid": false`
`15`	`15`	`},`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@`
`39`	`39`	`"valid": true`
`40`	`40`	`},`
`41`	`41`	`{`
`42`		`- "description": "an IPv6 address with out-of-range values",`
	`42`	`+ "description": "a group with 5 hex digits is invalid",`
`43`	`43`	`"data": "12345::",`
`44`	`44`	`"valid": false`
`45`	`45`	`},`