test meyzieu_newsletter: use values() instead of itervalues() (#38781)
This commit is contained in:
parent
5a64a916d2
commit
9d0629576e
|
@ -97,7 +97,7 @@ def test_get_newsletters(mock_get, app, conn):
|
|||
params={'apikey': API_KEY}, status=200)
|
||||
newsletters_ids = []
|
||||
newsletters_texts = []
|
||||
for v in WS_NEWSLETERS_RETURN.itervalues():
|
||||
for v in WS_NEWSLETERS_RETURN.values():
|
||||
for n in v:
|
||||
newsletters_ids.append(n['id_rubrique'])
|
||||
newsletters_texts.append(n['titre'])
|
||||
|
@ -120,7 +120,7 @@ def test_get_subscriptions_to_non_existing_newsletters(mock_get, app, conn):
|
|||
subscriptions = WS_SUBSCRIPTIONS_RETURN['abonnement']
|
||||
subscriptions['push_mail'].append({'id_abonnement': '42', 'auteur': TEST_USER})
|
||||
subs = []
|
||||
for sub in subscriptions.itervalues():
|
||||
for sub in subscriptions.values():
|
||||
subs.extend([i['id_abonnement'] for i in sub])
|
||||
for subscription in resp.json['data']:
|
||||
assert 'text' in subscription
|
||||
|
@ -138,7 +138,7 @@ def test_get_subscriptions(mock_get, app, conn):
|
|||
params={'apikey': API_KEY, 'email': TEST_USER}, status=200)
|
||||
subscriptions = WS_SUBSCRIPTIONS_RETURN['abonnement']
|
||||
subs = []
|
||||
for sub in subscriptions.itervalues():
|
||||
for sub in subscriptions.values():
|
||||
subs.extend([i['id_abonnement'] for i in sub])
|
||||
for subscription in resp.json['data']:
|
||||
assert 'text' in subscription
|
||||
|
|
Loading…
Reference in New Issue