Skip to content

Improve OGC API Process payload metadata, parameter mapping, and test coverage#68

Open
fabricebrito wants to merge 8 commits into
mainfrom
feature/ogc-api-input-fix
Open

Improve OGC API Process payload metadata, parameter mapping, and test coverage#68
fabricebrito wants to merge 8 commits into
mainfrom
feature/ogc-api-input-fix

Conversation

@fabricebrito
Copy link
Copy Markdown

@fabricebrito fabricebrito commented Jun 4, 2026

This PR strengthens the OGC API Process integration in three areas.

It improves service parameter discovery in ogc_api_process.py so more schema shapes are mapped correctly, including GeoJSON/polygon inputs, date-time values, date intervals, array-of-string parameters, and additional numeric/string cases.

It updates OGC job execution payload construction to send both:

  • inputs
  • properties

The properties block now includes the job title, application, and user metadata extracted from the authenticated token when available.

It also adds broad test coverage for the OGC platform implementation, including:

  • parameter type mapping for all supported types
  • job ID splitting
  • status mapping
  • API client creation
  • async job execution payloads
  • job status retrieval
  • STAC and GeoJSON result handling
  • fallback empty collection behavior
  • auth claim extraction helpers

Validation

pytest -q tests/platforms/test_ogc_api_process_platform.py
pytest -q tests/test_auth.py tests/platforms/test_ogc_api_process_platform.py

@fabricebrito fabricebrito requested a review from floeschau June 4, 2026 15:26
@fabricebrito fabricebrito marked this pull request as ready for review June 4, 2026 15:50
@fabricebrito fabricebrito changed the title Feature/ogc api input fix Improve OGC API Process payload metadata, parameter mapping, and test coverage Jun 4, 2026
Copy link
Copy Markdown
Contributor

@floeschau floeschau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants