Merge pull request #69 from p1c2u/kcem-master

Python2 compatibility module

.gitignore

@@ -2,6 +2,7 @@
 __pycache__/
 *.py[cod]
 *$py.class
+.pytest_cache/
 
 # C extensions
 *.so

.travis.yml

@@ -2,6 +2,7 @@ language: python
 sudo: false
 matrix:
   include:
+  - python: 2.7
   - python: 3.4
   - python: 3.5
   - python: 3.6

openapi_core/__init__.py

@@ -1,3 +1,4 @@
+# -*- coding: utf-8 -*-
 """OpenAPI core module"""
 from openapi_core.shortcuts import (
     create_spec, validate_parameters, validate_body, validate_data,

openapi_core/compat.py

@@ -0,0 +1,12 @@
+"""OpenAPI core python 2.7 compatibility module"""
+try:
+    from functools import lru_cache
+except ImportError:
+    from backports.functools_lru_cache import lru_cache
+
+try:
+    from functools import partialmethod
+except ImportError:
+    from backports.functools_partialmethod import partialmethod
+
+__all__ = ['lru_cache', 'partialmethod']

openapi_core/schema/components/factories.py

@@ -1,5 +1,4 @@
-from functools import lru_cache
+from openapi_core.compat import lru_cache
-
 from openapi_core.schema.components.models import Components
 from openapi_core.schema.schemas.generators import SchemasGenerator
 

openapi_core/schema/operations/generators.py

@@ -1,10 +1,9 @@
 # -*- coding: utf-8 -*-
 """OpenAPI core operations models module"""
-from functools import lru_cache
-
 from six import iteritems
 from openapi_spec_validator.validators import PathItemValidator
 
+from openapi_core.compat import lru_cache
 from openapi_core.schema.operations.models import Operation
 from openapi_core.schema.parameters.generators import ParametersGenerator
 from openapi_core.schema.request_bodies.factories import RequestBodyFactory

openapi_core/schema/parameters/generators.py

@@ -1,8 +1,7 @@
 """OpenAPI core parameters generators module"""
-from functools import lru_cache
-
 from six import iteritems
 
+from openapi_core.compat import lru_cache
 from openapi_core.schema.parameters.factories import ParameterFactory
 
 

openapi_core/schema/paths/generators.py

@@ -1,8 +1,7 @@
 """OpenAPI core paths generators module"""
-from functools import lru_cache
-
 from six import iteritems
 
+from openapi_core.compat import lru_cache
 from openapi_core.schema.operations.generators import OperationsGenerator
 from openapi_core.schema.paths.models import Path
 

openapi_core/schema/request_bodies/factories.py

@@ -1,6 +1,5 @@
 """OpenAPI core request bodies factories module"""
-from functools import lru_cache
+from openapi_core.compat import lru_cache
-
 from openapi_core.schema.media_types.generators import MediaTypeGenerator
 from openapi_core.schema.request_bodies.models import RequestBody
 

openapi_core/schema/responses/generators.py

@@ -1,8 +1,7 @@
 """OpenAPI core responses generators module"""
-from functools import lru_cache
-
 from six import iteritems
 
+from openapi_core.compat import lru_cache
 from openapi_core.schema.media_types.generators import MediaTypeGenerator
 from openapi_core.schema.parameters.generators import ParametersGenerator
 from openapi_core.schema.responses.models import Response

openapi_core/schema/schemas/factories.py

@@ -1,7 +1,7 @@
 """OpenAPI core schemas factories module"""
 import logging
-from functools import lru_cache
 
+from openapi_core.compat import lru_cache
 from openapi_core.schema.properties.generators import PropertiesGenerator
 from openapi_core.schema.schemas.models import Schema
 

openapi_core/schema/schemas/models.py

@@ -81,7 +81,7 @@ class Schema(object):
         )
 
     def get_all_required_properties_names(self):
-        required = self.required.copy()
+        required = self.required[:]
 
         for subschema in self.all_of:
             subschema_req = subschema.get_all_required_properties()

openapi_core/schema/schemas/util.py

@@ -1,10 +1,11 @@
 """OpenAPI core schemas util module"""
 from distutils.util import strtobool
 from json import dumps
+from six import string_types
 
 
 def forcebool(val):
-    if isinstance(val, str):
+    if isinstance(val, string_types):
         val = strtobool(val)
 
     return bool(val)

openapi_core/schema/servers/generators.py

@@ -1,8 +1,7 @@
 """OpenAPI core servers generators module"""
-from functools import lru_cache
-
 from six import iteritems
 
+from openapi_core.compat import lru_cache
 from openapi_core.schema.servers.models import Server, ServerVariable
 
 

openapi_core/schema/specs/factories.py

@@ -1,9 +1,9 @@
 # -*- coding: utf-8 -*-
 """OpenAPI core specs factories module"""
-from functools import lru_cache
 
 from openapi_spec_validator import openapi_v3_spec_validator
 
+from openapi_core.compat import lru_cache
 from openapi_core.schema.components.factories import ComponentsFactory
 from openapi_core.schema.infos.factories import InfoFactory
 from openapi_core.schema.paths.generators import PathsGenerator

openapi_core/schema/specs/models.py

@@ -1,8 +1,8 @@
 # -*- coding: utf-8 -*-
 """OpenAPI core specs models module"""
 import logging
-from functools import partialmethod
 
+from openapi_core.compat import partialmethod
 from openapi_core.schema.operations.exceptions import InvalidOperation
 from openapi_core.schema.servers.exceptions import InvalidServer
 

openapi_core/validation/util.py

@@ -1,5 +1,17 @@
 """OpenAPI core validation util module"""
-from yarl import URL
+from six.moves.urllib.parse import urlparse
+
+
+def is_absolute(url):
+    return url.startswith('//') or '://' in url
+
+
+def path_qs(url):
+    pr = urlparse(url)
+    result = pr.path
+    if pr.query:
+        result += '?' + pr.query
+    return result
 
 
 def get_operation_pattern(server_url, request_url_pattern):
@@ -7,6 +19,6 @@ def get_operation_pattern(server_url, request_url_pattern):
     if server_url[-1] == "/":
         # operations have to start with a slash, so do not remove it
         server_url = server_url[:-1]
-    if URL(server_url).is_absolute():
+    if is_absolute(server_url):
         return request_url_pattern.replace(server_url, "", 1)
-    return URL(request_url_pattern).path_qs.replace(server_url, "", 1)
+    return path_qs(request_url_pattern).replace(server_url, "", 1)

requirements.txt

@@ -1,4 +1,3 @@
 openapi-spec-validator
 six
-yarl<1.2.0
 lazy-object-proxy

requirements_2.7.txt

@@ -0,0 +1,6 @@
+openapi-spec-validator
+six
+lazy-object-proxy
+backports.functools-lru-cache
+backports.functools-partialmethod
+enum34

setup.py

@@ -25,6 +25,11 @@ def get_metadata(init_file):
     return dict(re.findall("__([a-z]+)__ = '([^']+)'", init_file))
 
 
+def install_requires():
+    py27 = '_2.7' if sys.version_info < (3,) else ''
+    return read_requirements('requirements{}.txt'.format(py27))
+
+
 class PyTest(TestCommand):
 
     """Command to run unit tests after in-place build."""
@@ -68,12 +73,13 @@ setup(
         'Intended Audience :: Developers',
         "Topic :: Software Development :: Libraries :: Python Modules",
         "Operating System :: OS Independent",
+        'Programming Language :: Python :: 2.7',
         'Programming Language :: Python :: 3.4',
         'Programming Language :: Python :: 3.5',
         'Programming Language :: Python :: 3.6',
         'Topic :: Software Development :: Libraries',
     ],
-    install_requires=read_requirements('requirements.txt'),
+    install_requires=install_requires(),
     tests_require=read_requirements('requirements_dev.txt'),
     extras_require={
         'flask':  ["werkzeug"],

tests/unit/validation/test_util.py

@@ -0,0 +1,18 @@
+from openapi_core.validation.util import path_qs
+
+
+class TestPathQs(object):
+
+    def test_path(self):
+        url = 'https://test.com:1234/path'
+
+        result = path_qs(url)
+
+        assert result == '/path'
+
+    def test_query(self):
+        url = 'https://test.com:1234/path?query=1'
+
+        result = path_qs(url)
+
+        assert result == '/path?query=1'