fix: fix test and log errors from handle_message/read method in DAP loop

Author: alehander92

src/db-backend/src/dap.rs

@@ -379,6 +379,7 @@ pub fn read_dap_message_from_reader<R: std::io::BufRead>(reader: &mut R) -> DapR
         error!("Read Line: {:?}", e);
         serde_json::Error::custom(e.to_string())
     })?;
+    info!("line read");
     if !header.to_ascii_lowercase().starts_with("content-length:") {
         // println!("no content-length!");
         return Err(serde_json::Error::custom("Missing Content-Length header").into());

src/db-backend/src/dap_server.rs

@@ -446,10 +446,21 @@ fn handle_client<R: std::io::BufRead, T: DapTransport>(
 
     let mut ctx = Ctx::default();
 
-    while let Ok(msg) = dap::read_dap_message_from_reader(reader) {
-        let _ = handle_message(&msg, transport, &mut ctx);
+    loop {
+        match dap::read_dap_message_from_reader(reader) {
+            Ok(msg) => {
+                let res = handle_message(&msg, transport, &mut ctx);
+                if let Err(e) = res {
+                    error!("handle_message error: {e:?}");
+                }
+            },
+            Err(e) => {
+                error!("error from read_dap_message_from_reader: {e:?}");
+                break;
+            }
+        }
     }
 
-    error!("maybe error from reader");
+
     Ok(())
 }

src/db-backend/src/handler.rs

@@ -1654,7 +1654,7 @@ mod tests {
 
         // Assert: Check that the Handler instance is correctly initialized
         assert_eq!(handler.step_id, StepId(0));
-        assert!(!handler.breakpoint_list.is_empty());
+        assert!(!handler.breakpoints.is_empty());
     }
 
     // Test single tracepoint