fix: filter empty lines in apt generator callbacks

command-signatures/src/generators/apt.rs

@@ -17,10 +17,12 @@ const LIST_ALL_DEB_FILES_COMMAND: &str = r#"find . -maxdepth 1 -type f -name '*.
 pub fn list_all_packages(output: &str) -> GeneratorResults {
     let mut targets = Vec::new();
     for package_name in output.lines() {
-        targets.push(Suggestion::with_description(
-            package_name.to_string(),
-            "package",
-        ));
+        if !package_name.is_empty() {
+            targets.push(Suggestion::with_description(
+                package_name.to_string(),
+                "package",
+            ));
+        }
     }
     targets.into_iter().collect_unordered_results()
 }
@@ -41,11 +43,13 @@ pub fn list_available_packages(output: &str) -> GeneratorResults {
 pub fn list_all_deb_files_in_cwd(output: &str) -> GeneratorResults {
     let mut targets = Vec::new();
     for file in output.lines() {
-        targets.push(
-            // We should prioritize .deb files over the already installed packages.
-            Suggestion::with_description(file.to_string(), ".deb file")
-                .with_priority(Priority::most_important()),
-        )
+        if !file.is_empty() {
+            targets.push(
+                // We should prioritize .deb files over the already installed packages.
+                Suggestion::with_description(file.to_string(), ".deb file")
+                    .with_priority(Priority::most_important()),
+            );
+        }
     }
     targets.into_iter().collect_unordered_results()
 }

command-signatures/src/generators/apt_tests.rs

@@ -0,0 +1,38 @@
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn test_list_all_packages_filters_empty_lines() {
+        let output = "vim\ngit\n\n";
+        let results = list_all_packages(output);
+        let suggestions: Vec<_> = results.into_iter().collect();
+        assert_eq!(suggestions.len(), 2);
+        assert_eq!(suggestions[0].name, "vim");
+        assert_eq!(suggestions[1].name, "git");
+    }
+
+    #[test]
+    fn test_list_all_packages_empty_input() {
+        let results = list_all_packages("");
+        let suggestions: Vec<_> = results.into_iter().collect();
+        assert_eq!(suggestions.len(), 0);
+    }
+
+    #[test]
+    fn test_list_all_deb_files_filters_empty_lines() {
+        let output = "./foo.deb\n./bar.deb\n\n";
+        let results = list_all_deb_files_in_cwd(output);
+        let suggestions: Vec<_> = results.into_iter().collect();
+        assert_eq!(suggestions.len(), 2);
+        assert_eq!(suggestions[0].name, "./foo.deb");
+        assert_eq!(suggestions[1].name, "./bar.deb");
+    }
+
+    #[test]
+    fn test_list_all_deb_files_empty_input() {
+        let results = list_all_deb_files_in_cwd("");
+        let suggestions: Vec<_> = results.into_iter().collect();
+        assert_eq!(suggestions.len(), 0);
+    }
+}