Add page size test cases

p2panda · Jun 14, 2024 · 8ac247b · 8ac247b
1 parent 5f9e737
commit 8ac247b
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/aquadoggo/src/graphql/queries/collection.rs b/aquadoggo/src/graphql/queries/collection.rs
@@ -1519,7 +1519,6 @@ mod tests {
             }
 
             // We should have received the same number of documents as the total count.
-            println!("{documents:#?}");
             assert_eq!(acc, total_count);
         })
     }
@@ -1565,16 +1564,18 @@ mod tests {
     }
 
     #[rstest]
-    fn vanishing_document_in_paginated_query_regression_test(key_pair: KeyPair) {
-        test_runner(|mut node: TestNode| async move {
+    fn incorrect_number_of_documents_in_paginated_query_regression_test(
+        #[values(1, 2, 3, 4, 5, 6, 7, 8, 9, 10)] page_size: usize,
+        key_pair: KeyPair,
+    ) {
+        test_runner(move |mut node: TestNode| async move {
             // Publish some lyrics to the node.
             let (lyric_schema, _) = here_be_some_lyrics(&mut node, &key_pair).await;
 
             // Init a GraphQL client we'll use to query the node.
             let client = http_test_client(&node).await;
 
             // We're making paginated queries, sorting by line in ascending order.
-            let page_size = 3; // @TODO: some page sizes don't fail here.
             let order_by = String::from("line");
             let order_direction = String::from("ASC");
             let mut acc = 0;
@@ -1621,19 +1622,20 @@ mod tests {
     }
 
     #[rstest]
-    fn incorrect_ordering_regression_test(key_pair: KeyPair) {
-        // @TODO: this test does not fail yet... We have observed a bug in the wild where documents in an ordered collection 
+    fn incorrect_ordering_regression_test(
+        #[values(1, 2, 3, 4, 5, 6, 7, 8, 9, 10)] page_size: usize,
+        key_pair: KeyPair) {
+        // @TODO: this test does not fail yet... We have observed a bug in the wild where documents in an ordered collection
         // query are returned out of order. This is a so far unsuccessful attempt to reproduce the bug in tests.
 
-        test_runner(|mut node: TestNode| async move {
+        test_runner(move |mut node: TestNode| async move {
             // Publish some lyrics to the node.
             let (lyric_schema, _) = here_be_some_lyrics(&mut node, &key_pair).await;
 
             // Init a GraphQL client we'll use to query the node.
             let client = http_test_client(&node).await;
 
             // We're making paginated queries, sorting by index in ascending order.
-            let page_size = 3;
             let order_by = String::from("index");
             let order_direction = String::from("ASC");
             let mut documents = Vec::new();