Turns out idle and noidle are not commands that should be listed

This commit is contained in:
Thomas Adamcik 2011-07-25 01:00:18 +02:00
parent 5d025a5721
commit c724fcd7c9
2 changed files with 6 additions and 2 deletions

View File

@ -27,6 +27,10 @@ def commands(context):
command_names.remove('command_list_ok_begin')
if 'command_list_end' in command_names:
command_names.remove('command_list_end')
if 'idle' in command_names:
command_names.remove('idle')
if 'noidle' in command_names:
command_names.remove('noidle')
if 'sticker' in command_names:
command_names.remove('sticker')

View File

@ -9,14 +9,14 @@ class ReflectionHandlerTest(protocol.BaseTestCase):
self.assertInResponse(u'command: commands')
self.assertInResponse(u'command: play')
self.assertInResponse(u'command: status')
self.assertInResponse(u'command: idle')
self.assertInResponse(u'command: noidle')
# Check if commands you do not have access to are not present
self.assertNotInResponse(u'command: kill')
# Check if the blacklisted commands are not present
self.assertNotInResponse(u'command: command_list_begin')
self.assertNotInResponse(u'command: command_list_ok_begin')
self.assertNotInResponse(u'command: command_list_end')
self.assertNotInResponse(u'command: idle')
self.assertNotInResponse(u'command: noidle')
self.assertNotInResponse(u'command: sticker')
self.assertInResponse(u'OK')