Allow for both reference and owned vec/strings in negotiate_languages

Author: Zibi Braniecki

src/negotiate/mod.rs

@@ -326,13 +326,17 @@ fn filter_matches<'a>(
     supported_locales
 }
 
-pub fn negotiate_languages<'a>(
-    requested: &[&'a str],
-    available: &[&'a str],
+pub fn negotiate_languages<'a, T: AsRef<str>>(
+    requested: &'a [T],
+    available: &'a [T],
     default: Option<&'a str>,
     strategy: &NegotiationStrategy,
 ) -> Vec<&'a str> {
-    let mut supported = filter_matches(requested, available, strategy);
+    let mut supported = filter_matches(
+        &requested.into_iter().map(AsRef::as_ref).collect::<Vec<_>>(),
+        &available.into_iter().map(AsRef::as_ref).collect::<Vec<_>>(),
+        strategy,
+    );
 
     if let Some(d) = default {
         if strategy == &NegotiationStrategy::Lookup {

tests/lib.rs

@@ -138,6 +138,7 @@ fn test_negotiate_fixtures(path: &str) {
         };
         match test.input {
             NegotiateTestInput::NoDefault(r, a) => {
+                // One with &str
                 let requested: Vec<&str> = r.iter().map(|v| v.as_str()).collect();
                 let available: Vec<&str> = a.iter().map(|v| v.as_str()).collect();
                 assert_eq!(
@@ -147,9 +148,8 @@ fn test_negotiate_fixtures(path: &str) {
                     path
                 );
             }
-            NegotiateTestInput::Default(r, a, default) => {
-                let requested: Vec<&str> = r.iter().map(|v| v.as_str()).collect();
-                let available: Vec<&str> = a.iter().map(|v| v.as_str()).collect();
+            NegotiateTestInput::Default(requested, available, default) => {
+                // One with String
                 assert_eq!(
                     negotiate_languages(&requested, &available, Some(default.as_str()), &strategy),
                     test.output,