From 21a2951cd85d32ece115a4d757fec798570a3b10 Mon Sep 17 00:00:00 2001
From: David Herberth <david.herberth@sentry.io>
Date: Wed, 9 Oct 2024 12:15:49 +0200
Subject: [PATCH] test(pattern): Assert all patterns when compiling in debug
 mode

---
 relay-pattern/src/typed.rs | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/relay-pattern/src/typed.rs b/relay-pattern/src/typed.rs
index 64dd73a722..773e879a24 100644
--- a/relay-pattern/src/typed.rs
+++ b/relay-pattern/src/typed.rs
@@ -180,7 +180,9 @@ impl<C: PatternConfig> FromIterator<String> for TypedPatterns<C> {
     fn from_iter<T: IntoIterator<Item = String>>(iter: T) -> Self {
         let mut builder = Self::builder();
         for pattern in iter.into_iter() {
-            let _ = builder.add(pattern);
+            let _err = builder.add(pattern);
+            #[cfg(debug_assertions)]
+            _err.expect("all patterns should be valid patterns");
         }
         builder.build()
     }
@@ -248,7 +250,9 @@ impl<'de, C: PatternConfig> serde::Deserialize<'de> for TypedPatterns<C> {
 
                 while let Some(item) = seq.next_element()? {
                     // Ignore invalid patterns as documented.
-                    let _ = builder.add(item);
+                    let _err = builder.add(item);
+                    #[cfg(debug_assertions)]
+                    _err.expect("all patterns should be valid patterns");
                 }
 
                 Ok(builder.build())