Skip to content

Commit

Permalink
Issue #195 flag 1.2 API version as production ready
Browse files Browse the repository at this point in the history
  • Loading branch information
soxofaan committed Nov 14, 2023
1 parent 3b73a7b commit 97d5192
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 12 deletions.
2 changes: 1 addition & 1 deletion openeo_driver/_version.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
__version__ = "0.76.1a1"
__version__ = "0.77.0a1"
6 changes: 3 additions & 3 deletions openeo_driver/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -49,10 +49,10 @@
"1.0.1": ApiVersionInfo(version="1.0.1", supported=True, wellknown=False, production=True),
"1.1.0": ApiVersionInfo(version="1.1.0", supported=True, wellknown=False, production=False),
"1.1": ApiVersionInfo(version="1.1.0", supported=True, wellknown=True, production=True),
"1": ApiVersionInfo(version="1.1.0", supported=True, wellknown=False, production=True),
"1.2": ApiVersionInfo(version="1.2.0", supported=True, wellknown=True, production=False),
"1.2": ApiVersionInfo(version="1.2.0", supported=True, wellknown=True, production=True),
"1": ApiVersionInfo(version="1.2.0", supported=True, wellknown=False, production=True),
}
API_VERSION_DEFAULT = "1.1.0"
API_VERSION_DEFAULT = "1.2"

_log.info("API Versions: {v}".format(v=API_VERSIONS))
_log.info("Default API Version: {v}".format(v=API_VERSION_DEFAULT))
Expand Down
21 changes: 13 additions & 8 deletions tests/test_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ def test_well_known_openeo(self, client):
assert by_api_version == {
"1.0.0": {"api_version": "1.0.0", "production": True, "url": "http://oeo.net/openeo/1.0/"},
"1.1.0": {"api_version": "1.1.0", "production": True, "url": "http://oeo.net/openeo/1.1/"},
"1.2.0": {"api_version": "1.2.0", "production": False, "url": "http://oeo.net/openeo/1.2/"},
"1.2.0": {"api_version": "1.2.0", "production": True, "url": "http://oeo.net/openeo/1.2/"},
}
assert resp.headers["Cache-Control"] == "max-age=900, public"

Expand All @@ -154,13 +154,18 @@ def test_https_proxy_handling(self, client, headers, expected):
for url in [v["url"] for v in resp.json["versions"]]:
assert url.startswith(expected)

@pytest.mark.parametrize(["url", "expected_version"], [
("/openeo/1.0/", "1.0.0"),
("/openeo/1.0.0/", "1.0.0"),
("/openeo/1.1/", "1.1.0"),
("/openeo/1.1.0/", "1.1.0"),
("/openeo/", "1.1.0"),
])
@pytest.mark.parametrize(
["url", "expected_version"],
[
("/openeo/1.0/", "1.0.0"),
("/openeo/1.0.0/", "1.0.0"),
("/openeo/1.1/", "1.1.0"),
("/openeo/1.1.0/", "1.1.0"),
("/openeo/1.2/", "1.2.0"),
("/openeo/1/", "1.2.0"),
("/openeo/", "1.2.0"),
],
)
def test_versioned_urls(self, client, url, expected_version):
resp = client.get(url)
assert resp.status_code == 200
Expand Down

0 comments on commit 97d5192

Please sign in to comment.