Fix more clippy errors.

samwho · Jun 4, 2023 · 51b400a · 51b400a
1 parent b10a592
commit 51b400a
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 14 deletions.
diff --git a/src/bin/hmm.rs b/src/bin/hmm.rs
@@ -185,9 +185,8 @@ mod tests {
         let path = new_tempfile_path();
         for message in &messages {
             let assert = run_with_path(&path, vec![message]);
-            assert_eq!(
+            assert!(
                 assert.get_output().stdout.is_empty(),
-                true,
                 "expected stdout to be empty"
             );
             assert.success();
@@ -206,7 +205,7 @@ mod tests {
 
             messages.push(entry.message().to_owned());
 
-            assert_eq!(true, &date <= entry.datetime());
+            assert!(&date <= entry.datetime());
             date = entry.datetime().to_owned();
         }
 
@@ -220,9 +219,8 @@ mod tests {
         let assert = HMM.command().args(args).assert();
         let stderr = String::from_utf8(assert.get_output().stderr.clone()).unwrap();
         assert.failure();
-        assert_eq!(
+        assert!(
             stderr.contains(error),
-            true,
             "could not find \"{}\" in \"{}\"",
             error,
             stderr

diff --git a/src/bin/hmmq.rs b/src/bin/hmmq.rs
@@ -342,9 +342,8 @@ mod tests {
         let assert = HMMQ.command().args(args).assert();
         let stderr = String::from_utf8(assert.get_output().stderr.clone()).unwrap();
         assert.failure();
-        assert_eq!(
+        assert!(
             stderr.contains(error),
-            true,
             "could not find \"{}\" in \"{}\"",
             error,
             stderr

diff --git a/src/entries.rs b/src/entries.rs
@@ -281,23 +281,23 @@ mod tests {
         assert_eq!(entries.next_entry()?.unwrap().message(), "4");
         assert_eq!(entries.next_entry()?.unwrap().message(), "5");
         assert_eq!(entries.next_entry()?.unwrap().message(), "6");
-        assert_eq!(entries.next_entry()?.is_none(), true);
+        assert!(entries.next_entry()?.is_none());
         assert_eq!(entries.prev_entry()?.unwrap().message(), "6");
         assert_eq!(entries.prev_entry()?.unwrap().message(), "5");
         assert_eq!(entries.prev_entry()?.unwrap().message(), "4");
         assert_eq!(entries.prev_entry()?.unwrap().message(), "3");
         assert_eq!(entries.prev_entry()?.unwrap().message(), "2");
         assert_eq!(entries.prev_entry()?.unwrap().message(), "1");
-        assert_eq!(entries.prev_entry()?.is_none(), true);
-        assert_eq!(entries.prev_entry()?.is_none(), true);
-        assert_eq!(entries.prev_entry()?.is_none(), true);
+        assert!(entries.prev_entry()?.is_none());
+        assert!(entries.prev_entry()?.is_none());
+        assert!(entries.prev_entry()?.is_none());
         assert_eq!(entries.next_entry()?.unwrap().message(), "1");
         assert_eq!(entries.next_entry()?.unwrap().message(), "2");
         assert_eq!(entries.next_entry()?.unwrap().message(), "3");
         assert_eq!(entries.next_entry()?.unwrap().message(), "4");
         assert_eq!(entries.next_entry()?.unwrap().message(), "5");
         assert_eq!(entries.next_entry()?.unwrap().message(), "6");
-        assert_eq!(entries.next_entry()?.is_none(), true);
+        assert!(entries.next_entry()?.is_none());
         Ok(())
     }
 
@@ -322,6 +322,6 @@ mod tests {
         assert_eq!(entries.next().unwrap().unwrap().message(), "4");
         assert_eq!(entries.next().unwrap().unwrap().message(), "5");
         assert_eq!(entries.next().unwrap().unwrap().message(), "6");
-        assert_eq!(entries.next().is_none(), true);
+        assert!(entries.next().is_none());
     }
 }
diff --git a/src/format.rs b/src/format.rs
@@ -122,7 +122,7 @@ mod tests {
         Format::with_template(template)
             .unwrap()
             .format_entry(&Entry::new(
-                Utc.ymd(2020, 1, 2).and_hms(3, 4, 5).into(),
+                DateTime::parse_from_rfc3339("2020-01-02T03:04:05Z").unwrap(),
                 "hello world".to_owned(),
             ))
             .unwrap()