Also run clippy on test code

Author: robsdedude

.github/workflows/tests.yaml

@@ -34,7 +34,7 @@ jobs:
       - name: cargo fmt
         run: cargo +nightly fmt --all -- --check
       - name: clippy all
-        run: cargo clippy --all --all-features -- -D warnings
+        run: cargo clippy --all --all-features --tests -- -D warnings
       - name: clippy driver
         run: cargo clippy -p neo4j -- -D warnings
       - name:  Check exported dependencies

.pre-commit-config.yaml

@@ -14,14 +14,14 @@ repos:
         language: system
         types: [rust]
       - id: clippy driver
-        name: cargo clippy (driver)
-        entry: cargo clippy -p neo4j -- -D warnings
+        name: cargo clipcpy (driver)
+        entry: cargo clippy --tests -p neo4j -- -D warnings
         language: system
         types: [rust]
         pass_filenames: false
       - id: clippy all
         name: cargo clippy (all)
-        entry: cargo clippy --all --all-features -- -D warnings
+        entry: cargo clippy --all --all-features --tests -- -D warnings
         language: system
         types: [rust]
         pass_filenames: false

neo4j/src/driver/config.rs

@@ -795,7 +795,7 @@ impl ConnectionConfig {
                 }
             }
             Some(query) => {
-                if query == "" {
+                if query.is_empty() {
                     Some(HashMap::new())
                 } else {
                     if !routing {

neo4j/src/driver/io/bolt/packstream/tests.rs

@@ -39,7 +39,7 @@ fn decode_raw(input: Vec<u8>) -> (Result<ValueReceive, PackStreamDeserializeErro
     let mut reader = input.as_slice();
     let mut deserializer = PackStreamDeserializerImpl::new(&mut reader);
     let result = deserializer.load(&translator);
-    let rest = reader.iter().cloned().collect();
+    let rest = reader.to_vec();
     (result, rest)
 }
 

neo4j/src/value/graph.rs

@@ -409,7 +409,7 @@ mod test {
     fn get_invalid_path_even_index_out_of_range_3() -> Path {
         let mut path = get_path();
         let rel_len_i: isize = path.relationships.len().try_into().unwrap();
-        path.indices[2] = rel_len_i * -1 - 1;
+        path.indices[2] = -rel_len_i - 1;
         path
     }
 
@@ -434,13 +434,13 @@ mod test {
     #[test]
     fn test_path_traverse() {
         let path = get_path();
-        let expected_node_ids = vec!["n1", "n2", "n1", "n3"];
-        let expected_directions = vec![
+        let expected_node_ids = ["n1", "n2", "n1", "n3"];
+        let expected_directions = [
             RelationshipDirection::From,
             RelationshipDirection::To,
             RelationshipDirection::To,
         ];
-        let expected_relationship_ids = vec!["r1", "r2", "r1"];
+        let expected_relationship_ids = ["r1", "r2", "r1"];
 
         let (start_node, hops) = path.traverse();
         assert_eq!(start_node.element_id, expected_node_ids[0]);