Bug: test_channel_worktree imports tests.recall.test_channel and fails in CI

[laynepenney]

Summary

recall#664 and recall#668 are red across the test matrix on an unrelated failure in tests/recall/test_channel_worktree.py.

The failing tests import _patch_data_dir via from tests.recall.test_channel import _patch_data_dir, which raises ModuleNotFoundError: No module named "tests" in GitHub Actions.

Evidence

Seen in the unit-tests matrix for recall#664 and recall#668 on 2026-04-11.

Example failure:

• tests/recall/test_channel_worktree.py:100
• tests/recall/test_channel_worktree.py:137

ModuleNotFoundError: No module named "tests"

Impact

This is currently masking the actual seam PR state because the matrix is red even when the seam changes themselves are not the cause.

Suggested fix

Stop importing test helpers through the top-level tests package path. Either:

• move the helper into a local fixture/helper module imported relatively within the test tree, or
• inline the small data-dir patch helper into test_channel_worktree.py.

Related

• recall#664
• recall#668

[laynepenney]

Fix is up in recall#670: #670

This makes tests/recall/test_channel_worktree.py self-contained so CI no longer depends on tests.recall.test_channel being importable as a package.