Merge pull request #805 from OliverZhaohaibin/main

Enhance video editing, metadata extraction, and CI test stability

Author: OliverZhaohaibin

src/iPhoto/io/metadata.py

@@ -219,7 +219,7 @@ def read_image_meta_with_exiftool(
     geometry_missing = info["w"] is None or info["h"] is None
     need_dt_fallback = info["dt"] is None
 
-    if (geometry_missing or need_dt_fallback) and Image is not None and UnidentifiedImageError is not None:
+    if (geometry_missing or need_dt_fallback) and Image is not None and UnidentifiedImageError is not None and path.exists():
         LOGGER.debug("Opening %s with Pillow to backfill metadata", path)
         try:
             with Image.open(path) as img:

tests/test_info_panel.py

@@ -80,7 +80,7 @@ def test_info_panel_video_shows_lens_when_available(qapp: QApplication) -> None:
 
     panel.set_asset_metadata(metadata)
 
-    assert panel._lens_label.isVisible()
+    assert not panel._lens_label.isHidden()
     assert "iPhone 12 back camera 4.2mm f/1.6" in panel._lens_label.text()
     panel.close()
 
@@ -199,7 +199,7 @@ def test_info_panel_video_shows_lens_spec_string_when_no_model_name(qapp: QAppli
 
     panel.set_asset_metadata(meta)
 
-    assert panel._lens_label.isVisible()
+    assert not panel._lens_label.isHidden()
     assert "23mm f/2" in panel._lens_label.text()
     panel.close()
 
@@ -232,7 +232,7 @@ def test_info_panel_lens_spec_string_not_duplicated_when_focal_and_fnumber_also_
     panel.set_asset_metadata(meta)
 
     label_text = panel._lens_label.text()
-    assert panel._lens_label.isVisible()
+    assert not panel._lens_label.isHidden()
     assert label_text == "23mm f/2"
     panel.close()
 
@@ -260,9 +260,9 @@ def test_info_panel_named_lens_model_gets_focal_appended(
     panel.set_asset_metadata(meta)
 
     label_text = panel._lens_label.text()
-    assert panel._lens_label.isVisible()
+    assert not panel._lens_label.isHidden()
     # The named model should be present and enriched with focal + aperture info.
     assert "XF23mmF2 R WR" in label_text
     assert "23" in label_text   # focal length must appear
-    assert "f/2" in label_text  # aperture must appear
+    assert "ƒ2" in label_text  # aperture must appear
     panel.close()