Merge branch 'use-clippy' into 'master'

Use clippy checks in CI

See merge request !19

.gitlab-ci.yml

@@ -16,6 +16,7 @@ image: "rust:latest"
 before_script:
 - apt-get update -yqq
 - apt-get install -yqq --no-install-recommends build-essential
+- rustup component add clippy
 
 # Use cargo to test the project
 test:cargo:
@@ -25,6 +26,10 @@ test:cargo:
   - cargo test --all --all-features --verbose
   - cargo test --all --no-default-features --verbose
 
+test:lint:
+  script:
+  - cargo clippy --all-targets --all-features -- -D warnings
+
 build:
   script:
   - cargo build

src/blocking.rs

@@ -233,7 +233,7 @@ mod tests {
         assert_eq!(0, query.len());
 
         let query = handle.search("spotify").unwrap();
-        assert!(query.len() > 0);
+        assert!(!query.is_empty());
         assert!(query[0].url.is_some());
     }
 

src/handle.rs

@@ -238,7 +238,7 @@ mod tests {
         assert_eq!(0, query.len());
 
         let query = handle.search("spotify").await.unwrap();
-        assert!(query.len() > 0);
+        assert!(!query.is_empty());
         assert!(query[0].url.is_some());
     }