diff --git a/plexapi/video.py b/plexapi/video.py index 661fa4c1..48bbcac7 100644 --- a/plexapi/video.py +++ b/plexapi/video.py @@ -179,8 +179,10 @@ class Movie(Video, Playable): url = self.getStreamURL(**kwargs) else: self._server.url('%s?download=1' % location.key) - filepath = utils.download(url, filename=name, savepath=savepath, session=self._server._session, **kwargs) - filepaths.append(filepath) + filepath = utils.download(url, filename=name, savepath=savepath, session=self._server._session) + if filepath: + filepaths.append(filepath) + return filepaths diff --git a/tests/test_server.py b/tests/test_server.py index 92f729dd..200b0d78 100644 --- a/tests/test_server.py +++ b/tests/test_server.py @@ -233,7 +233,7 @@ def test_server_account(plex): assert account.signInState == 'ok' assert isinstance(account.subscriptionActive, bool) if account.subscriptionActive: assert len(account.subscriptionFeatures) - else: assert account.subscriptionFeatures == [] + else: assert account.subscriptionFeatures == ['mediaproviders-news-platform-specific'] assert account.subscriptionState == 'Active' if account.subscriptionActive else 'Unknown' assert re.match(utils.REGEX_EMAIL, account.username) diff --git a/tests/test_video.py b/tests/test_video.py index 69ab5e39..5c0ab4c9 100644 --- a/tests/test_video.py +++ b/tests/test_video.py @@ -278,7 +278,7 @@ def test_video_Show_attrs(show): assert show.originallyAvailableAt.strftime('%Y-%m-%d') == '2011-04-17' assert show.rating >= 8.0 assert utils.is_int(show.ratingKey) - assert sorted([i.tag for i in show.roles])[:4] == ['Aidan Gillen', 'Alexander Siddig', 'Alfie Allen', 'Anton Lesser'] + assert sorted([i.tag for i in show.roles])[:4] == ['Aidan Gillen', 'Alexander Siddig', 'Alfie Allen', 'Art Parkinson'] assert sorted([i.tag for i in show.actors])[:4] == ['Aidan Gillen', 'Alexander Siddig', 'Alfie Allen', 'Anton Lesser'] assert show._server._baseurl == utils.SERVER_BASEURL assert show.studio == 'HBO'