formatting

Author: sydney-runkle

src/validators/url.rs

@@ -103,7 +103,7 @@ impl Validator for UrlValidator {
                     // given that we want isinstance to work properly for subclasses of Url
                     let py_url = match either_url {
                         EitherUrl::Py(py_url) => py_url.get().clone(),
-                        EitherUrl::Rust(rust_url) => PyUrl::new(rust_url)
+                        EitherUrl::Rust(rust_url) => PyUrl::new(rust_url),
                     };
 
                     let py_url = PyUrl::build(
@@ -294,14 +294,14 @@ impl Validator for MultiHostUrlValidator {
                     // given that we want isinstance to work properly for subclasses of Url
                     let py_url = match multi_url {
                         EitherMultiHostUrl::Py(py_url) => py_url.get().clone(),
-                        EitherMultiHostUrl::Rust(rust_url) => rust_url
+                        EitherMultiHostUrl::Rust(rust_url) => rust_url,
                     };
 
                     let hosts = py_url.hosts(py).map_or(None, |hosts| {
                         let mut host_parts_vec = Vec::new();
-                        for host in hosts.iter() {
+                        for host in &hosts {
                             if let Ok(py_dict) = host.downcast::<PyDict>() {
-                                if let Some(host_parts) = UrlHostParts::extract_bound(py_dict).ok() {
+                                if let Ok(host_parts) = UrlHostParts::extract_bound(py_dict) {
                                     host_parts_vec.push(host_parts);
                                 }
                             }
@@ -319,7 +319,7 @@ impl Validator for MultiHostUrlValidator {
                         None,
                         None,
                         None,
-                        None
+                        None,
                     )?;
                     Ok(py_url.into_py(py))
                 } else {

tests/test_errors.py

@@ -551,7 +551,7 @@ def multi_raise_py_error(v: Any) -> Any:
         s2.validate_python('anything')
 
     cause_group = exc_info.value.__cause__
-    assert isinstance(cause_group, BaseExceptionGroup)
+    assert isinstance(cause_group, BaseExceptionGroup)  # noqa: F821
     assert len(cause_group.exceptions) == 1
 
     cause = cause_group.exceptions[0]
@@ -576,7 +576,7 @@ def outer_raise_py_error(v: Any) -> Any:
     with pytest.raises(ValidationError) as exc_info:
         s3.validate_python('anything')
 
-    assert isinstance(exc_info.value.__cause__, BaseExceptionGroup)
+    assert isinstance(exc_info.value.__cause__, BaseExceptionGroup)  # noqa: F821
     assert len(exc_info.value.__cause__.exceptions) == 1
     cause = exc_info.value.__cause__.exceptions[0]
     assert cause.__notes__ and cause.__notes__[-1].startswith('\nPydantic: ')
@@ -585,7 +585,7 @@ def outer_raise_py_error(v: Any) -> Any:
     assert isinstance(subcause, ValidationError)
 
     cause_group = subcause.__cause__
-    assert isinstance(cause_group, BaseExceptionGroup)
+    assert isinstance(cause_group, BaseExceptionGroup)  # noqa: F821
     assert len(cause_group.exceptions) == 1
 
     cause = cause_group.exceptions[0]