mirror of
https://github.com/kuhyx/testsAndMisc.git
synced 2026-07-04 21:03:15 +02:00
Krzysztof kuhy Rudnicki
ee27d10fef
Fix ruff violations in ~15 source files and ~60+ test files to minimize per-file-ignores in pyproject.toml. Remaining ignores are justified with comments explaining why each suppression is necessary. Source fixes: FBT003 (keyword args), S310 (URL validation), SLF001 (private access), T201 (print→logging), C901 (complexity), E501 (line length), E402 (import order). Test fixes: SIM117 (combined with), FBT (boolean args), PERF203 (try in loop), S310/S607 (URLs/executables), E402/E501 (imports/lines), S108 (tmp paths), PLR0913 (too many args), ARG (unused args), ANN (type annotations), RUF059 (unused unpacked vars), PT019 (fixture naming). Remaining per-file-ignores (with justifications): - Tests: ARG, D, PLC0415, PLR2004, S101, SLF001 - music_gen sources: PLC0415 (heavy ML lazy imports) - moviepy_showcase: PLC0415 (circular dependency) - generate_images: PLR0913 (matplotlib helpers need many params) - praca_magisterska_video: E501, E402 (long paths, mpl.use)
406 lines
13 KiB
Python
406 lines
13 KiB
Python
"""Tests for running and strength data verification."""
|
|
|
|
from __future__ import annotations
|
|
|
|
from typing import TYPE_CHECKING
|
|
from unittest.mock import MagicMock
|
|
|
|
from python_pkg.screen_locker.tests.conftest import (
|
|
RunningData,
|
|
StrengthData,
|
|
create_locker,
|
|
setup_running_entries,
|
|
setup_strength_entries,
|
|
)
|
|
|
|
if TYPE_CHECKING:
|
|
from pathlib import Path
|
|
|
|
|
|
class TestVerifyRunningData:
|
|
"""Tests for verify_running_data method."""
|
|
|
|
def test_valid_running_data(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test valid running data triggers unlock attempt."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_running_entries(locker, RunningData("5", "25", "5"))
|
|
locker.log_file = tmp_path / "workout_log.json"
|
|
locker.workout_data = {"type": "running"}
|
|
object.__setattr__(locker, "_attempt_unlock", MagicMock())
|
|
|
|
locker.verify_running_data()
|
|
|
|
locker._attempt_unlock.assert_called_once()
|
|
|
|
def test_invalid_distance_zero(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test zero distance is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_running_entries(locker, RunningData("0", "25", "5"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_running_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Distance" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_distance_too_high(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test distance over max is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_running_entries(locker, RunningData("150", "600", "4"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_running_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Distance" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_time_zero(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test zero time is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_running_entries(locker, RunningData("5", "0", "5"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_running_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Time" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_time_too_high(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test time over max is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_running_entries(locker, RunningData("5", "700", "5"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_running_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Time" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_pace_zero(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test zero pace is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_running_entries(locker, RunningData("5", "25", "0"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_running_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Pace" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_pace_too_high(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test pace over max is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_running_entries(locker, RunningData("5", "25", "25"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_running_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Pace" in locker.show_error.call_args[0][0]
|
|
|
|
def test_pace_mismatch(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test pace mismatch is rejected."""
|
|
# 5km in 25 min should be 5 min/km, but we say 10 min/km
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_running_entries(locker, RunningData("5", "25", "10"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_running_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Pace doesn't match" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_number_format(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test non-numeric input is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_running_entries(locker, RunningData("abc", "25", "5"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_running_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "valid numbers" in locker.show_error.call_args[0][0]
|
|
|
|
|
|
class TestVerifyStrengthData:
|
|
"""Tests for verify_strength_data method."""
|
|
|
|
def test_valid_strength_data(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test valid strength data triggers unlock attempt."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Squat", "3", "10", "50", "1500"))
|
|
locker.log_file = tmp_path / "workout_log.json"
|
|
locker.workout_data = {"type": "strength"}
|
|
object.__setattr__(locker, "_attempt_unlock", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker._attempt_unlock.assert_called_once()
|
|
|
|
def test_valid_multiple_exercises(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test valid data with multiple exercises."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(
|
|
locker,
|
|
StrengthData("Squat, Bench Press", "3, 3", "10, 8", "50, 40", "2460"),
|
|
)
|
|
locker.log_file = tmp_path / "workout_log.json"
|
|
locker.workout_data = {"type": "strength"}
|
|
object.__setattr__(locker, "_attempt_unlock", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker._attempt_unlock.assert_called_once()
|
|
|
|
def test_mismatched_list_lengths(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test mismatched list lengths are rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(
|
|
locker,
|
|
StrengthData("Squat, Bench", "3", "10, 8", "50, 40", "2000"),
|
|
)
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "must match" in locker.show_error.call_args[0][0]
|
|
|
|
def test_short_exercise_name(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test short exercise names are rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Sq", "3", "10", "50", "1500"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "too short" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_sets_zero(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test zero sets is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Squat", "0", "10", "50", "0"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Sets" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_sets_too_high(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test sets over max is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Squat", "25", "10", "50", "12500"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Sets" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_reps_zero(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test zero reps is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Squat", "3", "0", "50", "0"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Reps" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_reps_too_high(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test reps over max is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Squat", "3", "150", "50", "22500"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Reps" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_weight_negative(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test negative weight is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Squat", "3", "10", "-10", "-300"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Weights" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_weight_too_high(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test weight over max is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Squat", "3", "10", "600", "18000"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Weights" in locker.show_error.call_args[0][0]
|
|
|
|
def test_total_weight_mismatch(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test total weight mismatch is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Squat", "3", "10", "50", "3000"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "Total weight doesn't match" in locker.show_error.call_args[0][0]
|
|
|
|
def test_invalid_format(
|
|
self,
|
|
mock_tk: MagicMock,
|
|
mock_sys_exit: MagicMock,
|
|
tmp_path: Path,
|
|
) -> None:
|
|
"""Test invalid format is rejected."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
setup_strength_entries(locker, StrengthData("Squat", "abc", "10", "50", "1500"))
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
|
|
locker.verify_strength_data()
|
|
|
|
locker.show_error.assert_called_once()
|
|
assert "valid data" in locker.show_error.call_args[0][0]
|
|
|
|
|
|
class TestVariableReps:
|
|
"""Tests for variable reps format in strength verification."""
|
|
|
|
def test_valid_variable_reps(
|
|
self, mock_tk: MagicMock, mock_sys_exit: MagicMock, tmp_path: Path
|
|
) -> None:
|
|
"""Test valid variable reps with + separator."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
# 3 sets, reps 12+11+12 (3 variable values matching 3 sets), weight 50
|
|
# Total = (12+11+12) * 50 = 1750
|
|
setup_strength_entries(
|
|
locker, StrengthData("Squat", "3", "12+11+12", "50", "1750")
|
|
)
|
|
locker.log_file = tmp_path / "workout_log.json"
|
|
locker.workout_data = {"type": "strength"}
|
|
object.__setattr__(locker, "_attempt_unlock", MagicMock())
|
|
locker.verify_strength_data()
|
|
locker._attempt_unlock.assert_called_once()
|
|
|
|
def test_variable_reps_count_mismatch(
|
|
self, mock_tk: MagicMock, mock_sys_exit: MagicMock, tmp_path: Path
|
|
) -> None:
|
|
"""Test variable reps count not matching sets."""
|
|
locker = create_locker(mock_tk, tmp_path)
|
|
# 5 sets but only 3 variable reps
|
|
setup_strength_entries(
|
|
locker, StrengthData("Squat", "5", "12+11+12", "50", "1750")
|
|
)
|
|
object.__setattr__(locker, "show_error", MagicMock())
|
|
locker.verify_strength_data()
|
|
locker.show_error.assert_called_once()
|
|
assert "variable reps count" in locker.show_error.call_args[0][0]
|