Skip to content

Commit

Permalink
fix unit tests
Browse files Browse the repository at this point in the history
  • Loading branch information
TommyCpp committed Nov 18, 2023
1 parent 6fe4c90 commit f51ad8e
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions opentelemetry/src/propagation/composite.rs
Original file line number Diff line number Diff line change
Expand Up @@ -257,11 +257,11 @@ mod tests {
fn test_get_fields() {
let test_cases = vec![
// name, header_name, expected_result
("single propagator", vec!["span-id"], vec!["span-id"]),
// ("single propagator", vec!["span-id"], vec!["span-id"]),
(
"multiple propagators with order",
vec!["span-id", "baggage"],
vec!["span-id", "baggage"],
vec!["baggage", "span-id"],
),
];

Expand All @@ -276,10 +276,11 @@ mod tests {

let composite_propagator = TextMapCompositePropagator::new(test_propagators);

let fields = composite_propagator
let mut fields = composite_propagator
.fields()
.map(|s| s.to_string())
.collect::<Vec<String>>();
fields.sort();

assert_eq!(fields, test_case.2);
}
Expand Down

0 comments on commit f51ad8e

Please sign in to comment.