Skip to content

Commit ba1b3e0

Browse files
committed
fix the indexeddb tests, that clippy tried to fix
1 parent 6435ea6 commit ba1b3e0

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

crates/matrix-sdk-crypto/src/store/integration_tests.rs

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -343,31 +343,31 @@ macro_rules! cryptostore_integration_tests {
343343
let (_account, store) = get_loaded_store(dir.clone()).await;
344344
let device = get_device();
345345

346-
assert!(store.update_tracked_user(device.user_id(), false).await.unwrap());
347-
assert!(!store.update_tracked_user(device.user_id(), false).await.unwrap());
346+
assert!(store.update_tracked_user(device.user_id(), false).await.unwrap(), "We were not tracked");
347+
assert!(!store.update_tracked_user(device.user_id(), false).await.unwrap(), "We were still tracked ");
348348

349349
assert!(store.is_user_tracked(device.user_id()));
350-
assert!(!store.users_for_key_query().contains(device.user_id()));
351-
assert!(!store.update_tracked_user(device.user_id(), true).await.unwrap());
352-
assert!(store.users_for_key_query().contains(device.user_id()));
350+
assert!(!store.users_for_key_query().contains(device.user_id()), "Unexpectedly key found");
351+
assert!(!store.update_tracked_user(device.user_id(), true).await.unwrap(), "User was there?");
352+
assert!(store.users_for_key_query().contains(device.user_id()), "Didn't find the key despite tracking");
353353
drop(store);
354354

355355
let store = get_store(dir.clone(), None).await;
356356

357357
store.load_account().await.unwrap();
358358

359-
assert!(store.is_user_tracked(device.user_id()));
360-
assert!(store.users_for_key_query().contains(device.user_id()));
359+
assert!(store.is_user_tracked(device.user_id()), "Reopened didn't track");
360+
assert!(store.users_for_key_query().contains(device.user_id()), "Reopened doesn't have the key");
361361

362362
store.update_tracked_user(device.user_id(), false).await.unwrap();
363-
assert!(!store.users_for_key_query().contains(device.user_id()));
363+
assert!(!store.users_for_key_query().contains(device.user_id()), "Reopened has the key despite us not tracking");
364364
drop(store);
365365

366366
let store = get_store(dir, None).await;
367367

368368
store.load_account().await.unwrap();
369369

370-
assert!(!store.users_for_key_query().contains(device.user_id()));
370+
assert!(!store.users_for_key_query().contains(device.user_id()), "Reloaded store has the account");
371371
}
372372

373373
#[async_test]

crates/matrix-sdk-indexeddb/src/cryptostore.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -427,7 +427,7 @@ impl IndexeddbStore {
427427
let user_ids = os.get_all_keys()?.await?;
428428
for user_id in user_ids.iter() {
429429
let dirty: bool =
430-
matches!(os.get(&user_id)?.await?.map(|v| v.into_serde()), Some(Ok(false)));
430+
!matches!(os.get(&user_id)?.await?.map(|v| v.into_serde()), Some(Ok(false)));
431431
let user = match user_id.as_string().map(UserId::parse) {
432432
Some(Ok(user)) => user,
433433
_ => continue,

0 commit comments

Comments
 (0)