bug fix on analyze without --sqlite and --runid flags

Author: JGsouzaa

src/analyze.rs

@@ -155,7 +155,9 @@ pub async fn analyze_folder(
             }
             num_killed += 1
         }
-        update_command_to_test_mutant(&test_command, &file_path, db_path.clone(), run_id.clone().unwrap())?;
+        if let Some(db_path) = db_path.clone() {
+            update_command_to_test_mutant(&test_command, &file_path, db_path, run_id.clone().unwrap_or_default())?;
+        }
     }
 
     // Generate report

src/main.rs

@@ -210,7 +210,7 @@ async fn main() -> Result<()> {
             runid,
         } => {
 
-            let db_path = match sqlite {
+            let db_path = match sqlite.clone() {
                 Some(Some(path)) => {
                     let mut full_path = PathBuf::from("db");
                     full_path.push(path);
@@ -220,16 +220,18 @@ async fn main() -> Result<()> {
                 None => None,
             };
 
-            if runid.is_none() {
-                return Err(MutationError::InvalidInput(
-                    "--sqlite requires --runid".to_string(),
-                ));
-            }
+            if sqlite.is_some() {
+                if runid.is_none() {
+                    return Err(MutationError::InvalidInput(
+                        "--sqlite requires --runid".to_string(),
+                    ));
+                }
 
-            if runid.is_some() && db_path.is_none() {
-                return Err(MutationError::InvalidInput(
-                    "--runid requires --sqlite".to_string(),
-                ));
+                if runid.is_some() && db_path.is_none() {
+                    return Err(MutationError::InvalidInput(
+                        "--runid requires --sqlite".to_string(),
+                    ));
+                }
             }
 
             analyze::run_analysis(folder, command, jobs, timeout, survival_threshold, db_path, runid).await?;

src/sqlite.rs

@@ -22,11 +22,10 @@ where
 pub fn update_command_to_test_mutant(
     command: &str,
     fullpath: &PathBuf,
-    db_path: Option<PathBuf>,
+    db_path: PathBuf,
     run_id: i64,
     ) -> Result<(), MutationError>{ 
 
-    let db_path = db_path.ok_or(MutationError::MissingDbPath)?;
     let connection = Connection::open(db_path.clone())?;
     let fullpath = fullpath.strip_prefix("./").unwrap_or(fullpath);
 
@@ -563,7 +562,7 @@ mod tests {
         println!("count: {:?}", count);
         assert_eq!(count, 1, "Must exist exactly 1 mutant");
 
-        let result = update_command_to_test_mutant("command", file_path, Some(db_path.to_path_buf()), run_id);
+        let result = update_command_to_test_mutant("command", file_path, db_path.to_path_buf(), run_id);
         assert!(result.is_ok());
 
         let proj_query_row: (i32, String, String) = connection.query_row(