Add unit tests for PoseEstimation widgets

CHANGELOG.md

@@ -8,6 +8,8 @@
 
 ## Improvements
 
+* Added comprehensive unit tests for PoseEstimation widgets covering discovery, widget creation, lazy loading, keypoint colors, error handling, and video mapping. [PR #15](https://github.com/catalystneuro/nwb-video-widgets/pull/15)
+
 # v0.1.5 (2026-02-03)
 
 ## Removals, Deprecations and Changes

pyproject.toml

@@ -41,6 +41,7 @@ dandi = ["dandi>=0.60.0", "remfile>=0.1.13", "h5py"]
 test = [
     "pytest>=7.0",
     "pytest-cov",
+    "ndx-pose",
 ]
 
 dev = [

tests/conftest.py

@@ -6,7 +6,11 @@
 import pytest
 from pynwb import NWBHDF5IO, read_nwb
 
-from tests.fixtures.synthetic_nwb import create_nwbfile_with_external_videos
+from tests.fixtures.synthetic_nwb import (
+    create_nwbfile_with_external_videos,
+    create_nwbfile_with_pose_estimation,
+    create_nwbfile_with_videos_and_pose,
+)
 from tests.fixtures.synthetic_video import create_synthetic_video
 
 
@@ -85,3 +89,62 @@ def nwbfile_with_explicit_timestamps(tmp_path, synthetic_video_path):
         io.write(nwbfile)
 
     return read_nwb(nwb_path)
+
+
+@pytest.fixture
+def nwbfile_with_single_camera_pose(tmp_path):
+    """Create an NWB file with pose estimation for a single camera."""
+    nwbfile = create_nwbfile_with_pose_estimation(
+        camera_names=["LeftCamera"],
+        keypoint_names=["Nose", "LeftEar", "RightEar"],
+        num_frames=30,
+    )
+    nwb_path = tmp_path / "test_pose.nwb"
+
+    with NWBHDF5IO(nwb_path, "w") as io:
+        io.write(nwbfile)
+
+    return read_nwb(nwb_path)
+
+
+@pytest.fixture
+def nwbfile_with_multiple_cameras_pose(tmp_path):
+    """Create an NWB file with pose estimation for multiple cameras."""
+    nwbfile = create_nwbfile_with_pose_estimation(
+        camera_names=["LeftCamera", "RightCamera", "BodyCamera"],
+        keypoint_names=["Nose", "LeftEar", "RightEar", "LeftPaw", "RightPaw"],
+        num_frames=30,
+    )
+    nwb_path = tmp_path / "test_multi_pose.nwb"
+
+    with NWBHDF5IO(nwb_path, "w") as io:
+        io.write(nwbfile)
+
+    return read_nwb(nwb_path)
+
+
+@pytest.fixture
+def nwbfile_with_videos_and_pose(tmp_path, synthetic_video_paths):
+    """Create an NWB file with both videos and pose estimation."""
+    copied_paths = {}
+    for name, path in synthetic_video_paths.items():
+        video_copy = tmp_path / path.name
+        shutil.copy(path, video_copy)
+        copied_paths[name] = video_copy
+
+    # Create pose estimation for cameras that match video names
+    # VideoLeftCamera -> LeftCamera, etc.
+    camera_names = [name.replace("Video", "") for name in copied_paths.keys()]
+
+    nwbfile = create_nwbfile_with_videos_and_pose(
+        video_paths=copied_paths,
+        camera_names=camera_names,
+        keypoint_names=["Nose", "LeftEar", "RightEar"],
+        num_frames=30,
+    )
+    nwb_path = tmp_path / "test_combined.nwb"
+
+    with NWBHDF5IO(nwb_path, "w") as io:
+        io.write(nwbfile)
+
+    return read_nwb(nwb_path)

tests/fixtures/synthetic_nwb.py

@@ -3,7 +3,8 @@
 from pathlib import Path
 
 import numpy as np
-from pynwb import NWBFile
+from ndx_pose import PoseEstimation, PoseEstimationSeries
+from pynwb import NWBFile, ProcessingModule
 from pynwb.image import ImageSeries
 from pynwb.testing.mock.file import mock_NWBFile
 
@@ -50,3 +51,166 @@ def create_nwbfile_with_external_videos(
         nwbfile.add_acquisition(image_series)
 
     return nwbfile
+
+
+def create_nwbfile_with_pose_estimation(
+    camera_names: list[str],
+    keypoint_names: list[str],
+    num_frames: int = 30,
+    timestamps: np.ndarray | None = None,
+) -> NWBFile:
+    """Create an NWBFile with PoseEstimation data.
+
+    Parameters
+    ----------
+    camera_names : list[str]
+        Names of cameras to create pose estimation data for
+    keypoint_names : list[str]
+        Names of keypoints to track
+    num_frames : int, optional
+        Number of frames of pose data, by default 30
+    timestamps : np.ndarray, optional
+        Timestamps for the pose data. If None, uses evenly spaced from 0 to 1.
+
+    Returns
+    -------
+    NWBFile
+        NWB file with pose estimation processing module
+    """
+    nwbfile = mock_NWBFile()
+
+    # Create pose_estimation processing module
+    pose_module = ProcessingModule(
+        name="pose_estimation",
+        description="Pose estimation data from DeepLabCut or similar",
+    )
+    nwbfile.add_processing_module(pose_module)
+
+    # Default timestamps
+    if timestamps is None:
+        timestamps = np.linspace(0.0, 1.0, num_frames)
+
+    # Create a PoseEstimation container for each camera
+    for camera_name in camera_names:
+        # Create PoseEstimationSeries for each keypoint
+        pose_series_list = []
+        for keypoint_name in keypoint_names:
+            # Generate synthetic pose data (x, y coordinates)
+            # Add some variation to make it more realistic
+            base_x = 100 + keypoint_names.index(keypoint_name) * 50
+            base_y = 200 + keypoint_names.index(keypoint_name) * 30
+            x_coords = base_x + np.random.randn(num_frames) * 5
+            y_coords = base_y + np.random.randn(num_frames) * 5
+            data = np.column_stack([x_coords, y_coords])
+
+            series = PoseEstimationSeries(
+                name=f"{keypoint_name}PoseEstimationSeries",
+                data=data,
+                reference_frame="top-left corner",
+                timestamps=timestamps,
+            )
+            pose_series_list.append(series)
+
+        # Create PoseEstimation container for this camera
+        pose_estimation = PoseEstimation(
+            name=camera_name,
+            pose_estimation_series=pose_series_list,
+            description=f"Pose estimation for {camera_name}",
+        )
+        pose_module.add(pose_estimation)
+
+    return nwbfile
+
+
+def create_nwbfile_with_videos_and_pose(
+    video_paths: dict[str, Path],
+    camera_names: list[str],
+    keypoint_names: list[str],
+    num_frames: int = 30,
+    timestamps: dict[str, np.ndarray] | None = None,
+) -> NWBFile:
+    """Create an NWBFile with both external videos and pose estimation.
+
+    Parameters
+    ----------
+    video_paths : dict[str, Path]
+        Mapping of video names to file paths
+    camera_names : list[str]
+        Names of cameras for pose estimation (should match video names pattern)
+    keypoint_names : list[str]
+        Names of keypoints to track
+    num_frames : int, optional
+        Number of frames of pose data, by default 30
+    timestamps : dict[str, np.ndarray], optional
+        Mapping of video names to timestamp arrays. If None, uses rate-based.
+
+    Returns
+    -------
+    NWBFile
+        NWB file with both videos and pose estimation
+    """
+    nwbfile = mock_NWBFile()
+
+    # Add videos
+    for name, path in video_paths.items():
+        external_path = f"./{path.name}"
+
+        if timestamps and name in timestamps:
+            image_series = ImageSeries(
+                name=name,
+                format="external",
+                external_file=[external_path],
+                timestamps=timestamps[name],
+            )
+        else:
+            image_series = ImageSeries(
+                name=name,
+                format="external",
+                external_file=[external_path],
+                starting_time=0.0,
+                rate=30.0,
+            )
+
+        nwbfile.add_acquisition(image_series)
+
+    # Add pose estimation
+    pose_module = ProcessingModule(
+        name="pose_estimation",
+        description="Pose estimation data from DeepLabCut or similar",
+    )
+    nwbfile.add_processing_module(pose_module)
+
+    # Use first video's timestamps if available
+    pose_timestamps = None
+    if timestamps:
+        first_video = next(iter(timestamps.values()))
+        pose_timestamps = first_video
+    else:
+        pose_timestamps = np.linspace(0.0, 1.0, num_frames)
+
+    # Create pose estimation for each camera
+    for camera_name in camera_names:
+        pose_series_list = []
+        for keypoint_name in keypoint_names:
+            base_x = 100 + keypoint_names.index(keypoint_name) * 50
+            base_y = 200 + keypoint_names.index(keypoint_name) * 30
+            x_coords = base_x + np.random.randn(len(pose_timestamps)) * 5
+            y_coords = base_y + np.random.randn(len(pose_timestamps)) * 5
+            data = np.column_stack([x_coords, y_coords])
+
+            series = PoseEstimationSeries(
+                name=f"{keypoint_name}PoseEstimationSeries",
+                data=data,
+                reference_frame="top-left corner",
+                timestamps=pose_timestamps,
+            )
+            pose_series_list.append(series)
+
+        pose_estimation = PoseEstimation(
+            name=camera_name,
+            pose_estimation_series=pose_series_list,
+            description=f"Pose estimation for {camera_name}",
+        )
+        pose_module.add(pose_estimation)
+
+    return nwbfile