lint fix

Author: pranshi06

crates/connectors/ndc-bigquery/tests/explain_tests.rs

@@ -8,23 +8,23 @@ use tests_common::request::run_explain;
 #[ignore]
 async fn select_by_pk() {
     let result = run_explain(create_router().await, "select_by_pk").await;
-    is_contained_in_lines(vec!["Aggregate", "Scan", "35"], result.details.plan);
+    is_contained_in_lines(&["Aggregate", "Scan", "35"], &result.details.plan);
     insta::assert_snapshot!(result.details.query);
 }
 
 #[tokio::test]
 #[ignore]
 async fn select_where_variable() {
     let result = run_explain(create_router().await, "select_where_variable").await;
-    is_contained_in_lines(vec!["Aggregate", "Seq Scan", "Filter"], result.details.plan);
+    is_contained_in_lines(&["Aggregate", "Seq Scan", "Filter"], &result.details.plan);
     insta::assert_snapshot!(result.details.query);
 }
 
 #[tokio::test]
 #[ignore]
 async fn select_where_name_nilike() {
     let result = run_explain(create_router().await, "select_where_name_nilike").await;
-    let keywords = vec!["Aggregate", "Subquery Scan", "Limit", "Seq Scan", "Filter"];
-    is_contained_in_lines(keywords, result.details.plan);
+    let keywords = &["Aggregate", "Subquery Scan", "Limit", "Seq Scan", "Filter"];
+    is_contained_in_lines(keywords, &result.details.plan);
     insta::assert_snapshot!(result.details.query);
 }

crates/tests/tests-common/src/assert.rs

@@ -4,7 +4,7 @@
 /// Used to check the output of EXPLAIN. We use this method instead of
 /// snapshot testing because small details (like cost) can change from
 /// run to run rendering the output unstable.
-pub fn is_contained_in_lines(keywords: Vec<&str>, lines: String) {
+pub fn is_contained_in_lines(keywords: &[&str], lines: &str) {
     tracing::info!("expected keywords: {:?}\nlines: {}", keywords, lines,);
     assert!(keywords.iter().all(|&s| lines.contains(s)));
 }