From 2ace220366a19ac456dd9fba0de907dd26ae286d Mon Sep 17 00:00:00 2001 From: Ivan Enderlin Date: Thu, 30 Mar 2023 15:06:19 +0200 Subject: [PATCH] chore(sdk): Rename `rooms_list` to `room_list`. --- bindings/matrix-sdk-ffi/src/sliding_sync.rs | 8 +- crates/matrix-sdk/src/sliding_sync/README.md | 12 +- crates/matrix-sdk/src/sliding_sync/builder.rs | 4 +- .../src/sliding_sync/list/builder.rs | 4 +- .../src/sliding_sync/list/frozen.rs | 14 +- .../matrix-sdk/src/sliding_sync/list/mod.rs | 216 +++++++++--------- .../sliding-sync-integration-test/src/lib.rs | 72 +++--- 7 files changed, 165 insertions(+), 165 deletions(-) diff --git a/bindings/matrix-sdk-ffi/src/sliding_sync.rs b/bindings/matrix-sdk-ffi/src/sliding_sync.rs index 1c043cb88..ffba8af4a 100644 --- a/bindings/matrix-sdk-ffi/src/sliding_sync.rs +++ b/bindings/matrix-sdk-ffi/src/sliding_sync.rs @@ -578,11 +578,11 @@ impl SlidingSyncList { &self, observer: Box, ) -> Arc { - let mut rooms_list_stream = self.inner.rooms_list_stream(); + let mut room_list_stream = self.inner.room_list_stream(); Arc::new(TaskHandle::new(RUNTIME.spawn(async move { loop { - if let Some(diff) = rooms_list_stream.next().await { + if let Some(diff) = room_list_stream.next().await { observer.did_receive_update(diff.into()); } } @@ -608,8 +608,8 @@ impl SlidingSyncList { #[uniffi::export] impl SlidingSyncList { /// Get the current list of rooms - pub fn current_rooms_list(&self) -> Vec { - self.inner.rooms_list() + pub fn current_room_list(&self) -> Vec { + self.inner.room_list() } /// Reset the ranges to a particular set diff --git a/crates/matrix-sdk/src/sliding_sync/README.md b/crates/matrix-sdk/src/sliding_sync/README.md index f23914be8..01b5aaa35 100644 --- a/crates/matrix-sdk/src/sliding_sync/README.md +++ b/crates/matrix-sdk/src/sliding_sync/README.md @@ -96,12 +96,12 @@ copy can be retrieved by calling `SlidingSync::list()`, providing the name of the list. Next to the configuration settings (like name and `timeline_limit`), the list provides the stateful [`maximum_number_of_rooms`](SlidingSyncList::maximum_number_of_rooms), -[`rooms_list`](SlidingSyncList::rooms_list) and +[`room_list`](SlidingSyncList::room_list) and [`state`](SlidingSyncList::state): - `maximum_number_of_rooms` is the number of rooms _total_ there were found matching the filters given. - - `rooms_list` is a vector of `maximum_number_of_rooms` [`RoomListEntry`]'s + - `room_list` is a vector of `maximum_number_of_rooms` [`RoomListEntry`]'s at the current state. `RoomListEntry`'s only hold `the room_id` if given, the [Rooms API](#rooms) holds the actual information about each room - `state` is a [`SlidingSyncMode`] signalling meta information about the @@ -188,7 +188,7 @@ the bandwidth back down to what is really needed. ## Extensions -Additionally to the rooms list and rooms with their state and latest +Additionally to the room list and rooms with their state and latest messages Matrix knows of many other exchange information. All these are modeled as specific, optional extensions in the [sliding sync protocol][MSC]. This includes end-to-end-encryption, to-device-messages, @@ -384,7 +384,7 @@ only adds them later, they will not be reading the data from storage (to avoid inconsistencies) and might require more data to be sent in their first request than if they were loaded form cold-cache. -When loading from storage `rooms_list` entries found are set to +When loading from storage `room_list` entries found are set to `Invalidated` — the initial setting here is communicated as a single `VecDiff::Replace` event through the [reactive API](#reactive-api). @@ -468,7 +468,7 @@ let sliding_sync = sliding_sync_builder let active_list = sliding_sync.list(&active_list_name).unwrap(); let list_state_stream = active_list.state_stream(); let list_count_stream = active_list.maximum_number_of_rooms_stream(); -let list_stream = active_list.rooms_list_stream(); +let list_stream = active_list.room_list_stream(); tokio::spawn(async move { pin_mut!(list_state_stream); @@ -487,7 +487,7 @@ tokio::spawn(async move { tokio::spawn(async move { pin_mut!(list_stream); while let Some(v_diff) = list_stream.next().await { - info!("active-list rooms list diff update: {v_diff:?}"); + info!("active-list room list diff update: {v_diff:?}"); } }); diff --git a/crates/matrix-sdk/src/sliding_sync/builder.rs b/crates/matrix-sdk/src/sliding_sync/builder.rs index 8838bb320..7dd145d87 100644 --- a/crates/matrix-sdk/src/sliding_sync/builder.rs +++ b/crates/matrix-sdk/src/sliding_sync/builder.rs @@ -248,9 +248,9 @@ impl SlidingSyncBuilder { { trace!(name, "frozen for list found"); - let FrozenSlidingSyncList { maximum_number_of_rooms, rooms_list, rooms } = + let FrozenSlidingSyncList { maximum_number_of_rooms, room_list, rooms } = frozen_list; - list.set_from_cold(maximum_number_of_rooms, rooms_list); + list.set_from_cold(maximum_number_of_rooms, room_list); for (key, frozen_room) in rooms.into_iter() { rooms_found.entry(key).or_insert_with(|| { diff --git a/crates/matrix-sdk/src/sliding_sync/list/builder.rs b/crates/matrix-sdk/src/sliding_sync/list/builder.rs index 0f01eda17..713e8ae95 100644 --- a/crates/matrix-sdk/src/sliding_sync/list/builder.rs +++ b/crates/matrix-sdk/src/sliding_sync/list/builder.rs @@ -61,7 +61,7 @@ impl SlidingSyncListBuilder { self } - /// Sort the rooms list by this. + /// Sort the room list by this. pub fn sort(mut self, value: Vec) -> Self { self.sort = value; self @@ -172,7 +172,7 @@ impl SlidingSyncListBuilder { // Default values for the type we are building. state: StdRwLock::new(Observable::new(SlidingSyncState::default())), maximum_number_of_rooms: StdRwLock::new(Observable::new(None)), - rooms_list: StdRwLock::new(ObservableVector::new()), + room_list: StdRwLock::new(ObservableVector::new()), }), }) } diff --git a/crates/matrix-sdk/src/sliding_sync/list/frozen.rs b/crates/matrix-sdk/src/sliding_sync/list/frozen.rs index b549be34e..7ed4321c7 100644 --- a/crates/matrix-sdk/src/sliding_sync/list/frozen.rs +++ b/crates/matrix-sdk/src/sliding_sync/list/frozen.rs @@ -11,7 +11,7 @@ pub struct FrozenSlidingSyncList { #[serde(default, rename = "rooms_count", skip_serializing_if = "Option::is_none")] pub maximum_number_of_rooms: Option, #[serde(default, skip_serializing_if = "Vector::is_empty")] - pub rooms_list: Vector, + pub room_list: Vector, #[serde(default, skip_serializing_if = "BTreeMap::is_empty")] pub(in super::super) rooms: BTreeMap, } @@ -22,9 +22,9 @@ impl FrozenSlidingSyncList { rooms_map: &BTreeMap, ) -> Self { let mut rooms = BTreeMap::new(); - let mut rooms_list = Vector::new(); + let mut room_list = Vector::new(); - for room_list_entry in source_list.inner.rooms_list.read().unwrap().iter() { + for room_list_entry in source_list.inner.room_list.read().unwrap().iter() { match room_list_entry { RoomListEntry::Filled(room_id) | RoomListEntry::Invalidated(room_id) => { rooms.insert( @@ -36,12 +36,12 @@ impl FrozenSlidingSyncList { _ => {} }; - rooms_list.push_back(room_list_entry.freeze_by_ref()); + room_list.push_back(room_list_entry.freeze_by_ref()); } FrozenSlidingSyncList { maximum_number_of_rooms: source_list.maximum_number_of_rooms(), - rooms_list, + room_list, rooms, } } @@ -64,7 +64,7 @@ mod tests { assert_eq!( serde_json::to_value(&FrozenSlidingSyncList { maximum_number_of_rooms: Some(42), - rooms_list: vector![RoomListEntry::Empty], + room_list: vector![RoomListEntry::Empty], rooms: { let mut rooms = BTreeMap::new(); rooms.insert( @@ -95,7 +95,7 @@ mod tests { .unwrap(), json!({ "rooms_count": 42, - "rooms_list": ["Empty"], + "room_list": ["Empty"], "rooms": { "!foo:bar.org": { "room_id": "!foo:bar.org", diff --git a/crates/matrix-sdk/src/sliding_sync/list/mod.rs b/crates/matrix-sdk/src/sliding_sync/list/mod.rs index 0dc14e7da..2f95f41cf 100644 --- a/crates/matrix-sdk/src/sliding_sync/list/mod.rs +++ b/crates/matrix-sdk/src/sliding_sync/list/mod.rs @@ -60,7 +60,7 @@ impl SlidingSyncList { pub(crate) fn set_from_cold( &mut self, maximum_number_of_rooms: Option, - rooms_list: Vector, + room_list: Vector, ) { Observable::set(&mut self.inner.state.write().unwrap(), SlidingSyncState::Preloaded); Observable::set( @@ -68,9 +68,9 @@ impl SlidingSyncList { maximum_number_of_rooms, ); - let mut lock = self.inner.rooms_list.write().unwrap(); + let mut lock = self.inner.room_list.write().unwrap(); lock.clear(); - lock.append(rooms_list); + lock.append(room_list); } /// Get the name of the list. @@ -165,17 +165,17 @@ impl SlidingSyncList { Observable::set(&mut self.inner.timeline_limit.write().unwrap(), timeline); } - /// Get the current rooms list. - pub fn rooms_list(&self) -> Vec + /// Get the current room list. + pub fn room_list(&self) -> Vec where R: for<'a> From<&'a RoomListEntry>, { - self.inner.rooms_list.read().unwrap().iter().map(|e| R::from(e)).collect() + self.inner.room_list.read().unwrap().iter().map(|e| R::from(e)).collect() } - /// Get a stream of rooms list. - pub fn rooms_list_stream(&self) -> impl Stream> { - ObservableVector::subscribe(&self.inner.rooms_list.read().unwrap()) + /// Get a stream of room list. + pub fn room_list_stream(&self) -> impl Stream> { + ObservableVector::subscribe(&self.inner.room_list.read().unwrap()) } /// Get the maximum number of rooms. See [`Self::maximum_number_of_rooms`] @@ -192,7 +192,7 @@ impl SlidingSyncList { /// Return the `room_id` at the given index. pub fn get_room_id(&self, index: usize) -> Option { self.inner - .rooms_list + .room_list .read() .unwrap() .get(index) @@ -211,7 +211,7 @@ impl SlidingSyncList { maximum_number_of_rooms: u32, sync_operations: &[v4::SyncOp], ) -> Result { - let result = self.inner.update_rooms_list(maximum_number_of_rooms, sync_operations)?; + let result = self.inner.update_room_list(maximum_number_of_rooms, sync_operations)?; self.inner.update_request_generator_state(maximum_number_of_rooms)?; Ok(result) @@ -234,7 +234,7 @@ pub(super) struct SlidingSyncListInner { /// Which [`SlidingSyncMode`] to start this list under. sync_mode: SlidingSyncMode, - /// Sort the rooms list by this. + /// Sort the room list by this. sort: Vec, /// Required states to return per room. @@ -256,7 +256,7 @@ pub(super) struct SlidingSyncListInner { maximum_number_of_rooms: StdRwLock>>, /// The rooms in order. - rooms_list: StdRwLock>, + room_list: StdRwLock>, /// The ranges windows of the list. ranges: StdRwLock>>, @@ -375,9 +375,9 @@ impl SlidingSyncListInner { }) } - // Update the [`Self::rooms_list`]. It also updates + // Update the [`Self::room_list`]. It also updates // `[Self::maximum_number_of_rooms]`. - fn update_rooms_list( + fn update_room_list( &self, maximum_number_of_rooms: u32, sync_operations: &[v4::SyncOp], @@ -387,10 +387,10 @@ impl SlidingSyncListInner { // Adjust room list entries. { let number_of_missing_rooms = (maximum_number_of_rooms as usize) - .saturating_sub(self.rooms_list.read().unwrap().len()); + .saturating_sub(self.room_list.read().unwrap().len()); if number_of_missing_rooms > 0 { - self.rooms_list.write().unwrap().append( + self.room_list.write().unwrap().append( iter::repeat(RoomListEntry::Empty).take(number_of_missing_rooms).collect(), ); @@ -413,9 +413,9 @@ impl SlidingSyncListInner { // Run the sync operations. if !sync_operations.is_empty() { - let mut rooms_list = self.rooms_list.write().unwrap(); + let mut room_list = self.room_list.write().unwrap(); - apply_sync_operations(sync_operations, &mut rooms_list)?; + apply_sync_operations(sync_operations, &mut room_list)?; new_changes = true; } @@ -514,7 +514,7 @@ impl SlidingSyncListInner { #[instrument(skip(operations))] fn apply_sync_operations( operations: &[v4::SyncOp], - rooms_list: &mut ObservableVector, + room_list: &mut ObservableVector, ) -> Result<(), Error> { for operation in operations { match &operation.op { @@ -540,20 +540,20 @@ fn apply_sync_operations( // The `end` bound of the range might not be correct… At the time of writing, // there is a bug in the Sliding Sync Proxy that can return - // ranges greater than the `rooms_list` size. + // ranges greater than the `room_list` size. // // For example, if the client asks for a range `0..=9`, and there is only one // room, the server will reply with one `room_id` (which is correct) but with // the range `0..=9` instead of `0..=0`. // // So, a safe workaround is to take the minimum between `end` and the - // `rooms_list`'s length. + // `room_list`'s length. // // The “safety” is ensured by the fact we also compare the size of the new range // with the size of the `operation.room_ids` length later on. // // See https://github.com/matrix-org/sliding-sync/issues/52. - end = min(end, rooms_list.len()); + end = min(end, room_list.len()); // Range is invalid. if start > end { @@ -577,13 +577,13 @@ fn apply_sync_operations( )); } - // Update parts `rooms_list`. + // Update parts `room_list`. // // The room entry index is given by the `room_entry_range` bounds. // The room ID is given by the `room_ids`. for (room_entry_index, room_id) in room_entry_range.zip(room_ids) { // Syncing means updating the room list to `Filled`. - rooms_list.set(room_entry_index, RoomListEntry::Filled(room_id.clone())); + room_list.set(room_entry_index, RoomListEntry::Filled(room_id.clone())); } } @@ -603,7 +603,7 @@ fn apply_sync_operations( .unwrap(); // Index is out of bounds. - if index >= rooms_list.len() { + if index >= room_list.len() { // OK, so, normally, we should raise an error. But the server sometimes sends a // `DELETE` for an index that doesn't exist. It happens with the existing // Sliding Sync Proxy (at the time of writing). It may be a bug or something @@ -612,7 +612,7 @@ fn apply_sync_operations( } // Removing the entry in the room list. - rooms_list.remove(index); + room_list.remove(index); } // Specification says: @@ -637,16 +637,16 @@ fn apply_sync_operations( })?; // Index is out of bounds. - if index > rooms_list.len() { + if index > room_list.len() { return Err(Error::BadResponse(format!( - "`index` is out of the `rooms_list`' bounds ({} > {})", + "`index` is out of the `room_list`' bounds ({} > {})", index, - rooms_list.len(), + room_list.len(), ))); } // Inserting a `Filled` entry in the room list . - rooms_list.insert(index, RoomListEntry::Filled(room_id)); + room_list.insert(index, RoomListEntry::Filled(room_id)); } // Specification says: @@ -680,17 +680,17 @@ fn apply_sync_operations( } // Range is too big. - if end > rooms_list.len() { + if end > room_list.len() { return Err(Error::BadResponse(format!( - "`range` is out of the `rooms_list`' bounds ({} > {})", + "`range` is out of the `room_list`' bounds ({} > {})", end, - rooms_list.len(), + room_list.len(), ))); } let room_entry_range = start..end; - // Invalidate parts of `rooms_list`. + // Invalidate parts of `room_list`. // // The room entry index is given by the `room_entry_range` bounds. for room_entry_index in room_entry_range { @@ -698,8 +698,8 @@ fn apply_sync_operations( // // If the previous room list entry is `Filled`, it becomes `Invalidated`. // Otherwise, for `Empty` or `Invalidated`, it stays as is. - if let Some(RoomListEntry::Filled(room_id)) = rooms_list.get(room_entry_index) { - rooms_list + if let Some(RoomListEntry::Filled(room_id)) = room_list.get(room_entry_index) { + room_list .set(room_entry_index, RoomListEntry::Invalidated(room_id.clone())); } } @@ -847,7 +847,7 @@ mod tests { name: "qux".to_owned(), state: StdRwLock::new(Observable::new(SlidingSyncState::FullyLoaded)), maximum_number_of_rooms: StdRwLock::new(Observable::new(Some(11))), - rooms_list: StdRwLock::new(ObservableVector::from(vector![RoomListEntry::Empty])), + room_list: StdRwLock::new(ObservableVector::from(vector![RoomListEntry::Empty])), ranges: StdRwLock::new(Observable::new(vec![(uint!(0), uint!(9))])), request_generator: StdRwLock::new(SlidingSyncListRequestGenerator::new_growing( 42, @@ -874,7 +874,7 @@ mod tests { assert_eq!(*Observable::get(&new_list.state.read().unwrap()), SlidingSyncState::NotLoaded); assert!(new_list.maximum_number_of_rooms.read().unwrap().deref().is_none()); - assert!(new_list.rooms_list.read().unwrap().deref().is_empty()); + assert!(new_list.room_list.read().unwrap().deref().is_empty()); } #[test] @@ -1471,7 +1471,7 @@ mod tests { .unwrap(); assert_eq!(**list.inner.maximum_number_of_rooms.read().unwrap(), None); - assert_eq!(list.inner.rooms_list.read().unwrap().len(), 0); + assert_eq!(list.inner.room_list.read().unwrap().len(), 0); let room0 = room_id!("!room0:bar.org"); let room1 = room_id!("!room1:bar.org"); @@ -1499,12 +1499,12 @@ mod tests { // The `maximum_number_of_rooms` has been updated as expected. assert_eq!(**list.inner.maximum_number_of_rooms.read().unwrap(), Some(5)); - // The `rooms_list` has the correct size and contains expected room entries. - let rooms_list = list.inner.rooms_list.read().unwrap(); + // The `room_list` has the correct size and contains expected room entries. + let room_list = list.inner.room_list.read().unwrap(); - assert_eq!(rooms_list.len(), 5); + assert_eq!(room_list.len(), 5); assert_eq!( - **rooms_list, + **room_list, vector![ RoomListEntry::Filled(room0.to_owned()), RoomListEntry::Filled(room1.to_owned()), @@ -1535,12 +1535,12 @@ mod tests { // The `maximum_number_of_rooms` has been updated as expected. assert_eq!(**list.inner.maximum_number_of_rooms.read().unwrap(), Some(5)); - // The `rooms_list` has the correct size and contains expected room entries. - let rooms_list = list.inner.rooms_list.read().unwrap(); + // The `room_list` has the correct size and contains expected room entries. + let room_list = list.inner.room_list.read().unwrap(); - assert_eq!(rooms_list.len(), 5); + assert_eq!(room_list.len(), 5); assert_eq!( - **rooms_list, + **room_list, vector![ RoomListEntry::Filled(room0.to_owned()), RoomListEntry::Filled(room1.to_owned()), @@ -1592,7 +1592,7 @@ mod tests { macro_rules! assert_sync_operations { ( - rooms_list = [ $( $room_list_entries:tt )* ], + room_list = [ $( $room_list_entries:tt )* ], operations = [ $( { $( $operation:tt )+ } @@ -1602,10 +1602,10 @@ mod tests { $(,)? => result = $result:tt, - rooms_list = [ $( $expected_room_list_entries:tt )* ] + room_list = [ $( $expected_room_list_entries:tt )* ] $(,)? ) => { - let mut rooms_list = ObservableVector::from(entries![ $( $room_list_entries )* ]); + let mut room_list = ObservableVector::from(entries![ $( $room_list_entries )* ]); let operations: &[v4::SyncOp] = &[ $( serde_json::from_value(json!({ @@ -1614,18 +1614,18 @@ mod tests { ),* ]; - let result = apply_sync_operations(operations, &mut rooms_list); + let result = apply_sync_operations(operations, &mut room_list); assert!(result.$result()); - assert_eq!(*rooms_list, entries![ $( $expected_room_list_entries )* ]); + assert_eq!(*room_list, entries![ $( $expected_room_list_entries )* ]); }; } #[test] fn test_sync_operations_sync() { - // All rooms list is updated. + // All room list is updated. assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Sync, @@ -1635,12 +1635,12 @@ mod tests { ] => result = is_ok, - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], }; // Partial update. assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Sync, @@ -1650,10 +1650,10 @@ mod tests { ] => result = is_ok, - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), E], + room_list = [F("!r0:x.y"), F("!r1:x.y"), E], }; assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Sync, @@ -1663,13 +1663,13 @@ mod tests { ] => result = is_ok, - rooms_list = [E, F("!r1:x.y"), F("!r2:x.y")], + room_list = [E, F("!r1:x.y"), F("!r2:x.y")], }; // The range returned by the server is too large compared to the `room_ids` but // we can fix it on-the-fly. assert_sync_operations! { - rooms_list = [E], + room_list = [E], operations = [ { "op": SlidingOp::Sync, @@ -1679,12 +1679,12 @@ mod tests { ] => result = is_ok, // <- because we have fixed it - rooms_list = [F("!r0:x.y")], + room_list = [F("!r0:x.y")], }; // Missing `range`. assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Sync, @@ -1693,12 +1693,12 @@ mod tests { ] => result = is_err, - rooms_list = [E, E, E], + room_list = [E, E, E], }; // Invalid `range`. assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Sync, @@ -1708,12 +1708,12 @@ mod tests { ] => result = is_err, - rooms_list = [E, E, E], + room_list = [E, E, E], }; // Missing `room_ids`. assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Sync, @@ -1722,12 +1722,12 @@ mod tests { ] => result = is_err, - rooms_list = [E, E, E], + room_list = [E, E, E], }; // Out of bounds operation. assert_sync_operations! { - rooms_list = [E, F("!r1:x.y"), E], + room_list = [E, F("!r1:x.y"), E], operations = [ { "op": SlidingOp::Sync, @@ -1737,13 +1737,13 @@ mod tests { ] => result = is_err, - rooms_list = [E, F("!r1:x.y"), E], + room_list = [E, F("!r1:x.y"), E], }; // The server replies with a particular range, but some room IDs are // missing. assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Sync, @@ -1753,13 +1753,13 @@ mod tests { ] => result = is_err, - rooms_list = [E, E, E], + room_list = [E, E, E], }; // The server replies with a particular range, but there is too much // room IDs. assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Sync, @@ -1769,7 +1769,7 @@ mod tests { ] => result = is_err, - rooms_list = [E, E, E], + room_list = [E, E, E], }; } @@ -1777,7 +1777,7 @@ mod tests { fn test_sync_operations_delete() { // Delete a room entry in the middle. assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Delete, @@ -1786,12 +1786,12 @@ mod tests { ] => result = is_ok, - rooms_list = [F("!r0:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r2:x.y")], }; // Delete a room entry at the beginning. assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Delete, @@ -1800,12 +1800,12 @@ mod tests { ] => result = is_ok, - rooms_list = [F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r1:x.y"), F("!r2:x.y")], }; // Delete a room entry at the end. assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Delete, @@ -1814,12 +1814,12 @@ mod tests { ] => result = is_ok, - rooms_list = [F("!r0:x.y"), F("!r1:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y")], }; // Delete an out of bounds room entry. assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Delete, @@ -1828,7 +1828,7 @@ mod tests { ] => result = is_ok, // <- that's surprising, see the code for more explanations! - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], }; } @@ -1836,7 +1836,7 @@ mod tests { fn test_sync_operations_insert() { // Insert a room entry in the middle. assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Insert, @@ -1846,12 +1846,12 @@ mod tests { ] => result = is_ok, - rooms_list = [E, F("!r1:x.y"), E, E], + room_list = [E, F("!r1:x.y"), E, E], }; // Insert a room entry at the beginning. assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Insert, @@ -1861,12 +1861,12 @@ mod tests { ] => result = is_ok, - rooms_list = [F("!r0:x.y"), E, E, E], + room_list = [F("!r0:x.y"), E, E, E], }; // Insert a room entry at the end assert_sync_operations! { - rooms_list = [E, E, E], + room_list = [E, E, E], operations = [ { "op": SlidingOp::Insert, @@ -1876,12 +1876,12 @@ mod tests { ] => result = is_ok, - rooms_list = [E, E, E, F("!r3:x.y")], + room_list = [E, E, E, F("!r3:x.y")], }; // Insert an out of bounds room entry. assert_sync_operations! { - rooms_list = [E, F("!r1:x.y"), E], + room_list = [E, F("!r1:x.y"), E], operations = [ { "op": SlidingOp::Insert, @@ -1891,7 +1891,7 @@ mod tests { ] => result = is_err, - rooms_list = [E, F("!r1:x.y"), E], + room_list = [E, F("!r1:x.y"), E], }; } @@ -1899,7 +1899,7 @@ mod tests { fn test_sync_operations_invalidate() { // Invalidating an empty room. assert_sync_operations! { - rooms_list = [E], + room_list = [E], operations = [ { "op": SlidingOp::Invalidate, @@ -1908,12 +1908,12 @@ mod tests { ] => result = is_ok, - rooms_list = [E], + room_list = [E], }; // Invalidating a filled room. assert_sync_operations! { - rooms_list = [F("!r0:x.y")], + room_list = [F("!r0:x.y")], operations = [ { "op": SlidingOp::Invalidate, @@ -1922,12 +1922,12 @@ mod tests { ] => result = is_ok, - rooms_list = [I("!r0:x.y")], + room_list = [I("!r0:x.y")], }; // Invalidating an invalidated room. assert_sync_operations! { - rooms_list = [I("!r0:x.y")], + room_list = [I("!r0:x.y")], operations = [ { "op": SlidingOp::Invalidate, @@ -1936,12 +1936,12 @@ mod tests { ] => result = is_ok, - rooms_list = [I("!r0:x.y")], + room_list = [I("!r0:x.y")], }; // Partial update. assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Invalidate, @@ -1950,10 +1950,10 @@ mod tests { ] => result = is_ok, - rooms_list = [I("!r0:x.y"), I("!r1:x.y"), F("!r2:x.y")], + room_list = [I("!r0:x.y"), I("!r1:x.y"), F("!r2:x.y")], }; assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Invalidate, @@ -1962,12 +1962,12 @@ mod tests { ] => result = is_ok, - rooms_list = [F("!r0:x.y"), I("!r1:x.y"), I("!r2:x.y")], + room_list = [F("!r0:x.y"), I("!r1:x.y"), I("!r2:x.y")], }; // Full update. assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Invalidate, @@ -1976,12 +1976,12 @@ mod tests { ] => result = is_ok, - rooms_list = [I("!r0:x.y"), I("!r1:x.y"), I("!r2:x.y")], + room_list = [I("!r0:x.y"), I("!r1:x.y"), I("!r2:x.y")], }; // Missing `range`. assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Delete, @@ -1989,12 +1989,12 @@ mod tests { ] => result = is_err, - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], }; // Invalid `range`. assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Delete, @@ -2003,12 +2003,12 @@ mod tests { ] => result = is_err, - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], }; // Out of bounds operation. assert_sync_operations! { - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], operations = [ { "op": SlidingOp::Delete, @@ -2017,7 +2017,7 @@ mod tests { ] => result = is_err, - rooms_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], + room_list = [F("!r0:x.y"), F("!r1:x.y"), F("!r2:x.y")], }; } } diff --git a/testing/sliding-sync-integration-test/src/lib.rs b/testing/sliding-sync-integration-test/src/lib.rs index 9eb25d63a..78628a45f 100644 --- a/testing/sliding-sync-integration-test/src/lib.rs +++ b/testing/sliding-sync-integration-test/src/lib.rs @@ -138,7 +138,7 @@ async fn modifying_timeline_limit() -> anyhow::Result<()> { let room_id = update_summary.rooms[0].clone(); // Let's fetch the room ID from the list too. - assert_matches!(list.rooms_list().get(0), Some(RoomListEntry::Filled(same_room_id)) => { + assert_matches!(list.room_list().get(0), Some(RoomListEntry::Filled(same_room_id)) => { assert_eq!(same_room_id, &room_id); }); @@ -166,7 +166,7 @@ async fn modifying_timeline_limit() -> anyhow::Result<()> { .add_list( SlidingSyncList::builder() .sync_mode(SlidingSyncMode::Selective) - .name("visible_rooms_list") + .name("visible_room_list") .add_range(0u32, 1) .timeline_limit(1u32) .build()?, @@ -179,7 +179,7 @@ async fn modifying_timeline_limit() -> anyhow::Result<()> { pin_mut!(stream); // Get the list. - let list = sync.list("visible_rooms_list").context("list `visible_rooms_list` isn't found")?; + let list = sync.list("visible_room_list").context("list `visible_room_list` isn't found")?; let mut all_event_ids = Vec::new(); @@ -252,7 +252,7 @@ async fn modifying_timeline_limit() -> anyhow::Result<()> { assert_eq!(room_id, update_summary.rooms[0]); // Let's fetch the room ID from the list too. - assert_matches!(list.rooms_list().get(0), Some(RoomListEntry::Filled(same_room_id)) => { + assert_matches!(list.room_list().get(0), Some(RoomListEntry::Filled(same_room_id)) => { assert_eq!(same_room_id, &room_id); }); @@ -370,7 +370,7 @@ async fn adding_list_later() -> anyhow::Result<()> { assert!(saw_update, "We didn't see the update come through the pipe"); // and let's update the order of all lists again - let room_id = assert_matches!(list1.rooms_list().get(4), Some(RoomListEntry::Filled(room_id)) => room_id.clone()); + let room_id = assert_matches!(list1.room_list().get(4), Some(RoomListEntry::Filled(room_id)) => room_id.clone()); let room = client.get_joined_room(&room_id).context("No joined room {room_id}")?; @@ -452,7 +452,7 @@ async fn live_lists() -> anyhow::Result<()> { // Let's trigger an update by sending a message to room pos=3, making it move to // pos 0 - let room_id = assert_matches!(list1.rooms_list().get(3), Some(RoomListEntry::Filled(room_id)) => room_id.clone()); + let room_id = assert_matches!(list1.room_list().get(3), Some(RoomListEntry::Filled(room_id)) => room_id.clone()); let Some(room) = client.get_joined_room(&room_id) else { bail!("No joined room {room_id}"); @@ -486,7 +486,7 @@ async fn live_lists() -> anyhow::Result<()> { pin_mut!(stream); // and let's update the order of all lists again - let room_id = assert_matches!(list1.rooms_list().get(4), Some(RoomListEntry::Filled(room_id)) => room_id.clone()); + let room_id = assert_matches!(list1.room_list().get(4), Some(RoomListEntry::Filled(room_id)) => room_id.clone()); let Some(room) = client.get_joined_room(&room_id) else { bail!("No joined room {room_id}"); @@ -558,10 +558,10 @@ async fn list_goes_live() -> anyhow::Result<()> { let _room_summary = stream.next().await.context("No room summary found, loop ended unsuccessfully")??; - let rooms_list = full_list.rooms_list::(); + let room_list = full_list.room_list::(); assert_eq!( - rooms_list, + room_list, repeat(RoomListEntryEasy::Filled) .take(20) .chain(once(RoomListEntryEasy::Empty)) @@ -573,9 +573,9 @@ async fn list_goes_live() -> anyhow::Result<()> { let _room_summary = stream.next().await.context("No room summary found, loop ended unsecessfully")??; - let rooms_list = full_list.rooms_list::(); + let room_list = full_list.room_list::(); - assert_eq!(rooms_list, repeat(RoomListEntryEasy::Filled).take(21).collect::>()); + assert_eq!(room_list, repeat(RoomListEntryEasy::Filled).take(21).collect::>()); assert_eq!(full_list.state(), SlidingSyncState::FullyLoaded, "full isn't fully loaded"); Ok(()) @@ -600,7 +600,7 @@ async fn resizing_sliding_window() -> anyhow::Result<()> { // we only heard about the ones we had asked for assert_eq!(summary.rooms.len(), 11); - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -610,7 +610,7 @@ async fn resizing_sliding_window() -> anyhow::Result<()> { .collect::>() ); - let _signal = list.rooms_list_stream(); + let _signal = list.room_list_stream(); // let's move the window @@ -626,7 +626,7 @@ async fn resizing_sliding_window() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -648,7 +648,7 @@ async fn resizing_sliding_window() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -672,7 +672,7 @@ async fn resizing_sliding_window() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -703,7 +703,7 @@ async fn moving_out_of_sliding_window() -> anyhow::Result<()> { let summary = room_summary?; // we only heard about the ones we had asked for assert_eq!(summary.rooms.len(), 10); - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -714,7 +714,7 @@ async fn moving_out_of_sliding_window() -> anyhow::Result<()> { .collect::>() ); - let _signal = list.rooms_list_stream(); + let _signal = list.room_list_stream(); // let's move the window @@ -729,7 +729,7 @@ async fn moving_out_of_sliding_window() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -752,7 +752,7 @@ async fn moving_out_of_sliding_window() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -766,7 +766,7 @@ async fn moving_out_of_sliding_window() -> anyhow::Result<()> { // now we "move" the room of pos 3 to pos 0; // this is a bordering case - let room_id = assert_matches!(list.rooms_list().get(3), Some(RoomListEntry::Filled(room_id)) => room_id.clone()); + let room_id = assert_matches!(list.room_list().get(3), Some(RoomListEntry::Filled(room_id)) => room_id.clone()); let room = client.get_joined_room(&room_id).context("No joined room {room_id}")?; @@ -783,7 +783,7 @@ async fn moving_out_of_sliding_window() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -795,7 +795,7 @@ async fn moving_out_of_sliding_window() -> anyhow::Result<()> { ); // items has moved, thus we shouldn't find it where it was - assert!(list.rooms_list::().get(3).unwrap().as_room_id().unwrap() != room_id); + assert!(list.room_list::().get(3).unwrap().as_room_id().unwrap() != room_id); // let's move the window again @@ -810,7 +810,7 @@ async fn moving_out_of_sliding_window() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -822,7 +822,7 @@ async fn moving_out_of_sliding_window() -> anyhow::Result<()> { ); // and check that our room move has been accepted properly, too. - assert_eq!(list.rooms_list::().get(0).unwrap().as_room_id().unwrap(), &room_id); + assert_eq!(list.room_list::().get(0).unwrap().as_room_id().unwrap(), &room_id); Ok(()) } @@ -916,7 +916,7 @@ async fn growing_sync_keeps_going() -> anyhow::Result<()> { let _summary = room_summary?; } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -930,7 +930,7 @@ async fn growing_sync_keeps_going() -> anyhow::Result<()> { let room_summary = stream.next().await.context("sync has closed unexpectedly")?; let _summary = room_summary?; - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!(collection_simple, repeat(RoomListEntryEasy::Filled).take(20).collect::>()); @@ -969,7 +969,7 @@ async fn continue_on_reset() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple.iter().fold(0, |acc, i| if *i == RoomListEntryEasy::Filled { @@ -1008,7 +1008,7 @@ async fn continue_on_reset() -> anyhow::Result<()> { assert!(error_seen, "We have not seen the UnknownPos error"); - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple.iter().fold(0, |acc, i| if *i == RoomListEntryEasy::Filled { @@ -1051,7 +1051,7 @@ async fn noticing_new_rooms_in_growing() -> anyhow::Result<()> { stream.next().await.context("No room summary found, loop ended unsuccessfully")??; } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple.iter().fold(0, |acc, i| if *i == RoomListEntryEasy::Filled { @@ -1084,7 +1084,7 @@ async fn noticing_new_rooms_in_growing() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple.iter().fold(0, |acc, i| if *i == RoomListEntryEasy::Filled { @@ -1125,11 +1125,11 @@ async fn restart_room_resubscription() -> anyhow::Result<()> { // we only heard about the ones we had asked for assert_eq!(room_summary.rooms.len(), 3); - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!(collection_simple, repeat(RoomListEntryEasy::Filled).take(3).collect::>()); - let _signal = list.rooms_list_stream(); + let _signal = list.room_list_stream(); // let's move the window @@ -1144,7 +1144,7 @@ async fn restart_room_resubscription() -> anyhow::Result<()> { } } - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple, @@ -1156,7 +1156,7 @@ async fn restart_room_resubscription() -> anyhow::Result<()> { // let's get that first entry - let room_id = assert_matches!(list.rooms_list().get(0), Some(RoomListEntry::Invalidated(room_id)) => room_id.clone()); + let room_id = assert_matches!(list.room_list().get(0), Some(RoomListEntry::Invalidated(room_id)) => room_id.clone()); // send a message @@ -1241,7 +1241,7 @@ async fn restart_room_resubscription() -> anyhow::Result<()> { let sliding_sync_room = sync_proxy.get_room(&room_id).expect("Slidin Sync room not found"); let event = sliding_sync_room.latest_event().await.expect("No even found"); - let collection_simple = list.rooms_list::(); + let collection_simple = list.room_list::(); assert_eq!( collection_simple,