diff --git a/steam_backlog_enforcer/scanning.py b/steam_backlog_enforcer/scanning.py index cee8b95..0d21e1d 100644 --- a/steam_backlog_enforcer/scanning.py +++ b/steam_backlog_enforcer/scanning.py @@ -151,12 +151,11 @@ def _pick_playable_candidate( ) return game logger.info( - "Skipping %s (AppID=%d): ProtonDB %s (trending %s) — %s", + "Skipping %s (AppID=%d): ProtonDB %s (trending %s)", game.name, game.app_id, rating.tier, rating.trending_tier, - rating.unplayable_reason, ) offset += _PROTONDB_BATCH_SIZE diff --git a/steam_backlog_enforcer/tests/test_protondb.py b/steam_backlog_enforcer/tests/test_protondb.py index 2a0caa1..55f2069 100644 --- a/steam_backlog_enforcer/tests/test_protondb.py +++ b/steam_backlog_enforcer/tests/test_protondb.py @@ -92,6 +92,10 @@ class TestProtonDBRating: r = ProtonDBRating(app_id=1, tier="unknown_tier") assert r.is_playable is False + def test_unplayable_reason_no_trending_tier(self) -> None: + r = ProtonDBRating(app_id=1, tier="borked") + assert "tier<" in r.unplayable_reason + def test_unplayable_reason_for_silver_silver(self) -> None: r = ProtonDBRating(app_id=1, tier="silver", trending_tier="silver") assert "no gold tier" in r.unplayable_reason diff --git a/steam_backlog_enforcer/tests/test_scanning.py b/steam_backlog_enforcer/tests/test_scanning.py index de058ec..98be132 100644 --- a/steam_backlog_enforcer/tests/test_scanning.py +++ b/steam_backlog_enforcer/tests/test_scanning.py @@ -215,27 +215,6 @@ class TestPickPlayableCandidate: result = _pick_playable_candidate([game]) assert result is not None - def test_logs_explicit_protondb_skip_reason(self) -> None: - """Unplayable candidate logs concrete reason, not just raw tiers.""" - bad = _game(app_id=1, name="Bad") - good = _game(app_id=2, name="Good") - with ( - patch( - "python_pkg.steam_backlog_enforcer.scanning.fetch_protondb_ratings", - return_value={ - 1: ProtonDBRating(app_id=1, tier="silver", trending_tier="silver"), - 2: ProtonDBRating(app_id=2, tier="platinum"), - }, - ), - patch("python_pkg.steam_backlog_enforcer.scanning._echo"), - patch("python_pkg.steam_backlog_enforcer.scanning.logger.info") as mock_log, - ): - result = _pick_playable_candidate([bad, good]) - - assert result is not None - assert result.app_id == 2 - assert any("no gold tier" in str(call) for call in mock_log.call_args_list) - class TestPickNextGame: """Tests for pick_next_game."""