misc: fix consider-iterating-dictionary pylint error (#56288)
This commit is contained in:
parent
f16a542959
commit
ed8ca23d0c
|
@ -834,18 +834,18 @@ def test_wcs_search_engines(settings, app):
|
|||
settings.KNOWN_SERVICES = {}
|
||||
search_engines = engines.get_engines()
|
||||
assert 'tracking-code' not in search_engines.keys()
|
||||
assert len([x for x in search_engines.keys() if x.startswith('formdata:')]) == 0
|
||||
assert len([x for x in search_engines.keys() if x.startswith('cards:')]) == 0
|
||||
assert len([x for x in search_engines if x.startswith('formdata:')]) == 0
|
||||
assert len([x for x in search_engines if x.startswith('cards:')]) == 0
|
||||
|
||||
settings.KNOWN_SERVICES = {'wcs': {'default': {'title': 'test', 'url': 'http://127.0.0.1:8999/'}}}
|
||||
|
||||
settings.TEMPLATE_VARS['is_portal_agent'] = False
|
||||
search_engines = engines.get_engines()
|
||||
assert len([x for x in search_engines.keys() if x.startswith('formdata:')]) == 0
|
||||
assert len([x for x in search_engines if x.startswith('formdata:')]) == 0
|
||||
|
||||
settings.TEMPLATE_VARS['is_portal_agent'] = True
|
||||
search_engines = engines.get_engines()
|
||||
assert len([x for x in search_engines.keys() if x.startswith('formdata:')]) == 1
|
||||
assert len([x for x in search_engines if x.startswith('formdata:')]) == 1
|
||||
for key, engine in search_engines.items():
|
||||
if key.startswith('formdata:'):
|
||||
assert '&include-anonymised=off' in engine['url']
|
||||
|
@ -856,16 +856,16 @@ def test_wcs_search_engines(settings, app):
|
|||
# no card model found
|
||||
mock_wcs.return_value = {}
|
||||
search_engines = engines.get_engines()
|
||||
assert len([x for x in search_engines.keys() if x.startswith('cards:')]) == 0
|
||||
assert len([x for x in search_engines if x.startswith('cards:')]) == 0
|
||||
assert mock_wcs.call_args_list[0][0][1] == 'api/cards/@list'
|
||||
mock_wcs.return_value = {'data': []}
|
||||
search_engines = engines.get_engines()
|
||||
assert len([x for x in search_engines.keys() if x.startswith('cards:')]) == 0
|
||||
assert len([x for x in search_engines if x.startswith('cards:')]) == 0
|
||||
|
||||
# card model found, but related page does not exist
|
||||
mock_wcs.return_value = {'data': [{'id': 'card-bar', 'text': 'Card Bar'}]}
|
||||
search_engines = engines.get_engines()
|
||||
assert len([x for x in search_engines.keys() if x.startswith('cards:')]) == 0
|
||||
assert len([x for x in search_engines if x.startswith('cards:')]) == 0
|
||||
|
||||
# related page exists
|
||||
page = Page.objects.create(slug='bar', title='Bar')
|
||||
|
@ -873,7 +873,7 @@ def test_wcs_search_engines(settings, app):
|
|||
page.sub_slug = sub_slug
|
||||
page.save()
|
||||
search_engines = engines.get_engines()
|
||||
assert len([x for x in search_engines.keys() if x.startswith('cards:')]) == 1
|
||||
assert len([x for x in search_engines if x.startswith('cards:')]) == 1
|
||||
assert 'cards:c21f969b:card-bar' in search_engines.keys()
|
||||
card_engine = search_engines['cards:c21f969b:card-bar']
|
||||
assert card_engine['url'] == (
|
||||
|
|
|
@ -2171,11 +2171,11 @@ def test_wcs_search_engines(app):
|
|||
settings.TEMPLATE_VARS['is_portal_agent'] = True
|
||||
search_engines = engines.get_engines()
|
||||
assert 'tracking-code' in search_engines.keys()
|
||||
assert len([x for x in search_engines.keys() if x.startswith('formdata:')]) == 2
|
||||
assert len([x for x in search_engines if x.startswith('formdata:')]) == 2
|
||||
settings.TEMPLATE_VARS['is_portal_agent'] = False
|
||||
search_engines = engines.get_engines()
|
||||
assert 'tracking-code' not in search_engines.keys()
|
||||
assert len([x for x in search_engines.keys() if x.startswith('formdata:')]) == 0
|
||||
assert len([x for x in search_engines if x.startswith('formdata:')]) == 0
|
||||
|
||||
|
||||
def test_backoffice_submission_cell_render(context):
|
||||
|
|
Loading…
Reference in New Issue