refactor: simplify effects metadata tests and remove obsolete instance checks
This commit is contained in:
1 file changed
+2
-11
+2
-11
@@ -89,22 +89,13 @@ class TestAudioEffects(unittest.TestCase):
|
||||
self.assertFalse(toggled)
|
||||
self.assertFalse(pipeline.is_active("distortion"))
|
||||
|
||||
def test_effects_metadata_does_not_contain_badge_or_description(self):
|
||||
def test_effects_metadata_structure(self):
|
||||
from zyrarbot.audio.effects.registry import get_available_effects_metadata
|
||||
|
||||
metadata = get_available_effects_metadata()
|
||||
self.assertGreater(len(metadata), 0)
|
||||
for entry in metadata:
|
||||
self.assertIn("id", entry)
|
||||
self.assertIn("name", entry)
|
||||
self.assertIn("icon", entry)
|
||||
self.assertNotIn("badge", entry)
|
||||
self.assertNotIn("description", entry)
|
||||
|
||||
def test_effect_instances_have_no_badge_or_description_attribute(self):
|
||||
for effect in create_default_effects():
|
||||
self.assertFalse(hasattr(effect, "badge"))
|
||||
self.assertFalse(hasattr(effect, "description"))
|
||||
self.assertEqual(set(entry.keys()), {"id", "name", "icon"})
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
|
||||
Reference in new issue
Block a user