diff --git a/api-test/src/schema/string_list.rs b/api-test/src/schema/string_list.rs
index 19608bac..04ae178a 100644
--- a/api-test/src/schema/string_list.rs
+++ b/api-test/src/schema/string_list.rs
@@ -63,11 +63,8 @@ impl<'de> serde::de::Visitor<'de> for StringListVisitor {
 
     fn visit_seq<A: serde::de::SeqAccess<'de>>(self, mut seq: A) -> Result<Self::Value, A::Error> {
         let mut out = seq.size_hint().map_or_else(Vec::new, Vec::with_capacity);
-        loop {
-            match seq.next_element::<String>()? {
-                Some(el) => out.push(el),
-                None => break,
-            }
+        while let Some(el) = seq.next_element::<String>()? {
+            out.push(el);
         }
         Ok(out)
     }
diff --git a/api-test/src/schema/string_set.rs b/api-test/src/schema/string_set.rs
index 20133e4d..d041bbf1 100644
--- a/api-test/src/schema/string_set.rs
+++ b/api-test/src/schema/string_set.rs
@@ -70,11 +70,8 @@ impl<'de> serde::de::Visitor<'de> for StringSetVisitor {
         let mut out = seq
             .size_hint()
             .map_or_else(HashSet::new, HashSet::with_capacity);
-        loop {
-            match seq.next_element::<String>()? {
-                Some(el) => out.insert(el),
-                None => break,
-            };
+        while let Some(el) = seq.next_element::<String>()? {
+            out.insert(el);
         }
         Ok(out)
     }