Resolve import conflicts

.travis.yml

@@ -6,7 +6,7 @@ python:
 
 install:
   - python setup.py develop
-  - pip install pytest pytest-cov python-coveralls
+  - pip install -e ."[test]"
 
 script: 
   - pytest --cov=hapic tests

hapic/__init__.py

@@ -1,7 +1,6 @@
 # -*- coding: utf-8 -*-
 from hapic.hapic import Hapic
 from hapic.data import HapicData
-from hapic import ext
 
 _hapic_default = Hapic()
 

hapic/context.py

@@ -52,3 +52,12 @@ class ContextInterface(object):
         :return: OpenAPI path
         """
         raise NotImplementedError()
+
+    def by_pass_output_wrapping(self, response: typing.Any) -> bool:
+        """
+        Return True if the controller response is in final state: we do not
+        have to apply output wrapper on it.
+        :param response: the original response of controller
+        :return:
+        """
+        raise NotImplementedError()

hapic/decorator.py

@@ -5,7 +5,6 @@ from http import HTTPStatus
 
 # TODO BS 20171010: bottle specific !  # see #5
 import marshmallow
-from bottle import HTTPResponse
 from multidict import MultiDict
 
 from hapic.data import HapicData
@@ -215,7 +214,7 @@ class OutputControllerWrapper(InputOutputControllerWrapper):
 
     def after_wrapped_function(self, response: typing.Any) -> typing.Any:
         try:
-            if isinstance(response, HTTPResponse):
+            if self.context.by_pass_output_wrapping(response):
                 return response
 
             processed_response = self.processor.process(response)

hapic/ext/__init__.py

@@ -1,4 +1 @@
 # -*- coding: utf-8 -*-
-from hapic.ext import bottle
-from hapic.ext import pyramid
-from hapic.ext import flask

hapic/ext/bottle/context.py

@@ -112,3 +112,8 @@ class BottleContext(ContextInterface):
 
     def get_swagger_path(self, contextualised_rule: str) -> str:
         return BOTTLE_RE_PATH_URL.sub(r'{\1}', contextualised_rule)
+
+    def by_pass_output_wrapping(self, response: typing.Any) -> bool:
+        if isinstance(response, bottle.HTTPResponse):
+            return True
+        return False

hapic/ext/flask/context.py

@@ -101,3 +101,6 @@ class FlaskContext(ContextInterface):
     def get_swagger_path(self, contextualised_rule: str) -> str:
         # TODO - G.M - 2017-12-05 Check if all route path are handled correctly
         return FLASK_RE_PATH_URL.sub(r'{\1}', contextualised_rule)
+
+    def by_pass_output_wrapping(self, response: typing.Any) -> bool:
+        return False

hapic/ext/pyramid/context.py

@@ -120,3 +120,6 @@ class PyramidContext(ContextInterface):
         # TODO BS 20171110: Pyramid allow route like '/{foo:\d+}', so adapt
         # and USE regular expression (see https://docs.pylonsproject.org/projects/pyramid/en/latest/narr/urldispatch.html#custom-route-predicates)  # nopep8
         return contextualised_rule
+
+    def by_pass_output_wrapping(self, response: typing.Any) -> bool:
+        return False

setup.py

@@ -7,9 +7,6 @@ from os import path
 here = path.abspath(path.dirname(__file__))
 
 install_requires = [
-    # TODO: Bottle will be an extension in future, see #1
-    # TODO: marshmallow an extension too ? see #2
-    'bottle',
     'marshmallow',
     'apispec==0.27.1-algoo',
     'multidict'
@@ -25,8 +22,7 @@ tests_require = [
     'webtest',
 ]
 dev_require = [
-    'requests',
-]
+] + tests_require
 
 setup(
     name='hapic',