SearchPersonResponse make scroll_token optional (#29)

* SearchPersonResponse make scroll_token optional The response from the API, in the event of an error, does not contain a scroll token. This causes a failure to deserialize when an error response is received. * update version * update test * remove old test --------- Co-authored-by: vvillait88 <vvillait88@yahoo.com>
peopledatalabs · Jun 12, 2024 · 93e7063 · 93e7063
1 parent 7b306c6
commit 93e7063
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "peopledatalabs"
-version = "2.0.1"
+version = "2.1.0"
 edition = "2021"
 description = "A Rust client for the People Data Labs API"
 documentation = "https://docs.peopledatalabs.com/docs/rust-sdk"

diff --git a/src/api/person.rs b/src/api/person.rs
@@ -300,7 +300,7 @@ mod tests {
 
         assert_eq!(resp.status, 200);
         assert_eq!(resp.data.unwrap().len(), num_results);
-        assert_eq!(resp.scroll_token.is_empty(), false);
+        assert_eq!(resp.scroll_token.is_some() && !resp.scroll_token.as_ref().unwrap().is_empty(), true);
     }
 
     #[test]

diff --git a/src/models/person.rs b/src/models/person.rs
@@ -363,7 +363,7 @@ pub struct SearchPersonResponse {
 
     pub data: Option<Vec<Person>>,
     pub total: i32,
-    pub scroll_token: String,
+    pub scroll_token: Option<String>,
 }
 
 #[derive(Debug, Serialize, Deserialize)]