Added more tests, now 119

Author: trypsynth

src/parser/path.rs

@@ -25,4 +25,21 @@ mod tests {
 		assert_eq!(extract_title_from_path("  spaced.txt  "), "spaced");
 		assert_eq!(extract_title_from_path(""), "Untitled");
 	}
+
+	#[test]
+	fn extracts_title_from_paths_without_extension() {
+		assert_eq!(extract_title_from_path("README"), "README");
+		assert_eq!(extract_title_from_path("/var/log/system"), "system");
+	}
+
+	#[test]
+	fn returns_untitled_for_whitespace_only_or_trailing_separator() {
+		assert_eq!(extract_title_from_path("   "), "Untitled");
+		assert_eq!(extract_title_from_path(" /tmp/dir/ "), "Untitled");
+	}
+
+	#[test]
+	fn extracts_title_from_multi_dot_filename() {
+		assert_eq!(extract_title_from_path("archive.tar.gz"), "archive.tar");
+	}
 }

src/update.rs

@@ -203,6 +203,21 @@ mod tests {
 		assert_eq!(parse_semver_value("v"), None);
 	}
 
+	#[test]
+	fn parse_semver_trims_whitespace() {
+		assert_eq!(parse_semver_value("  v2.3.4  "), Some((2, 3, 4)));
+	}
+
+	#[test]
+	fn parse_semver_ignores_extra_segments_after_patch() {
+		assert_eq!(parse_semver_value("1.2.3.99"), Some((1, 2, 3)));
+	}
+
+	#[test]
+	fn parse_semver_rejects_missing_major_component() {
+		assert_eq!(parse_semver_value(".2.3"), None);
+	}
+
 	#[test]
 	fn pick_download_url_prefers_installer() {
 		let assets = vec![
@@ -228,4 +243,30 @@ mod tests {
 		let url = pick_download_url(false, &assets);
 		assert_eq!(url.as_deref(), Some("https://example.com/PAPERBACK.ZIP"));
 	}
+
+	#[test]
+	fn pick_download_url_returns_none_when_preferred_asset_missing() {
+		let assets = vec![ReleaseAsset {
+			name: "notes.txt".to_string(),
+			browser_download_url: "https://example.com/notes.txt".to_string(),
+		}];
+		assert!(pick_download_url(true, &assets).is_none());
+		assert!(pick_download_url(false, &assets).is_none());
+	}
+
+	#[test]
+	fn pick_download_url_uses_flag_to_choose_between_exe_and_zip() {
+		let assets = vec![
+			ReleaseAsset {
+				name: "paperback.zip".to_string(),
+				browser_download_url: "https://example.com/paperback.zip".to_string(),
+			},
+			ReleaseAsset {
+				name: "paperback_setup.exe".to_string(),
+				browser_download_url: "https://example.com/paperback_setup.exe".to_string(),
+			},
+		];
+		assert_eq!(pick_download_url(false, &assets).as_deref(), Some("https://example.com/paperback.zip"));
+		assert_eq!(pick_download_url(true, &assets).as_deref(), Some("https://example.com/paperback_setup.exe"));
+	}
 }

src/zip.rs

@@ -83,4 +83,31 @@ mod tests {
 		let contents = fs::read_to_string(&output_path).expect("read output");
 		assert_eq!(contents, "nested");
 	}
+
+	#[test]
+	fn extract_zip_entry_to_file_reports_missing_entry() {
+		let mut archive = build_test_archive();
+		let output_path = unique_temp_path("nested/missing.txt");
+		assert!(extract_zip_entry_to_file(&mut archive, "does-not-exist.txt", &output_path).is_err());
+	}
+
+	#[test]
+	fn extract_zip_entry_to_file_overwrites_existing_file_contents() {
+		let mut archive = build_test_archive();
+		let output_path = unique_temp_path("nested/overwrite.txt");
+		if let Some(parent) = output_path.parent() {
+			fs::create_dir_all(parent).expect("create parent");
+		}
+		fs::write(&output_path, "old").expect("seed file");
+		extract_zip_entry_to_file(&mut archive, "foo.txt", &output_path).expect("extract entry");
+		let contents = fs::read_to_string(&output_path).expect("read output");
+		assert_eq!(contents, "hello world");
+	}
+
+	#[test]
+	fn read_zip_entry_by_name_reads_nested_entry() {
+		let mut archive = build_test_archive();
+		let contents = read_zip_entry_by_name(&mut archive, "nested/bar.txt").expect("read nested entry");
+		assert_eq!(contents, "nested");
+	}
 }