From 4013a2ec9afd9a08ad0230f5fb71fe0750deae3e Mon Sep 17 00:00:00 2001
From: Stein Magnus Jodal <stein.magnus@jodal.no>
Date: Mon, 20 Jun 2011 14:58:29 +0300
Subject: [PATCH] Test and implement mpris.CanPlay property

---
 mopidy/frontends/mpris.py                     |  9 +++++++--
 .../frontends/mpris/player_interface_test.py  | 19 +++++++++++++++++++
 2 files changed, 26 insertions(+), 2 deletions(-)

diff --git a/mopidy/frontends/mpris.py b/mopidy/frontends/mpris.py
index db77c845..e00075a6 100644
--- a/mopidy/frontends/mpris.py
+++ b/mopidy/frontends/mpris.py
@@ -151,8 +151,7 @@ class MprisObject(dbus.service.Object):
             'CanGoNext': (False, None),
             # TODO True if CanControl and backend.playback.track_at_previous
             'CanGoPrevious': (False, None),
-            # TODO True if CanControl and backend.playback.current_track
-            'CanPlay': (False, None),
+            'CanPlay': (self.get_CanPlay, None),
             'CanPause': (self.get_CanPause, None),
             'CanSeek': (self.get_CanSeek, None),
             'CanControl': (self.get_CanControl, None),
@@ -408,6 +407,12 @@ class MprisObject(dbus.service.Object):
     def get_Position(self):
         return self.backend.playback.time_position.get() * 1000
 
+    def get_CanPlay(self):
+        if not self.get_CanControl():
+            return False
+        return (self.backend.playback.current_track.get() is not None
+            or self.backend.playback.track_at_next.get() is not None)
+
     def get_CanPause(self):
         if not self.get_CanControl():
             return False
diff --git a/tests/frontends/mpris/player_interface_test.py b/tests/frontends/mpris/player_interface_test.py
index c1590d92..f44b61ee 100644
--- a/tests/frontends/mpris/player_interface_test.py
+++ b/tests/frontends/mpris/player_interface_test.py
@@ -177,6 +177,25 @@ class PlayerInterfaceTest(unittest.TestCase):
         result = self.mpris.Get(mpris.PLAYER_IFACE, 'MaximumRate')
         self.assert_(result >= 1.0)
 
+    def test_can_play_is_true_if_can_control_and_current_track(self):
+        self.mpris.get_CanControl = lambda *_: True
+        self.backend.current_playlist.append([Track(uri='a')])
+        self.backend.playback.play()
+        self.assertTrue(self.backend.playback.current_track.get())
+        result = self.mpris.Get(mpris.PLAYER_IFACE, 'CanPlay')
+        self.assertTrue(result)
+
+    def test_can_play_is_false_if_no_current_track(self):
+        self.mpris.get_CanControl = lambda *_: True
+        self.assertFalse(self.backend.playback.current_track.get())
+        result = self.mpris.Get(mpris.PLAYER_IFACE, 'CanPlay')
+        self.assertFalse(result)
+
+    def test_can_play_if_false_if_can_control_is_false(self):
+        self.mpris.get_CanControl = lambda *_: False
+        result = self.mpris.Get(mpris.PLAYER_IFACE, 'CanPlay')
+        self.assertFalse(result)
+
     def test_can_pause_is_true_if_can_control_and_track_can_be_paused(self):
         self.mpris.get_CanControl = lambda *_: True
         result = self.mpris.Get(mpris.PLAYER_IFACE, 'CanPause')