Skip to content
This repository has been archived by the owner on Oct 2, 2024. It is now read-only.

add webserver smoketest #114

Open
kkuehlz opened this issue Mar 18, 2019 · 0 comments
Open

add webserver smoketest #114

kkuehlz opened this issue Mar 18, 2019 · 0 comments

Comments

@kkuehlz
Copy link
Member

kkuehlz commented Mar 18, 2019

#113 caused jinja to hard fail since the show_topic method didn't have a docstring. This is especially annoying because Marathon uses the webserver to healthcheck the bot—so a broken webserver leaves ircbot in a deploy loop. We can't reasonably expect people to remember to docstring methods when writing plugins. We should have make test run a smoke test that ensures the webserver 200s to catch these errors.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant