Skip to content

Commit 8910a30

Browse files
committed
Fix tests
Should we just remove the cache and require that the user reloads this? or i can reload in the tests.
1 parent ef0556f commit 8910a30

File tree

2 files changed

+14
-5
lines changed

2 files changed

+14
-5
lines changed

plexapi/playlist.py

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,24 +66,32 @@ def addItems(self, items):
6666
key = '%s/items%s' % (self.key, utils.joinArgs({
6767
'uri': 'library://%s/directory//library/metadata/%s' % (uuid, ratingKeys)
6868
}))
69-
return self._server.query(key, method=self._server._session.put)
69+
result = self._server.query(key, method=self._server._session.put)
70+
self.reload()
71+
return result
7072

7173
def removeItem(self, item):
7274
""" Remove a file from a playlist. """
7375
key = '%s/items/%s' % (self.key, item.playlistItemID)
74-
return self._server.query(key, method=self._server._session.delete)
76+
result = self._server.query(key, method=self._server._session.delete)
77+
self.reload()
78+
return result
7579

7680
def moveItem(self, item, after=None):
7781
""" Move a to a new position in playlist. """
7882
key = '%s/items/%s/move' % (self.key, item.playlistItemID)
7983
if after:
8084
key += '?after=%s' % after.playlistItemID
81-
return self._server.query(key, method=self._server._session.put)
85+
result = self._server.query(key, method=self._server._session.put)
86+
self.reload()
87+
return result
8288

8389
def edit(self, title=None, summary=None):
8490
""" Edit playlist. """
8591
key = '/library/metadata/%s%s' % (self.ratingKey, utils.joinArgs({'title': title, 'summary': summary}))
86-
return self._server.query(key, method=self._server._session.put)
92+
result = self._server.query(key, method=self._server._session.put)
93+
self.reload()
94+
return result
8795

8896
def delete(self):
8997
""" Delete playlist. """

tests/test_server.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,8 @@ def test_server_account(plex):
233233
assert account.signInState == 'ok'
234234
assert isinstance(account.subscriptionActive, bool)
235235
if account.subscriptionActive: assert len(account.subscriptionFeatures)
236-
else: assert sorted(account.subscriptionFeatures) == ['download_certificates', 'federated-auth', 'news']
236+
else: assert sorted(account.subscriptionFeatures) == ['adaptive_bitrate', 'download_certificates',
237+
'federated-auth', 'news']
237238
assert account.subscriptionState == 'Active' if account.subscriptionActive else 'Unknown'
238239
assert re.match(utils.REGEX_EMAIL, account.username)
239240

0 commit comments

Comments
 (0)