fix: dependencies and lint

Author: coroiu

crates/bitwarden-threading/Cargo.toml

@@ -19,11 +19,11 @@ serde = { workspace = true }
 serde_json = { workspace = true }
 thiserror = { workspace = true }
 tokio = { features = ["sync", "time", "rt"], workspace = true }
+tokio-util = { version = "0.7.15" }
 
 [target.'cfg(target_arch="wasm32")'.dependencies]
 gloo-timers = { version = "0.3.0", features = ["futures"], optional = true }
 js-sys = { workspace = true }
-tokio-util = { version = "0.7.15" }
 tsify-next = { workspace = true }
 wasm-bindgen = { workspace = true }
 wasm-bindgen-futures = { workspace = true }

crates/bitwarden-threading/src/time.rs

@@ -19,9 +19,10 @@ mod test {
     #[wasm_bindgen_test]
     #[allow(dead_code)] // Not actually dead, but rust-analyzer doesn't understand `wasm_bindgen_test`
     async fn should_sleep_wasm() {
-        use super::*;
         use js_sys::Date;
 
+        use super::*;
+
         console_error_panic_hook::set_once();
         let start = Date::now();
 
@@ -36,9 +37,10 @@ mod test {
     // #[cfg(not(target_arch = "wasm32"))]
     #[tokio::test]
     async fn should_sleep_tokio() {
-        use super::*;
         use std::time::Instant;
 
+        use super::*;
+
         let start = Instant::now();
 
         sleep(Duration::from_millis(100)).await;

crates/bitwarden-threading/tests/cancellation_token/wasm.rs

@@ -1,9 +1,12 @@
-use bitwarden_threading::cancellation_token::{
-    wasm::{AbortController, AbortControllerExt, CancellationTokenExt},
-    CancellationToken,
-};
-use bitwarden_threading::time::sleep;
 use std::time::Duration;
+
+use bitwarden_threading::{
+    cancellation_token::{
+        wasm::{AbortController, AbortControllerExt, CancellationTokenExt},
+        CancellationToken,
+    },
+    time::sleep,
+};
 use wasm_bindgen::externref_heap_live_count;
 use wasm_bindgen_test::wasm_bindgen_test;
 
@@ -145,7 +148,8 @@ mod to_bidirectional_abort_controller {
         // Dropping the token should the internal strong reference to the JS object, leaving us
         // with only the strong reference that was returned to the caller.
         // We check this because the reference is kept within a spawn_local future and it'll only be
-        // dropped when the future is dropped. And the future needs to be dropped or we'll leak memory.
+        // dropped when the future is dropped. And the future needs to be dropped or we'll leak
+        // memory.
         assert_eq!(
             heap_count_after_dropping_guard,
             heap_count_after_creating_abort_controller - 1,
@@ -292,7 +296,8 @@ mod to_bidirectional_cancellation_token {
         // Dropping the token should the internal strong reference to the JS object, leaving us
         // with only the strong reference that was returned to the caller.
         // We check this because the reference is kept within a spawn_local future and it'll only be
-        // dropped when the future is dropped. And the future needs to be dropped or we'll leak memory.
+        // dropped when the future is dropped. And the future needs to be dropped or we'll leak
+        // memory.
         assert_eq!(
             heap_count_after_dropping_guard, heap_count_before_creating_cancellation_token,
             "Dropping the token should drop the internal strong reference to the JS object"