Skip to content

Split TMGimporter's libtmg test into an OS-specific test_linux_libtmg#949

Merged
GaryGriffin merged 1 commit into
gramps-project:maintenance/gramps60from
eduralph:split/tmg-os-test-split
Jun 15, 2026
Merged

Split TMGimporter's libtmg test into an OS-specific test_linux_libtmg#949
GaryGriffin merged 1 commit into
gramps-project:maintenance/gramps60from
eduralph:split/tmg-os-test-split

Conversation

@eduralph

Copy link
Copy Markdown
Contributor

Split out of #820 per the one-change-per-PR review. This is a per-addon test edit, not
CI infrastructure, so it ships on its own.

What

Renames TMGimporter/tests/test_libtmg.pytest_linux_libtmg.py, following the
test_<os>_* naming convention the CI harness uses to scope per-OS test runs (the
convention #820 introduces — this is its demonstrator).

Draft until its own review/sign-off. Companion split-outs from #820: the dependency
detector (#948) and #820 itself (now CI-infrastructure only).

🤖 Generated with Claude Code

Split out of the CI-pipeline PR (gramps-project#820) per the one-change-per-PR review:
this is a per-addon test edit, not CI infrastructure.

Renames TMGimporter/tests/test_libtmg.py -> test_linux_libtmg.py to follow
the test_<os>_* naming convention the CI harness uses to scope per-OS test
runs (the convention gramps-project#820 introduces; this is its demonstrator).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@eduralph eduralph marked this pull request as ready for review June 13, 2026 14:09
@eduralph eduralph marked this pull request as draft June 13, 2026 14:29
@eduralph eduralph marked this pull request as ready for review June 13, 2026 14:30
@GaryGriffin GaryGriffin merged commit a245326 into gramps-project:maintenance/gramps60 Jun 15, 2026
@GaryGriffin

Copy link
Copy Markdown
Member

Cherry picking this to gramps61 failed. Need a new PR for the changes for gramps61.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants