wehub-resource-sync
b4fbd6fe9f
Deploy Site / deploy-vercel (push) Has been skipped
Deploy Site / deploy-docs (push) Has been skipped
Build Skills Index / build-index (push) Has been skipped
Build Skills Index / trigger-deploy (push) Waiting to run
CI / Deny unrelated histories (push) Has been skipped
CI / CI timing report (push) Blocked by required conditions
CI / Detect affected areas (push) Successful in 27m35s
CI / OSV scan (push) Failing after 4s
CI / Build&Test Docker image (push) Successful in 9s
CI / Supply-chain scan (push) Has been skipped
CI / Lint Docker scripts (push) Failing after 5m13s
CI / Check contributors (push) Failing after 12m8s
CI / Docs Site (push) Failing after 12m8s
CI / TypeScript (push) Failing after 12m8s
CI / Python lints (push) Failing after 12m9s
CI / Python tests (push) Failing after 12m9s
CI / Check uv.lock (push) Failing after 23m22s
CI / All required checks pass (push) Waiting to run
72 lines
2.4 KiB
Python
72 lines
2.4 KiB
Python
"""Tests for hermes_cli/tips.py — random tip display at session start."""
|
|
|
|
from hermes_cli.tips import TIPS, get_random_tip
|
|
|
|
|
|
class TestTipsCorpus:
|
|
"""Validate the tip corpus itself."""
|
|
|
|
def test_has_at_least_200_tips(self):
|
|
assert len(TIPS) >= 200, f"Expected 200+ tips, got {len(TIPS)}"
|
|
|
|
def test_no_duplicates(self):
|
|
assert len(TIPS) == len(set(TIPS)), "Duplicate tips found"
|
|
|
|
def test_all_tips_are_strings(self):
|
|
for i, tip in enumerate(TIPS):
|
|
assert isinstance(tip, str), f"Tip {i} is not a string: {type(tip)}"
|
|
|
|
def test_no_empty_tips(self):
|
|
for i, tip in enumerate(TIPS):
|
|
assert tip.strip(), f"Tip {i} is empty or whitespace-only"
|
|
|
|
def test_max_length_reasonable(self):
|
|
"""Tips should fit on a single terminal line (~120 chars max)."""
|
|
for i, tip in enumerate(TIPS):
|
|
assert len(tip) <= 150, (
|
|
f"Tip {i} too long ({len(tip)} chars): {tip[:60]}..."
|
|
)
|
|
|
|
def test_no_leading_trailing_whitespace(self):
|
|
for i, tip in enumerate(TIPS):
|
|
assert tip == tip.strip(), f"Tip {i} has leading/trailing whitespace"
|
|
|
|
|
|
class TestGetRandomTip:
|
|
"""Validate the get_random_tip() function."""
|
|
|
|
def test_returns_string(self):
|
|
tip = get_random_tip()
|
|
assert isinstance(tip, str)
|
|
assert len(tip) > 0
|
|
|
|
def test_returns_tip_from_corpus(self):
|
|
tip = get_random_tip()
|
|
assert tip in TIPS
|
|
|
|
def test_randomness(self):
|
|
"""Multiple calls should eventually return different tips."""
|
|
seen = set()
|
|
for _ in range(50):
|
|
seen.add(get_random_tip())
|
|
# With 200+ tips and 50 draws, we should see at least 10 unique
|
|
assert len(seen) >= 10, f"Only got {len(seen)} unique tips in 50 draws"
|
|
|
|
|
|
class TestTipIntegrationInCLI:
|
|
"""Test that the tip display code in cli.py works correctly."""
|
|
|
|
def test_tip_import_works(self):
|
|
"""The import used in cli.py must succeed."""
|
|
from hermes_cli.tips import get_random_tip
|
|
assert callable(get_random_tip)
|
|
|
|
def test_tip_display_format(self):
|
|
"""Verify the Rich markup format doesn't break."""
|
|
tip = get_random_tip()
|
|
color = "#B8860B"
|
|
markup = f"[dim {color}]✦ Tip: {tip}[/]"
|
|
# Should not contain nested/broken Rich tags
|
|
assert markup.count("[/]") == 1
|
|
assert "[dim #B8860B]" in markup
|