From 5c6d75a1757febc44b6bc5e88126f52a6c5b620b Mon Sep 17 00:00:00 2001 From: Voker57 Date: Sat, 27 Apr 2024 14:09:22 +0300 Subject: [PATCH] Fix and improve testcase for multiple fields --- tracing-subscriber/src/filter/env/mod.rs | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/tracing-subscriber/src/filter/env/mod.rs b/tracing-subscriber/src/filter/env/mod.rs index 4819e7936..a3f2e36a8 100644 --- a/tracing-subscriber/src/filter/env/mod.rs +++ b/tracing-subscriber/src/filter/env/mod.rs @@ -924,7 +924,7 @@ mod tests { #[test] fn callsite_enabled_includes_span_directive_multiple_fields() { - let filter = EnvFilter::new("app[mySpan{field=\"value\",field2=2}]=debug") + let filter = EnvFilter::try_new("app[mySpan{field=\"value\",field2=2}]=debug").unwrap() .with_collector(NoCollector); static META: &Metadata<'static> = &Metadata::new( "mySpan", @@ -939,6 +939,20 @@ mod tests { let interest = filter.register_callsite(META); assert!(interest.is_never()); + + static META2: &Metadata<'static> = &Metadata::new( + "mySpan", + "app", + Level::TRACE, + None, + None, + None, + FieldSet::new(&["field", "field2"], identify_callsite!(&Cs)), + Kind::SPAN, + ); + + let interest = filter.register_callsite(META); + assert!(interest.is_always()); } #[test]