steam-backlog-enforcer

kuhyx/steam-backlog-enforcer

Fork 0

mirror of https://github.com/kuhyx/steam-backlog-enforcer.git synced 2026-07-04 15:23:05 +02:00

Commit Graph

Author	SHA1	Message	Date
Krzysztof kuhy Rudnicki	093c5afd3b	fix: prevent tests from touching real Steam files Tests for pick_next_game were calling uninstall_other_games and state.save against real filesystem paths, deleting installed games and overwriting the user's state.json whenever tests or pre-commit ran. - Add conftest.py safety net that redirects STEAMAPPS_PATH, CONFIG_DIR, STATE_FILE, SNAPSHOT_FILE, CONFIG_FILE, and HOSTS_FILE to tmp_path in all steam_backlog_enforcer tests - Add missing uninstall_other_games mock to 4 tests in test_scanning.py (test_picks_shortest, test_skips_finished, test_unknown_hours, test_picks_game_no_hours)	2026-03-25 21:15:40 +01:00
Krzysztof kuhy Rudnicki	2ed98ce4db	Reduce per-file-ignores by fixing lint violations across codebase 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)	2026-03-25 18:58:05 +01:00
Krzysztof kuhy Rudnicki	1f02540eb5	test: achieve 100% branch coverage across all python_pkg packages - Add comprehensive tests for all packages (3572 tests, 100% branch coverage) - Split oversized test files to stay under 500-line limit - Add per-file ruff ignores for test-appropriate suppressions - Fix _cache_decks.py to properly convert JSON lists to tuples - Add session-scoped conftest fixture for logging handler cleanup (Python 3.14) - Update ruff pre-commit hook to v0.15.2 - Add codespell ignore words for test data - Add generated output files to .gitignore	2026-03-21 17:51:36 +01:00

Author

SHA1

Message

Date

Krzysztof kuhy Rudnicki

093c5afd3b

fix: prevent tests from touching real Steam files

Tests for pick_next_game were calling uninstall_other_games and
state.save against real filesystem paths, deleting installed games
and overwriting the user's state.json whenever tests or pre-commit
ran.

- Add conftest.py safety net that redirects STEAMAPPS_PATH,
  CONFIG_DIR, STATE_FILE, SNAPSHOT_FILE, CONFIG_FILE, and
  HOSTS_FILE to tmp_path in all steam_backlog_enforcer tests
- Add missing uninstall_other_games mock to 4 tests in
  test_scanning.py (test_picks_shortest, test_skips_finished,
  test_unknown_hours, test_picks_game_no_hours)

2026-03-25 21:15:40 +01:00

Krzysztof kuhy Rudnicki

2ed98ce4db

Reduce per-file-ignores by fixing lint violations across codebase

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)

2026-03-25 18:58:05 +01:00

Krzysztof kuhy Rudnicki

1f02540eb5

test: achieve 100% branch coverage across all python_pkg packages

- Add comprehensive tests for all packages (3572 tests, 100% branch coverage)
- Split oversized test files to stay under 500-line limit
- Add per-file ruff ignores for test-appropriate suppressions
- Fix _cache_decks.py to properly convert JSON lists to tuples
- Add session-scoped conftest fixture for logging handler cleanup (Python 3.14)
- Update ruff pre-commit hook to v0.15.2
- Add codespell ignore words for test data
- Add generated output files to .gitignore

2026-03-21 17:51:36 +01:00

3 Commits