Remove debug statement and add tests

src/app.rs

@@ -2586,11 +2586,7 @@ impl App {
     fn refresh_selection(mut self) -> Result<Self> {
         // Should be able to select broken symlink
         self.selection.retain(|n| {
-            let s = PathBuf::from(&n.absolute_path).symlink_metadata().is_ok();
-            if !s {
-                println!("{:?}", &n);
-            }
-            s
+            PathBuf::from(&n.absolute_path).symlink_metadata().is_ok()
         });
         Ok(self)
     }

src/bin/xplr.rs

@@ -67,6 +67,16 @@ fn main() {
 mod tests {
     use assert_cmd::Command;
 
+    #[test]
+    fn test_no_debug_in_lib() {
+        for pat in ["print!", "println!"].iter() {
+            Command::new("grep")
+                .args(&["-R", pat, "src", "--exclude-dir", "bin/"])
+                .assert()
+                .failure();
+        }
+    }
+
     #[test]
     fn test_cli_version() {
         Command::cargo_bin("xplr")

src/lib.rs

@@ -14,3 +14,15 @@ pub mod pipe_reader;
 pub mod pwd_watcher;
 pub mod runner;
 pub mod ui;
+
+#[cfg(test)]
+mod tests {
+
+    use super::*;
+
+    #[test]
+    fn test_upgrade_guide_has_latest_version() {
+        let guide = include_str!("../docs/en/src/upgrade-guide.md");
+        assert!(guide.contains(app::VERSION));
+    }
+}

src/lua.rs

@@ -161,10 +161,4 @@ mod tests {
         // Next bugfix release is ERR (Change when we get to v1)
         assert!(check_version("0.15.3", "foo path").is_err());
     }
-
-    #[test]
-    fn test_upgrade_guide_has_latest_version() {
-        let guide = include_str!("../docs/en/src/upgrade-guide.md");
-        assert!(guide.contains(VERSION));
-    }
 }