Optional cursors

CHANGELOG.md

@@ -4,6 +4,9 @@
 - ([#296](https://github.com/ramsayleung/rspotify/pull/296)) The `Avr` variant in `DeviceType` is actually case insensitive.
 - ([#302](https://github.com/ramsayleung/rspotify/pull/302)) Added undocumented `label` field to `FullAlbum`.
 
+**Breaking changes:**
+- ([#303](https://github.com/ramsayleung/rspotify/pull/303)) The `cursors` field in `CursorBasedPage` is now optional.
+
 ## 0.11.3 (2021.11.29)
 
 - ([#281](https://github.com/ramsayleung/rspotify/issues/281)) The documentation website for the Spotify API has been changed, so the links in the `rspotify` crate have been updated. Unfortunately, the object model section has been removed, so we've had to delete most reference links in `rspotify-model`. From now on, you should refer to the endpoints where they are used to see their definition.

rspotify-model/src/page.rs

@@ -21,7 +21,7 @@ pub struct CursorBasedPage<T> {
     pub items: Vec<T>,
     pub limit: u32,
     pub next: Option<String>,
-    pub cursors: Cursor,
+    pub cursors: Option<Cursor>,
     /// Absent if it has read all data items. This field doesn't match what
     /// Spotify document says
     pub total: Option<u32>,

tests/test_with_credential.rs

@@ -164,7 +164,7 @@ async fn test_fake_playlist() {
         .await
         .playlist(&playlist_id, None, None)
         .await;
-    assert!(!playlist.is_ok());
+    assert!(playlist.is_err());
 }
 
 mod test_pagination {

tests/test_with_oauth.rs

@@ -217,9 +217,7 @@ async fn test_current_user_saved_albums() {
         .map(|a| a.album.id)
         .collect::<Vec<_>>();
     assert!(
-        album_ids
-            .iter()
-            .all(|item| all_uris.contains(&(*item).to_owned())),
+        album_ids.iter().all(|item| all_uris.contains(item)),
         "couldn't find the new saved albums"
     );