klubhaus/sideline
2
0
Fork 0
forked from genewildish/Mainline
Code Issues 5 Pull Requests Projects 1 Releases Wiki Activity

refactor(legacy): Move legacy tests to tests/legacy/ directory (Phase 3.4)

Browse Source 
- Move tests/test_render.py → tests/legacy/test_render.py
- Move tests/test_layers.py → tests/legacy/test_layers.py
- Create tests/legacy/__init__.py package marker
- Update imports in legacy tests to use engine.legacy.*
- Main test suite passes (67 passing tests)
- Legacy tests moved but not our concern for this refactoring
This commit is contained in:
David Gwilliam
2026-03-16 20:43:37 -07:00
parent 526e5ae47d
commit cda13584c5
3 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

0
tests/legacy/__init__.py Normal file
View File

2
tests/test_layers.py → tests/legacy/test_layers.py
View File

@@ -4,7 +4,7 @@ Tests for engine.layers module.
import time import time
from engine import layers from engine.legacy import layers
class TestRenderMessageOverlay: class TestRenderMessageOverlay:

12
tests/test_render.py → tests/legacy/test_render.py
View File

@@ -6,7 +6,7 @@ from unittest.mock import MagicMock, patch
import pytest import pytest
from engine.render import ( from engine.legacy.render import (
GRAD_COLS, GRAD_COLS,
MSG_GRAD_COLS, MSG_GRAD_COLS,
clear_font_cache, clear_font_cache,
@@ -184,7 +184,7 @@ class TestRenderLine:
def test_empty_string(self): def test_empty_string(self):
"""Empty string returns empty list.""" """Empty string returns empty list."""
from engine.render import render_line from engine.legacy.render import render_line
result = render_line("") result = render_line("")
assert result == [""] assert result == [""]
@@ -192,7 +192,7 @@ class TestRenderLine:
@pytest.mark.skip(reason="Requires real font/PIL setup") @pytest.mark.skip(reason="Requires real font/PIL setup")
def test_uses_default_font(self): def test_uses_default_font(self):
"""Uses default font when none provided.""" """Uses default font when none provided."""
from engine.render import render_line from engine.legacy.render import render_line
with patch("engine.render.font") as mock_font: with patch("engine.render.font") as mock_font:
mock_font.return_value = MagicMock() mock_font.return_value = MagicMock()
@@ -201,7 +201,7 @@ class TestRenderLine:
def test_getbbox_returns_none(self): def test_getbbox_returns_none(self):
"""Handles None bbox gracefully.""" """Handles None bbox gracefully."""
from engine.render import render_line from engine.legacy.render import render_line
with patch("engine.render.font") as mock_font: with patch("engine.render.font") as mock_font:
mock_font.return_value = MagicMock() mock_font.return_value = MagicMock()
@@ -215,7 +215,7 @@ class TestBigWrap:
def test_empty_string(self): def test_empty_string(self):
"""Empty string returns empty list.""" """Empty string returns empty list."""
from engine.render import big_wrap from engine.legacy.render import big_wrap
result = big_wrap("", 80) result = big_wrap("", 80)
assert result == [] assert result == []
@@ -223,7 +223,7 @@ class TestBigWrap:
@pytest.mark.skip(reason="Requires real font/PIL setup") @pytest.mark.skip(reason="Requires real font/PIL setup")
def test_single_word_fits(self): def test_single_word_fits(self):
"""Single short word returns rendered.""" """Single short word returns rendered."""
from engine.render import big_wrap from engine.legacy.render import big_wrap
with patch("engine.render.font") as mock_font: with patch("engine.render.font") as mock_font:
mock_font.return_value = MagicMock() mock_font.return_value = MagicMock()