fixup: riscv: clippy fixes for CSR test assertions

rust-embedded · Sep 25, 2024 · d9c1774 · d9c1774
1 parent 37b8d1d
commit d9c1774
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/riscv/src/register/tests/read_only_csr.rs b/riscv/src/register/tests/read_only_csr.rs
@@ -62,24 +62,24 @@ fn test_mtest_read_only() {
     assert_eq!(mtest.bits(), 0);
 
     // check that single bit field getter/setters work.
-    assert_eq!(mtest.single(), false);
+    assert!(!mtest.single());
 
     mtest = Mtest::from_bits(1);
-    assert_eq!(mtest.single(), true);
+    assert!(mtest.single());
 
     mtest = Mtest::from_bits(0);
 
     // check that single bit range field getter/setters work.
     for i in 1..=3 {
-        assert_eq!(mtest.multi_range(i), false);
+        assert!(!mtest.multi_range(i));
         assert_eq!(mtest.try_multi_range(i), Ok(false));
 
         mtest = Mtest::from_bits(1 << i);
-        assert_eq!(mtest.multi_range(i), true);
+        assert!(mtest.multi_range(i));
         assert_eq!(mtest.try_multi_range(i), Ok(true));
 
         mtest = Mtest::from_bits(0 << i);
-        assert_eq!(mtest.multi_range(i), false);
+        assert!(!mtest.multi_range(i));
         assert_eq!(mtest.try_multi_range(i), Ok(false));
     }
 

diff --git a/riscv/src/register/tests/read_write_csr.rs b/riscv/src/register/tests/read_write_csr.rs
@@ -72,23 +72,23 @@ fn test_mtest_read_write() {
     assert_eq!(mtest.bits(), 0);
 
     // check that single bit field getter/setters work.
-    assert_eq!(mtest.single(), false);
+    assert!(!mtest.single());
 
     mtest.set_single(true);
-    assert_eq!(mtest.single(), true);
+    assert!(mtest.single());
 
     mtest.set_single(false);
-    assert_eq!(mtest.single(), false);
+    assert!(!mtest.single());
 
     // check that single bit range field getter/setters work.
     for i in 1..=3 {
-        assert_eq!(mtest.multi_range(i), false);
+        assert!(!mtest.multi_range(i));
 
         mtest.set_multi_range(i, true);
-        assert_eq!(mtest.multi_range(i), true);
+        assert!(mtest.multi_range(i));
 
         mtest.set_multi_range(i, false);
-        assert_eq!(mtest.multi_range(i), false);
+        assert!(!mtest.multi_range(i));
     }
 
     // check that multi-bit field getter/setters work.