Skip to content

Commit

Permalink
Issue #12 try to add logging output to junit report for easier post-m…
Browse files Browse the repository at this point in the history
…orten debugging
  • Loading branch information
soxofaan committed Jan 18, 2024
1 parent 48a940d commit b2a0998
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions pytest.ini
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,4 @@ addopts = --verbose --strict-markers --log-cli-level=DEBUG -ra --numprocesses 3
markers =
requires_custom_processes: for tests that require a backend that includes tests/data/custom_processes.py (like done in submit.sh)
batchjob: tests that require a backend that can launch batch jobs
junit_logging = log
4 changes: 4 additions & 0 deletions tests/test_integration.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,10 @@ def _polygon_bbox(polygon: Polygon) -> dict:


def assert_batch_job(job: BatchJob, assertion: bool, extra_message: str = ""):
# TODO: this helper function breaks the advanced assert handling feature of pytest because
# the original compared values of the condition are lost.
# Its usage often also obfuscates what is actually going wrong in the test
# https://github.com/Open-EO/openeo-geopyspark-integrationtests/issues/12
try:
assert assertion
except AssertionError as e:
Expand Down

0 comments on commit b2a0998

Please sign in to comment.