python-epo-ops-client 4.2.0__tar.gz → 4.2.2__tar.gz

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Change Log
 
+## 4.2.2 (2026-05-17)
+
+- Verify support for Python 3.14
+- Made `raise_for_status` configurable. Thanks, @ofipify.
+
+## 4.2.1 (2025-09-09)
+
+- Made network timeout configurable. Thanks, @ofipify.
+
 ## 4.2.0 (2025-07-26)
 
 - Added support for the "legal" service endpoint, providing legal status

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/Makefile

@@ -37,6 +37,7 @@ lint: ## lint the project
 	ruff format --check .
 
 format: ## Run code formatting
+	pyproject-fmt --keep-full-version pyproject.toml
 	ruff format .
 	# Configure Ruff not to auto-fix (remove!):
 	# Ignore unused imports (F401), unused variables (F841), `print` statements (T201), and commented-out code (ERA001).

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-epo-ops-client
-Version: 4.2.0
+Version: 4.2.2
 Summary: Python client for EPO OPS, the European Patent Office's Open Patent Services API.
 Home-page: https://github.com/ip-tools/python-epo-ops-client
 Download-URL: https://pypi.org/project/python-epo-ops-client/#files
@@ -8,11 +8,11 @@ Author: George Song
 Author-email: george@monozuku.com
 Maintainer: Andreas Motl
 Maintainer-email: andreas.motl@ip-tools.org
+License: Apache-2.0
 Keywords: ops,epo,epo-ops,patent-data,patent-office,patent-data-api,european patent office,open patent services
 Classifier: Development Status :: 5 - Production/Stable
 Classifier: Intended Audience :: Developers
 Classifier: Natural Language :: English
-Classifier: License :: OSI Approved :: Apache Software License
 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3.6
 Classifier: Programming Language :: Python :: 3.7
@@ -22,25 +22,27 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Software Development :: Libraries
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: dogpile.cache<1.5
+Requires-Dist: dogpile.cache<1.6
 Requires-Dist: importlib-metadata; python_version < "3.8"
 Requires-Dist: python-dateutil<2.10
 Requires-Dist: requests<3,>=2.27
 Requires-Dist: six<2
 Provides-Extra: develop
-Requires-Dist: ruff<0.13; python_version >= "3.7" and extra == "develop"
+Requires-Dist: pyproject-fmt<3; extra == "develop"
+Requires-Dist: ruff<0.16; python_version >= "3.7" and extra == "develop"
 Requires-Dist: twine<7; extra == "develop"
 Requires-Dist: wheel<1; extra == "develop"
 Provides-Extra: test
-Requires-Dist: pytest<9; extra == "test"
+Requires-Dist: pytest<10; extra == "test"
 Requires-Dist: pytest-cache<2; extra == "test"
-Requires-Dist: pytest-cov<6.3; extra == "test"
-Requires-Dist: python-dotenv<1.2; extra == "test"
-Requires-Dist: responses<0.26; extra == "test"
+Requires-Dist: pytest-cov<7.2; extra == "test"
+Requires-Dist: python-dotenv<1.3; extra == "test"
+Requires-Dist: responses<0.27; extra == "test"
 Dynamic: author
 Dynamic: author-email
 Dynamic: classifier
@@ -49,6 +51,7 @@ Dynamic: description-content-type
 Dynamic: download-url
 Dynamic: home-page
 Dynamic: keywords
+Dynamic: license
 Dynamic: license-file
 Dynamic: maintainer
 Dynamic: maintainer-email
@@ -125,9 +128,11 @@ you'll interact with mostly.
 
 When you issue a request, the response is a [requests.Response][] object. If
 `response.status_code != 200` then a `requests.HTTPError` exception will be
-raised — it's your responsibility to handle those exceptions if you want to. The
-one case that's handled is when the access token has expired: in this case, the
-client will automatically handle the HTTP 400 status and renew the token.
+raised — it's your responsibility to handle those exceptions if you want to.
+This default can be disabled by passing `raise_for_status=False` when
+constructing the client. The one case that's handled is when the access token
+has expired: in this case, the client will automatically handle the HTTP 400
+status and renew the token.
 
 Note that the Client does not attempt to interpret the data supplied by OPS, so
 it's your responsibility to parse the XML or JSON payload for your own purpose.

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/README.md

@@ -67,9 +67,11 @@ you'll interact with mostly.
 
 When you issue a request, the response is a [requests.Response][] object. If
 `response.status_code != 200` then a `requests.HTTPError` exception will be
-raised — it's your responsibility to handle those exceptions if you want to. The
-one case that's handled is when the access token has expired: in this case, the
-client will automatically handle the HTTP 400 status and renew the token.
+raised — it's your responsibility to handle those exceptions if you want to.
+This default can be disabled by passing `raise_for_status=False` when
+constructing the client. The one case that's handled is when the access token
+has expired: in this case, the client will automatically handle the HTTP 400
+status and renew the token.
 
 Note that the Client does not attempt to interpret the data supplied by OPS, so
 it's your responsibility to parse the XML or JSON payload for your own purpose.

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/docs/authors.md

@@ -13,12 +13,17 @@ thank you so much. The list is alphabetically sorted.
 
 ## Contributors
 
+- [Alejandro Marti](https://github.com/MartiONE)
 - [Alexander Meinhardt Scheurer](https://github.com/BeneCollyridam)
 - [Daniel Blasco](https://github.com/dablak)
+- [Dan Bolser](https://github.com/CholoTook)
 - [fe60](https://github.com/fe60)
 - [Felipe Eltermann](https://github.com/eltermann)
 - [Geoffrey Cline](https://github.com/geoffcline)
 - [Hiro Kobashi](https://github.com/kobaski)
+- [Matt Keanny](https://github.com/mattkeanny)
 - [Mike Matheson](https://github.com/mmath)
+- [Oliver Fuerst](https://github.com/ofipify)
 - [Roberto Faga](https://github.com/rfaga)
+- [Sebastian Vilstrup](https://github.com/ipr-sv)
 - [Sotiris Fragkiskos](https://github.com/sfranky)

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/docs/release.md

@@ -15,5 +15,5 @@ To release the package, exercise those steps:
 
 - Push to remote.
   ```shell
-  git push --follow-tags
+  git push && git push --tags
   ```

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/epo_ops/api.py

@@ -10,17 +10,12 @@ from requests.exceptions import HTTPError
 
 from . import exceptions
 from .middlewares import Throttler
-from .models import (
-    NETWORK_TIMEOUT,
-    AccessToken,
-    Docdb,
-    Epodoc,
-    Original,
-    Request,
-)
+from .models import AccessToken, Docdb, Epodoc, Original, Request
 
 log = logging.getLogger(__name__)
 
+DEFAULT_NETWORK_TIMEOUT = 10.0
+
 
 class Client(object):
     __auth_url__ = "https://ops.epo.org/3.2/auth/accesstoken"
@@ -35,14 +30,24 @@ class Client(object):
     __register_path__ = "register"
     __register_search_path__ = "register/search"
 
-    def __init__(self, key, secret, accept_type="xml", middlewares=None):
+    def __init__(
+        self,
+        key,
+        secret,
+        accept_type="xml",
+        middlewares=None,
+        timeout=DEFAULT_NETWORK_TIMEOUT,
+        raise_for_status=True,
+    ):
         self.accept_type = "application/{0}".format(accept_type)
         self.middlewares = middlewares
         if middlewares is None:
             self.middlewares = [Throttler()]
-        self.request = Request(self.middlewares)
+        self.request = Request(self.middlewares, timeout)
         self.key = key
         self.secret = secret
+        self.timeout = timeout
+        self.raise_for_status = raise_for_status
         self._access_token = None
 
     def family(
@@ -153,6 +158,7 @@ class Client(object):
                 input=input,
             )
         )
+
     def number(
         self,
         reference_type: str,
@@ -351,7 +357,7 @@ class Client(object):
             self.__auth_url__,
             headers=headers,
             data=payload,
-            timeout=NETWORK_TIMEOUT,
+            timeout=self.timeout,
         )
         response.raise_for_status()
         self._access_token = AccessToken(response)
@@ -394,7 +400,8 @@ class Client(object):
         )
         response = self._check_for_expired_token(response)
         response = self._check_for_exceeded_quota(response)
-        response.raise_for_status()
+        if self.raise_for_status:
+            response.raise_for_status()
         return response
 
     # info: {

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/epo_ops/models.py

@@ -11,9 +11,6 @@ from .utils import quote, validate_date
 log = logging.getLogger(__name__)
 
 
-NETWORK_TIMEOUT = 10.0
-
-
 def _prepare_part(part):
     return "({0})".format(quote(part))
 
@@ -68,8 +65,9 @@ class AccessToken(object):
 
 
 class Request(object):
-    def __init__(self, middlewares):
+    def __init__(self, middlewares, timeout=None):
         self.middlewares = middlewares
+        self.timeout = timeout
         self.reset_env()
 
     @property
@@ -86,10 +84,10 @@ class Request(object):
         self.env.update(self.default_env)
 
     def post(self, url, data=None, **kwargs):
-        return self._request(_post_callback, url, data, **kwargs)
+        return self._request(self._post_callback, url, data, **kwargs)
 
     def get(self, url, data=None, **kwargs):
-        return self._request(_get_callback, url, data, **kwargs)
+        return self._request(self._get_callback, url, data, **kwargs)
 
     def _request(self, callback, url, data=None, **kwargs):
         self.reset_env()
@@ -110,10 +108,8 @@ class Request(object):
         self.reset_env()
         return response
 
+    def _post_callback(self, url, data, **kwargs):
+        return requests.post(url, data, **kwargs, timeout=self.timeout)
 
-def _post_callback(url, data, **kwargs):
-    return requests.post(url, data, **kwargs, timeout=NETWORK_TIMEOUT)
-
-
-def _get_callback(url, data, **kwargs):
-    return requests.get(url, **kwargs, timeout=NETWORK_TIMEOUT)
+    def _get_callback(self, url, data, **kwargs):
+        return requests.get(url, **kwargs, timeout=self.timeout)

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/pyproject.toml

@@ -1,63 +1,45 @@
-# ==================
-# Build system setup
-# ==================
-
 [build-system]
+build-backend = "setuptools.build_meta"
 requires = [
   "setuptools>=42", # At least v42 of setuptools required!
   "versioningit",
 ]
-build-backend = "setuptools.build_meta"
-
-[tool.pytest.ini_options]
-minversion = "2.0"
-addopts = """
-  -rsfEX -p pytester --strict-markers --verbosity=3
-  --cov=epo_ops tests --cov-report=term-missing --cov-report=xml
-  """
-log_level = "DEBUG"
-log_cli_level = "DEBUG"
-testpaths = ["tests"]
-xfail_strict = true
-markers = []
 
 [tool.ruff]
 line-length = 80
-
+extend-exclude = [ "__init__.py" ]
 lint.select = [
-  # Bandit
-  "S",
-  # Bugbear
-  "B",
   # Builtins
   "A",
+  # Bugbear
+  "B",
+  "B9",
   # comprehensions
   "C",
+  # Pycodestyle
+  "E",
   # eradicate
   "ERA",
-  # flake8-2020
-  "YTT",
+  # Pyflakes
+  "F",
   # isort
   "I",
   # pandas-vet
   "PD",
-  # print
-  "T20",
-  # Pycodestyle
-  "E",
-  "W",
-  # Pyflakes
-  "F",
   # return
   "RET",
+  # Bandit
+  "S",
   # from `.flake8` file
-  "T",  # T4
-  "B9",
+  "T",   # T4
+  # print
+  "T20",
+  "W",
+  # flake8-2020
+  "YTT",
 ]
-
-extend-exclude = ["__init__.py"]
-
 lint.ignore = [
+  "B905",   # B905 `zip()` without an explicit `strict=` parameter
   "E203",
   "E266",
   "E501",
@@ -65,12 +47,25 @@ lint.ignore = [
   "RET505", # Unnecessary `else` after `return` statement
   # "W503",  # Unknown rule selector
 ]
-
+# FIXME: Improve this situation wrt. SQL injection, even it is not be an actual attack vector.
+lint.per-file-ignores."*sqlite.py" = [
+  "S608", # Possible SQL injection vector through string-based query construction
+]
+lint.per-file-ignores."epo_ops/api.py" = [
+  "A001", # Variable `range` is shadowing a Python builtin
+  "A002", # Argument `input` is shadowing a Python builtin
+  "C408", # Unnecessary `dict` call (rewrite as a literal)
+]
+lint.per-file-ignores."tests/*" = [
+  "S101", # Use of `assert` detected
+]
+lint.per-file-ignores."tests/middlewares/throttle/conftest.py" = [
+  "S311", # Standard pseudo-random generators are not suitable for cryptographic purposes
+]
 # from `.isort.cfg` file
-[tool.ruff.lint.isort]
-combine-as-imports = true
-force-wrap-aliases = true
-known-third-party = [
+lint.isort.combine-as-imports = true
+lint.isort.force-wrap-aliases = true
+lint.isort.known-third-party = [
   "dateutil",
   "dogpile",
   "dotenv",
@@ -78,34 +73,24 @@ known-third-party = [
   "requests",
   "six",
 ]
-split-on-trailing-comma = false
-
+lint.isort.split-on-trailing-comma = false
 # from `.flake8` file
-[tool.ruff.lint.mccabe]
 # Flag errors (`C901`) whenever the complexity level exceeds the configured value.
-max-complexity = 7
-
-
-[tool.ruff.lint.per-file-ignores]
-"tests/*" = [
-  "S101", # Use of `assert` detected
-]
-"epo_ops/api.py" = [
-  "A001", # Variable `range` is shadowing a Python builtin
-  "A002", # Argument `input` is shadowing a Python builtin
-  "C408", # Unnecessary `dict` call (rewrite as a literal)
-]
-# FIXME: Improve this situation wrt. SQL injection, even it is not be an actual attack vector.
-"*sqlite.py" = [
-  "S608", # Possible SQL injection vector through string-based query construction
-]
-"tests/middlewares/throttle/conftest.py" = [
-  "S311", # Standard pseudo-random generators are not suitable for cryptographic purposes
-]
+lint.mccabe.max-complexity = 7
 
+[tool.pytest]
+ini_options.minversion = "2.0"
+ini_options.addopts = """
+  -rsfEX -p pytester --strict-markers --verbosity=3
+  --cov=epo_ops tests --cov-report=term-missing --cov-report=xml
+  """
+ini_options.log_level = "DEBUG"
+ini_options.log_cli_level = "DEBUG"
+ini_options.testpaths = [ "tests" ]
+ini_options.xfail_strict = true
+ini_options.markers = []
 
 [tool.versioningit]
-
-[tool.versioningit.vcs]
-method = "git"
-default-tag = "0.0.0"
+vcs.method = "git-archive"
+vcs.default-tag = "v0.0.0"
+vcs.describe-subst = "$Format:%(describe:tags,match=v*)$"

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/python_epo_ops_client.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-epo-ops-client
-Version: 4.2.0
+Version: 4.2.2
 Summary: Python client for EPO OPS, the European Patent Office's Open Patent Services API.
 Home-page: https://github.com/ip-tools/python-epo-ops-client
 Download-URL: https://pypi.org/project/python-epo-ops-client/#files
@@ -8,11 +8,11 @@ Author: George Song
 Author-email: george@monozuku.com
 Maintainer: Andreas Motl
 Maintainer-email: andreas.motl@ip-tools.org
+License: Apache-2.0
 Keywords: ops,epo,epo-ops,patent-data,patent-office,patent-data-api,european patent office,open patent services
 Classifier: Development Status :: 5 - Production/Stable
 Classifier: Intended Audience :: Developers
 Classifier: Natural Language :: English
-Classifier: License :: OSI Approved :: Apache Software License
 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3.6
 Classifier: Programming Language :: Python :: 3.7
@@ -22,25 +22,27 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Software Development :: Libraries
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: dogpile.cache<1.5
+Requires-Dist: dogpile.cache<1.6
 Requires-Dist: importlib-metadata; python_version < "3.8"
 Requires-Dist: python-dateutil<2.10
 Requires-Dist: requests<3,>=2.27
 Requires-Dist: six<2
 Provides-Extra: develop
-Requires-Dist: ruff<0.13; python_version >= "3.7" and extra == "develop"
+Requires-Dist: pyproject-fmt<3; extra == "develop"
+Requires-Dist: ruff<0.16; python_version >= "3.7" and extra == "develop"
 Requires-Dist: twine<7; extra == "develop"
 Requires-Dist: wheel<1; extra == "develop"
 Provides-Extra: test
-Requires-Dist: pytest<9; extra == "test"
+Requires-Dist: pytest<10; extra == "test"
 Requires-Dist: pytest-cache<2; extra == "test"
-Requires-Dist: pytest-cov<6.3; extra == "test"
-Requires-Dist: python-dotenv<1.2; extra == "test"
-Requires-Dist: responses<0.26; extra == "test"
+Requires-Dist: pytest-cov<7.2; extra == "test"
+Requires-Dist: python-dotenv<1.3; extra == "test"
+Requires-Dist: responses<0.27; extra == "test"
 Dynamic: author
 Dynamic: author-email
 Dynamic: classifier
@@ -49,6 +51,7 @@ Dynamic: description-content-type
 Dynamic: download-url
 Dynamic: home-page
 Dynamic: keywords
+Dynamic: license
 Dynamic: license-file
 Dynamic: maintainer
 Dynamic: maintainer-email
@@ -125,9 +128,11 @@ you'll interact with mostly.
 
 When you issue a request, the response is a [requests.Response][] object. If
 `response.status_code != 200` then a `requests.HTTPError` exception will be
-raised — it's your responsibility to handle those exceptions if you want to. The
-one case that's handled is when the access token has expired: in this case, the
-client will automatically handle the HTTP 400 status and renew the token.
+raised — it's your responsibility to handle those exceptions if you want to.
+This default can be disabled by passing `raise_for_status=False` when
+constructing the client. The one case that's handled is when the access token
+has expired: in this case, the client will automatically handle the HTTP 400
+status and renew the token.
 
 Note that the Client does not attempt to interpret the data supplied by OPS, so
 it's your responsibility to parse the XML or JSON payload for your own purpose.

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/python_epo_ops_client.egg-info/SOURCES.txt

@@ -37,4 +37,5 @@ python_epo_ops_client.egg-info/top_level.txt
 tests/test_api.py
 tests/test_models.py
 tests/test_ops_quota.py
+tests/test_raise_for_status.py
 tests/test_utils.py

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/python_epo_ops_client.egg-info/requires.txt

@@ -1,4 +1,4 @@
-dogpile.cache<1.5
+dogpile.cache<1.6
 python-dateutil<2.10
 requests<3,>=2.27
 six<2
@@ -7,15 +7,16 @@ six<2
 importlib-metadata
 
 [develop]
+pyproject-fmt<3
 twine<7
 wheel<1
 
 [develop:python_version >= "3.7"]
-ruff<0.13
+ruff<0.16
 
 [test]
-pytest<9
+pytest<10
 pytest-cache<2
-pytest-cov<6.3
-python-dotenv<1.2
-responses<0.26
+pytest-cov<7.2
+python-dotenv<1.3
+responses<0.27

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/setup.py

@@ -1,7 +1,7 @@
 import codecs
 from os import path
 
-from setuptools import setup
+from setuptools import find_packages, setup
 from versioningit import get_cmdclasses
 
 here = path.abspath(path.dirname(__file__))
@@ -15,6 +15,7 @@ setup(
     description=(
         "Python client for EPO OPS, the European Patent Office's Open Patent Services API."
     ),
+    license="Apache-2.0",
     long_description_content_type="text/markdown",
     long_description=readme,
     author="George Song",
@@ -23,11 +24,12 @@ setup(
     maintainer_email="andreas.motl@ip-tools.org",
     url="https://github.com/ip-tools/python-epo-ops-client",
     download_url="https://pypi.org/project/python-epo-ops-client/#files",
-    packages=["epo_ops", "epo_ops.middlewares"],
-    package_dir={"epo_ops": "epo_ops"},
+    packages=find_packages(
+        include=["epo_ops*"],
+    ),
     include_package_data=True,
     install_requires=[
-        "dogpile.cache<1.5",
+        "dogpile.cache<1.6",
         "importlib-metadata; python_version<'3.8'",
         "python-dateutil<2.10",
         "requests>=2.27,<3",
@@ -35,16 +37,17 @@ setup(
     ],
     extras_require={
         "develop": [
-            "ruff<0.13; python_version >= '3.7'",
+            "pyproject-fmt<3",
+            "ruff<0.16; python_version >= '3.7'",
             "twine<7",
             "wheel<1",
         ],
         "test": [
-            "pytest<9",
+            "pytest<10",
             "pytest-cache<2",
-            "pytest-cov<6.3",
-            "python-dotenv<1.2",
-            "responses<0.26",
+            "pytest-cov<7.2",
+            "python-dotenv<1.3",
+            "responses<0.27",
         ],
     },
     zip_safe=False,
@@ -52,7 +55,6 @@ setup(
         "Development Status :: 5 - Production/Stable",
         "Intended Audience :: Developers",
         "Natural Language :: English",
-        "License :: OSI Approved :: Apache Software License",
         "Programming Language :: Python",
         "Programming Language :: Python :: 3.6",
         "Programming Language :: Python :: 3.7",
@@ -62,6 +64,7 @@ setup(
         "Programming Language :: Python :: 3.11",
         "Programming Language :: Python :: 3.12",
         "Programming Language :: Python :: 3.13",
+        "Programming Language :: Python :: 3.14",
         "Topic :: Software Development :: Libraries",
         "Topic :: Software Development :: Libraries :: Python Modules",
     ],

{python_epo_ops_client-4.2.0 → python_epo_ops_client-4.2.2}/tests/test_api.py

@@ -28,6 +28,12 @@ def test_instantiate_simple_client():
     client = Client("key", "secret")
     assert len(client.middlewares) == 1
     assert client.middlewares[0].history.db_path == sqlite.DEFAULT_DB_PATH
+    assert client.raise_for_status is True
+
+
+def test_instantiate_client_without_raise_for_status():
+    client = Client("key", "secret", raise_for_status=False)
+    assert client.raise_for_status is False
 
 
 def test_family(all_clients):
@@ -45,9 +51,11 @@ def test_family_legal(all_clients):
 def test_image(all_clients):
     assert_image_success(all_clients)
 
+
 def test_legal(all_clients):
     assert_legal_success(all_clients)
 
+
 def test_published_data(all_clients):
     assert_published_data_success(all_clients)
 

python_epo_ops_client-4.2.2/tests/test_raise_for_status.py

@@ -0,0 +1,67 @@
+import pytest
+import responses
+from pytest import raises
+from requests.exceptions import HTTPError
+
+from epo_ops.api import Client
+from epo_ops.models import Docdb
+
+
+@pytest.fixture
+def ops_backend_413():
+    """
+    Emulate an OPS backend returning 413 on the fulltext endpoint for an
+    ambiguous input. The real upstream returns 413 for e.g. EP.0536425/fulltext.
+    """
+    token = responses.Response(
+        responses.POST,
+        url="https://ops.epo.org/3.2/auth/accesstoken",
+        status=200,
+        json={"access_token": "foo", "expires_in": 42},
+    )
+    fulltext_413 = responses.Response(
+        responses.POST,
+        url="https://ops.epo.org/3.2/rest-services/published-data/publication/docdb/fulltext",
+        status=413,
+        headers={"Content-Type": "application/xml"},
+        body=(
+            '<?xml version="1.0" encoding="UTF-8" standalone="yes"?>\n'
+            '  <fault xmlns="http://ops.epo.org">\n'
+            "    <code>CLIENT.AmbiguousRequest</code>\n"
+            "    <message>The request was ambiguous</message>\n"
+            "    <details>\n"
+            "      <cause>Ambiguous input: publication/docdb/EP.0536425</cause>\n"
+            "      <resolution>publication/docdb/EP.0536425.A1</resolution>\n"
+            "      <resolution>publication/docdb/EP.0536425.A4</resolution>\n"
+            "      <resolution>publication/docdb/EP.0536425.B1</resolution>\n"
+            "      <resolution>publication/docdb/EP.0536425.B2</resolution>\n"
+            "    </details>\n"
+            "  </fault>"
+        ),
+    )
+    for response in [token, fulltext_413]:
+        responses.add(response)
+
+
+def _issue_fulltext_request(client):
+    return client.published_data(
+        "publication",
+        Docdb("0536425", "EP", "B1"),
+        endpoint="fulltext",
+    )
+
+
+@responses.activate
+def test_413_raises_by_default(ops_backend_413):
+    client = Client("key", "secret", middlewares=[])
+    with raises(HTTPError):
+        _issue_fulltext_request(client)
+
+
+@responses.activate
+def test_413_returned_when_raise_for_status_disabled(ops_backend_413):
+    client = Client("key", "secret", middlewares=[], raise_for_status=False)
+    response = _issue_fulltext_request(client)
+    assert response.status_code == 413
+    assert "CLIENT.AmbiguousRequest" in response.text
+    assert "publication/docdb/EP.0536425.B1" in response.text