fix: remove dead code in unplayable_reason; add coverage for playable path

This commit is contained in:
Krzysztof kuhy Rudnicki 2026-05-08 20:34:29 +02:00
parent 7f7a5b68fb
commit c828aafdf9
2 changed files with 6 additions and 4 deletions

View File

@ -90,7 +90,7 @@ class ProtonDBRating:
return "" return ""
tier_rank = TIER_ORDER.get(self.tier, 99) tier_rank = TIER_ORDER.get(self.tier, 99)
min_rank = TIER_ORDER[MIN_PLAYABLE_TIER] TIER_ORDER[MIN_PLAYABLE_TIER]
silver_rank = TIER_ORDER["silver"] silver_rank = TIER_ORDER["silver"]
if not self.trending_tier: if not self.trending_tier:
@ -99,9 +99,7 @@ class ProtonDBRating:
trend_rank = TIER_ORDER.get(self.trending_tier, 99) trend_rank = TIER_ORDER.get(self.trending_tier, 99)
if tier_rank > silver_rank or trend_rank > silver_rank: if tier_rank > silver_rank or trend_rank > silver_rank:
return f"below silver ({self.tier}/{self.trending_tier})" return f"below silver ({self.tier}/{self.trending_tier})"
if tier_rank > min_rank and trend_rank > min_rank: return f"no gold tier ({self.tier}/{self.trending_tier})"
return f"no gold tier ({self.tier}/{self.trending_tier})"
return "fails ProtonDB rule"
def _load_cache() -> dict[str, Any]: def _load_cache() -> dict[str, Any]:

View File

@ -100,6 +100,10 @@ class TestProtonDBRating:
r = ProtonDBRating(app_id=1, tier="gold", trending_tier="bronze") r = ProtonDBRating(app_id=1, tier="gold", trending_tier="bronze")
assert "below silver" in r.unplayable_reason assert "below silver" in r.unplayable_reason
def test_unplayable_reason_empty_when_playable(self) -> None:
r = ProtonDBRating(app_id=1, tier="gold")
assert r.unplayable_reason == ""
class TestProtonDBCache: class TestProtonDBCache:
"""Tests for cache I/O.""" """Tests for cache I/O."""