Allow for playback duration to be missing

roku/_async/core.py

@@ -157,11 +157,15 @@ async def get_media_player(self):
             roku=self,
         )
 
+        # Duration isn't provided by all apps in all cases, e.g. Netflix / NOW TV
+        duration_element = root.find("duration")
+        duration = int(duration_element.text.split(" ", 1)[0]) if duration_element is not None else None
+
         mp = MediaPlayer(
             state=root.get("state"),
             app=app,
             position=int(root.find("position").text.split(" ", 1)[0]),
-            duration=int(root.find("duration").text.split(" ", 1)[0]),
+            duration=duration,
         )
         return mp
 

roku/core.py

@@ -162,11 +162,15 @@ def media_player(self):
         resp = self._get("/query/media-player")
         root = ET.fromstring(resp)
 
+        # Duration isn't provided by all apps in all cases, e.g. Netflix / NOW TV
+        duration_element = root.find("duration")
+        duration = int(duration_element.text.split(" ", 1)[0]) if duration_element is not None else None
+
         mp = MediaPlayer(
             state=root.get("state"),
             app=self[int(root.find("plugin").get("id"))],
             position=int(root.find("position").text.split(" ", 1)[0]),
-            duration=int(root.find("duration").text.split(" ", 1)[0]),
+            duration=duration,
         )
         return mp
 

roku/models.py

@@ -94,9 +94,9 @@ def __init__(self, state, app, position, duration):
         self.duration = duration
 
     def __repr__(self):
-        return "<MediaPlayer: %s in %s at %s/%s ms>" % (
+        progress = f("%s/%s", self.position, self.duration) if self.duration is not None else f("%s", self.position)
+        return "<MediaPlayer: %s in %s at %s ms>" % (
             self.state,
             self.app.name,
-            self.position,
-            self.duration,
+            progress,
         )