chore: get rid of SlidingSyncListBuilder::set_range and replace it with add_range in tests

Signed-off-by: Benjamin Bouvier <public@benj.me>
This commit is contained in:
Benjamin Bouvier
2023-05-22 16:12:59 +02:00
parent dbd491383f
commit 9c1919ab4c
3 changed files with 7 additions and 13 deletions

View File

@@ -220,7 +220,7 @@ impl Match for SlidingSyncMatcher {
async fn test_timeline_basic() -> Result<()> {
let (server, sliding_sync) = new_sliding_sync(vec![SlidingSyncList::builder("foo")
.sync_mode(SlidingSyncMode::Selective)
.set_range(0..=10)])
.add_range(0..=10)])
.await?;
let stream = sliding_sync.sync();
@@ -267,7 +267,7 @@ async fn test_timeline_basic() -> Result<()> {
async fn test_timeline_duplicated_events() -> Result<()> {
let (server, sliding_sync) = new_sliding_sync(vec![SlidingSyncList::builder("foo")
.sync_mode(SlidingSyncMode::Selective)
.set_range(0..=10)])
.add_range(0..=10)])
.await?;
let stream = sliding_sync.sync();

View File

@@ -142,12 +142,6 @@ impl SlidingSyncListBuilder {
self
}
/// Set a single range to fetch.
pub fn set_range(mut self, range: RangeInclusive<Bound>) -> Self {
self.ranges = vec![range];
self
}
/// Set the ranges to fetch.
pub fn add_range(mut self, range: RangeInclusive<Bound>) -> Self {
self.ranges.push(range);

View File

@@ -791,7 +791,7 @@ mod tests {
async fn test_subscribe_to_room() -> Result<()> {
let (_server, sliding_sync) = new_sliding_sync(vec![SlidingSyncList::builder("foo")
.sync_mode(SlidingSyncMode::Selective)
.set_range(0..=10)])
.add_range(0..=10)])
.await?;
let _stream = sliding_sync.sync();
@@ -839,7 +839,7 @@ mod tests {
async fn test_to_device_token_properly_cached() -> Result<()> {
let (_server, sliding_sync) = new_sliding_sync(vec![SlidingSyncList::builder("foo")
.sync_mode(SlidingSyncMode::Selective)
.set_range(0..=10)])
.add_range(0..=10)])
.await?;
// When no to-device token is present, `prepare_extensions_config` doesn't fill
@@ -876,7 +876,7 @@ mod tests {
async fn test_add_list() -> Result<()> {
let (_server, sliding_sync) = new_sliding_sync(vec![SlidingSyncList::builder("foo")
.sync_mode(SlidingSyncMode::Selective)
.set_range(0..=10)])
.add_range(0..=10)])
.await?;
let _stream = sliding_sync.sync();
@@ -886,7 +886,7 @@ mod tests {
.add_list(
SlidingSyncList::builder("bar")
.sync_mode(SlidingSyncMode::Selective)
.set_range(50..=60),
.add_range(50..=60),
)
.await?;
@@ -904,7 +904,7 @@ mod tests {
async fn test_stop_sync_loop() -> Result<()> {
let (_server, sliding_sync) = new_sliding_sync(vec![SlidingSyncList::builder("foo")
.sync_mode(SlidingSyncMode::Selective)
.set_range(0..=10)])
.add_range(0..=10)])
.await?;
let stream = sliding_sync.sync();