personal memory agent
Make describe config tests less fragile
Remove hardcoded category counts and specific category lists that broke
when categories were added/removed. Keep tests that validate invariants
(required fields, prompt presence, alphabetical ordering) without
depending on specific category names or counts.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
+3
-57
+3
-57
tests/test_describe_config.py
+3
-57
tests/test_describe_config.py
···
7
7
"""Test that categories are discovered on import."""
8
8
CATEGORIES = describe_module.CATEGORIES
9
9
10
-
# Should have discovered all 9 categories
11
-
assert len(CATEGORIES) == 9
12
-
13
-
# All expected categories should be present
14
-
expected = [
15
-
"terminal",
16
-
"code",
17
-
"messaging",
18
-
"meeting",
19
-
"browsing",
20
-
"reading",
21
-
"media",
22
-
"gaming",
23
-
"productivity",
24
-
]
25
-
for cat in expected:
26
-
assert cat in CATEGORIES, f"Expected category {cat} not found"
10
+
# Should have discovered at least some categories
11
+
assert isinstance(CATEGORIES, dict)
12
+
assert len(CATEGORIES) > 0
27
13
28
14
29
15
def test_categories_have_required_fields():
···
59
45
), f"Category {category} has followup=true but no prompt"
60
46
assert isinstance(metadata["prompt"], str)
61
47
assert len(metadata["prompt"]) > 0
62
-
63
-
64
-
def test_non_followup_categories():
65
-
"""Test that non-followup categories don't have prompts."""
66
-
CATEGORIES = describe_module.CATEGORIES
67
-
68
-
non_followup = ["terminal", "code", "media", "gaming"]
69
-
for category in non_followup:
70
-
assert category in CATEGORIES
71
-
assert CATEGORIES[category]["followup"] is False
72
-
assert "prompt" not in CATEGORIES[category]
73
-
74
-
75
-
def test_meeting_category_config():
76
-
"""Test that meeting category has correct configuration."""
77
-
CATEGORIES = describe_module.CATEGORIES
78
-
79
-
assert "meeting" in CATEGORIES
80
-
meeting = CATEGORIES["meeting"]
81
-
assert meeting["followup"] is True
82
-
assert meeting["output"] == "json"
83
-
assert meeting["iq"] == "flash" # Meeting needs flash for complex JSON output
84
-
85
-
86
-
def test_text_categories_config():
87
-
"""Test that text-based categories have correct configuration."""
88
-
CATEGORIES = describe_module.CATEGORIES
89
-
90
-
text_categories = ["messaging", "browsing", "reading", "productivity"]
91
-
for category in text_categories:
92
-
assert category in CATEGORIES
93
-
cat_meta = CATEGORIES[category]
94
-
assert cat_meta["followup"] is True
95
-
assert cat_meta["output"] == "markdown"
96
-
97
-
# Messaging uses flash for better text extraction
98
-
assert CATEGORIES["messaging"]["iq"] == "flash"
99
-
# Others default to lite
100
-
for category in ["browsing", "reading", "productivity"]:
101
-
assert CATEGORIES[category].get("iq", "lite") == "lite"
102
48
103
49
104
50
def test_categorization_prompt_built():