Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WP5 indiv process testing: code style improvements #21

Closed
wants to merge 7 commits into from
Closed
4 changes: 2 additions & 2 deletions src/openeo_test_suite/lib/process_runner/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,15 +38,15 @@ def encode_labeled_array(self, data: Dict) -> Any:
to the internal backend representation openEO process tests.
specification -> backend
"""
raise Exception("labeled arrays not implemented yet")
raise NotImplementedError("labeled arrays not implemented yet")

def encode_datacube(self, data: Dict) -> Any:
"""
Converts a datacube from the JSON object representation (type: datacube) to the
internal backend representation openEO process tests.
specification -> backend
"""
raise Exception("datacubes not implemented yet")
raise NotImplementedError("datacubes not implemented yet")

def encode_data(self, data: Any) -> Any:
"""
Expand Down
2 changes: 1 addition & 1 deletion src/openeo_test_suite/lib/process_runner/dask.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ def encode_data(self, data):
try:
return BoundingBox(**data)
except Exception as e:
raise Exception("Failed to parse bounding box: {}".format(str(e)))
raise ValueError("Failed to parse bounding box") from e

return data

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ def auto_authenticate() -> bool:
def connection(
request, runner: str, auto_authenticate: bool, pytestconfig
) -> ProcessTestRunner:
# TODO: avoid overriding existing "connection" fixture with a different kind of object (ProcessTestRunner)
if runner == "dask":
from openeo_test_suite.lib.process_runner.dask import Dask

Expand Down
Loading