f4lco
c957fc5b5e
Using json.JSONDecodeError should replace using the 'message' accessor of exceptions, which Python 3 removed. This works as long as "simplejson" is not added as project dependency, because "requests" gives preferential treatment to simplejson, which ships its own version of JSONDecodeError.
54 lines
1.5 KiB
Python
54 lines
1.5 KiB
Python
# -*- encoding: utf-8 -*-
|
|
|
|
import json
|
|
import logging
|
|
import os
|
|
import pytest
|
|
|
|
from stw_potsdam import feed
|
|
from stw_potsdam.config import read_canteen_config
|
|
from stw_potsdam.canteen_api import download_menu
|
|
from stw_potsdam.canteen_api import MenuParams
|
|
|
|
# pragma pylint: disable=invalid-name,redefined-outer-name
|
|
|
|
ENV_ENABLED = 'ENABLE_API_QUERY'
|
|
|
|
# Because log messages are automatically part of the Pytest report, below
|
|
# explicitly avoids adding log handlers via logging#basicConfig, for example.
|
|
logging.getLogger().setLevel(logging.DEBUG)
|
|
|
|
CANTEENS = read_canteen_config()
|
|
|
|
|
|
@pytest.fixture(params=CANTEENS.values(), ids=lambda canteen: canteen.name)
|
|
def canteen(request):
|
|
return request.param
|
|
|
|
|
|
def is_enabled():
|
|
user_enabled = bool(os.getenv(ENV_ENABLED))
|
|
travis_enabled = os.getenv('TRAVIS_EVENT_TYPE') == 'cron'
|
|
return user_enabled or travis_enabled
|
|
|
|
|
|
requires_online_api = pytest.mark.skipif(
|
|
not is_enabled(),
|
|
reason='Querying the online API is disabled. '
|
|
'Turn on by setting env variable %s.' % ENV_ENABLED
|
|
)
|
|
|
|
|
|
@requires_online_api
|
|
def test_retrieval(canteen):
|
|
feed.render_meta(canteen, "/canteens/{}/menu".format(canteen.key))
|
|
params = MenuParams(canteen_id=canteen.id, chash=canteen.chash)
|
|
|
|
try:
|
|
menu = download_menu(params)
|
|
except json.JSONDecodeError as e:
|
|
pytest.xfail('JSON endpoint returned garbage (issue #6)')
|
|
raise e # Appease PyCharm inspection - xfail always raises
|
|
|
|
feed.render_menu(menu)
|