Fix: on range=1 without - it means from and not to this value

Author: nichtsfrei

rust/crates/greenbone-scanner-framework/src/get_scans_id_results.rs

@@ -83,11 +83,7 @@ where
                             Some(x) => x.parse().ok(),
                             None => None,
                         };
-                        if from.is_some() && to.is_none() {
-                            (None, from)
-                        } else {
-                            (from, to)
-                        }
+                        (from, to)
                     })
                     .next()
                     .unwrap_or_default(),
@@ -174,8 +170,13 @@ mod tests {
                     io::Error::other("oh no"),
                 )))]));
             }
+            let to = if from.is_some() {
+                // for the test case ?range=99
+                to.unwrap_or(101)
+            } else {
+                to.unwrap_or_default()
+            };
             let from = from.unwrap_or_default();
-            let to = to.unwrap_or_default();
             let result: Vec<Result<models::Result, _>> = (from..to)
                 .map(|id| {
                     Ok(models::Result {
@@ -248,23 +249,23 @@ mod tests {
     }
 
     #[tokio::test]
-    async fn scan_results_to() {
+    async fn scan_results_from() {
         let entry_point = test_utilities::entry_point(
             Authentication::MTLS,
             create_single_handler!(GetScansIdResultsHandler::from(Test {})),
             Some(ClientHash::from("ok")),
         );
 
         let req = Request::builder()
-            .uri("/scans/id/results?range=100")
+            .uri("/scans/id/results?range=99")
             .method(Method::GET)
             .body(Empty::<Bytes>::new())
             .unwrap();
         let resp = entry_point.call(req).await.unwrap();
         assert_eq!(resp.status(), StatusCode::OK);
         let bytes = resp.into_body().collect().await.unwrap().to_bytes();
         let resp: Vec<models::Result> = serde_json::from_slice(bytes.as_ref()).unwrap();
-        assert_eq!(resp.len(), 100);
+        assert_eq!(resp.len(), 2);
         insta::assert_ron_snapshot!(resp);
     }
 

rust/crates/greenbone-scanner-framework/src/snapshots/greenbone_scanner_framework__get_scans_id_results__tests__scan_results.snap

@@ -1,5 +1,5 @@
 ---
-source: src/get_scans_id_results.rs
+source: crates/greenbone-scanner-framework/src/get_scans_id_results.rs
 expression: resp
 ---
 []

rust/crates/greenbone-scanner-framework/src/snapshots/greenbone_scanner_framework__get_scans_id_results__tests__scan_results_from.snap

@@ -0,0 +1,14 @@
+---
+source: crates/greenbone-scanner-framework/src/get_scans_id_results.rs
+expression: resp
+---
+[
+  Result(
+    id: 99,
+    type: log,
+  ),
+  Result(
+    id: 100,
+    type: log,
+  ),
+]