mpd: Accept listall and listallinfo without the URI argument

docs/changelog.rst

@@ -42,9 +42,9 @@ v0.15.0 (UNRELEASED)
   to start providing their own custom libraries instead of being stuck with
   just our tag cache as the only option.
 
-- Converted local backend to use new `local:playlist:path` and
-  `local:track:path` uri scheme. Also moves support of `file://` to streaming
-  backend.
+- Converted local backend to use new ``local:playlist:path`` and
+  ``local:track:path`` URI scheme. Also moves support of ``file://`` to
+  streaming backend.
 
 **Spotify backend**
 
@@ -60,6 +60,10 @@ v0.15.0 (UNRELEASED)
 - Replace newline, carriage return and forward slash in playlist names. (Fixes:
   :issue:`474`, :issue:`480`)
 
+- Accept ``listall`` and ``listallinfo`` commands without the URI parameter.
+  The methods are still not implemented, but now the commands are accepted as
+  valid.
+
 
 v0.14.2 (2013-07-01)
 ====================

mopidy/frontends/mpd/protocol/music_db.py

@@ -245,8 +245,9 @@ def _list_date(context, query):
     return dates
 
 
-@handle_request(r'^listall "(?P<uri>[^"]+)"')
-def listall(context, uri):
+@handle_request(r'^listall$')
+@handle_request(r'^listall "(?P<uri>[^"]+)"$')
+def listall(context, uri=None):
     """
     *musicpd.org, music database section:*
 
@@ -257,8 +258,9 @@ def listall(context, uri):
     raise MpdNotImplemented  # TODO
 
 
-@handle_request(r'^listallinfo "(?P<uri>[^"]+)"')
-def listallinfo(context, uri):
+@handle_request(r'^listallinfo$')
+@handle_request(r'^listallinfo "(?P<uri>[^"]+)"$')
+def listallinfo(context, uri=None):
     """
     *musicpd.org, music database section:*
 

tests/frontends/mpd/protocol/music_db_test.py

@@ -82,11 +82,19 @@ class MusicDatabaseHandlerTest(protocol.BaseTestCase):
         self.assertEqual(playlists[0].tracks[0].uri, 'dummy:a')
         self.assertInResponse('OK')
 
-    def test_listall(self):
+    def test_listall_without_uri(self):
+        self.sendRequest('listall')
+        self.assertEqualResponse('ACK [0@0] {} Not implemented')
+
+    def test_listall_with_uri(self):
         self.sendRequest('listall "file:///dev/urandom"')
         self.assertEqualResponse('ACK [0@0] {} Not implemented')
 
-    def test_listallinfo(self):
+    def test_listallinfo_without_uri(self):
+        self.sendRequest('listallinfo')
+        self.assertEqualResponse('ACK [0@0] {} Not implemented')
+
+    def test_listallinfo_with_uri(self):
         self.sendRequest('listallinfo "file:///dev/urandom"')
         self.assertEqualResponse('ACK [0@0] {} Not implemented')