urllib request mocking for pystac/load_stac related tests?
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Empieza por tests/rest/test_connection.py alrededor de las líneas 2950-2957 y revisa cómo se utilizan actualmente rutas locales temporales en lugar de recursos HTTP. Investiga una forma actualizada de simular mediante mocks las solicitudes de urllib/urllib3 utilizadas por pystac y, después, cubre las solicitudes fallidas y las URLs relativas para poder probar el comportamiento HTTP real sin archivos locales.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The openEO python client uses the popular requests module for HTTP requests. Testing of that is currently done with the requests_mock module.
For STAC support (e.g. load_stac) we are depending more and more on pystac which uses urllib/urllib3 internally. This is obviously not covered by requests_mock so we need something else as urllib mock utility. I might have missed something obvious but didn't find anything up to date yet
Because of lack of real URL mocking for urrlib, we currently workaround that with local paths to temp files, which is accepted by pystac. e.g.:
https://github.com/Open-EO/openeo-python-client/blob/026881df787de007205e110f765260aeaf2e9876/tests/rest/test_connection.py#L2950-L2957
That works for simple use cases, but it hinders testings of real HTTP functionality (e.g. handling of failed requests, handling relative URLs, ...)
- Lenguaje dominante
- Python
- Estrellas
- 219
- Forks
- 58
- Merge medio
- 2 d 19 h
- PR fusionados (30 d)
- 10
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Open-EO/openeo-python-client
-
JupyterLite supportAbiertoauth feature request
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
Open-EO/openeo-python-client#953 ·
Los mantenedores suelen responder en 1 día
-
Stop providing conda packages?Abiertocollect feedback
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Open-EO/openeo-python-client#952 · 10 comentarios ·
Los mantenedores suelen responder en 1 día
-
Further decouple CubeMetadata from CollectionMetadata, and properly apply spatio-temporal filteringPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Open-EO/openeo-python-client#949 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
asset download: support file:local_path to determine target file namePosiblemente ocupada @soxofaan la tomó hace 48 días. Abiertofeature request
Open-EO/openeo-python-client#902 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Add split_points functionalityPosiblemente ocupada @HansVRP la tomó hace 3 días. Abiertojob manager
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Open-EO/openeo-python-client#899 · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de Open-EO/openeo-python-client
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
jbacalso24/mimry#291 ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 Menos de una hora Aptitud para principiantes 82/100
eli-alvarado/us-watchlist-alerts#14 ·
Los mantenedores suelen responder en 1 día
-
from-review-extraction python severity:nit
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
LearningCircuit/local-deep-research#7462 ·
Los mantenedores suelen responder en 2 días
-
mermaid_extract.py silently discards single-line state-diagram notes; sequence notes are keptAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
cathrynlavery/diagram-design#441 ·
Los mantenedores suelen responder en 2 días
-
[Cleanup] Include the public header in work_split.cppPosiblemente ocupada @blozano-tt la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
tenstorrent/tt-metal#60850 ·
Los mantenedores suelen responder en 1 día