autoland: remove pycharm testrunner PYTHONPATH hack (
bug 1337517) r?smacleod
Removed the pycharm testrunner PYTHONPATH hack in our test suite. The tests now pass without it.
MozReview-Commit-ID: IfmO5VjhiCI
--- a/autoland/webapi/tests/test_dockerflow_container_api.py
+++ b/autoland/webapi/tests/test_dockerflow_container_api.py
@@ -5,22 +5,17 @@
Tests to ensure we honour the Dockerflow container API specification.
See https://github.com/mozilla-services/Dockerflow for details.
"""
import json
import pytest
-# FIXME: need to get pycharm test runner to add /app to the container
-# PYTHONPATH
-import sys
-sys.path.insert(0, '/app')
-
-from autolandweb.server import make_app # noqa
+from autolandweb.server import make_app
@pytest.fixture
def app():
"""Returns the tornado.Application instance we'll be testing against.
Required for pytest-tornado to function.
"""
--- a/autoland/webapi/tests/test_repo_check.py
+++ b/autoland/webapi/tests/test_repo_check.py
@@ -5,17 +5,18 @@
Test repository handling logic.
"""
from autolandweb.server import make_app
from autolandweb.testing import MountebankClient
import pytest
-# FIXME: need to get pycharm test runner to add /app to the container PYTHONPATH # noqa
+# FIXME: need to get pycharm test runner to add /app to the container
+# PYTHONPATH
import sys
sys.path.insert(0, '/app')
class FakeReviewBoard:
def __init__(self, mountebank_client):
self.mountebank = mountebank_client