From c89909f80767d79fbe674324ee9785abc771f224 Mon Sep 17 00:00:00 2001 From: Jarek Potiuk Date: Mon, 7 Mar 2022 12:04:39 +0100 Subject: [PATCH] Update limits of dependencies after `dask` test disabling Some of the tests failed previously with typing extensions above 4. This PR attempts to relax the limit and check if the problems still appear. Also new tests (S3) started to fail when a new `responses` library version has been released today. So this change also add limits to the responses library in order to make sure the tests pass. Issue https://github.com/getsentry/responses/issues/511 has been opened to raise it to `responses` library maintainers. --- setup.cfg | 5 +---- setup.py | 3 +++ 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/setup.cfg b/setup.cfg index 38d42aaaf4c52..bd5be7e33d0ae 100644 --- a/setup.cfg +++ b/setup.cfg @@ -163,10 +163,7 @@ install_requires = tabulate>=0.7.5 tenacity>=6.2.0 termcolor>=1.1.0 - # typing-extensions can be removed under two scenarios: dropping support for python 3.7 - # or bumping the minimum version of airflow for providers to 2.2.* which would allow the use of airflow.typing_compat - # Kubernetes Tests also rely on typing-extensions < 4.0.0 - fixing the tests should allow to remove the upperbound - typing-extensions>=3.7.4,<4.0.0 + typing-extensions>=3.7.4 unicodecsv>=0.14.1 # Werkzeug is known to cause breaking changes and it is very closely tied with FlaskAppBuilder and other # Flask dependencies and the limit to 1.* line should be reviewed when we upgrade Flask and remove diff --git a/setup.py b/setup.py index d68f0bd04866f..a1a824dc7b8ee 100644 --- a/setup.py +++ b/setup.py @@ -639,6 +639,9 @@ def write_version(filename: str = os.path.join(*[my_dir, "airflow", "git_version 'pytest-xdist', 'python-jose', 'pywinrm', + # The Responses 0.19.0 released on 07.03.2022 break our S3 tests. This limitation should be + # Removed when https://github.com/getsentry/responses/issues/511 is solved. + 'responses<0.19.0', 'qds-sdk>=1.9.6', 'pytest-httpx', 'requests_mock',