Integration tests don't check screenshots
Los mantenedores suelen responder en 5 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 55/100
Línea de trabajo
Start with the integration tests for messages and exceptions and inspect how IntegrationTester handles quitting after capture. Check how app-runner uses sentry-api-client to fetch event attachments, since the issue says it lacks a function for reading them. Done when tests wait for screenshots where applicable and assert that screenshot.jpg is nonempty, skipping batch mode and WebGL.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The integration tests don't check screenshots. They fetch the event the test app printed and check its fields, but not its screenshot.jpg, so a broken or empty screenshot goes unnoticed. This let the Metal readback error fixed in #2917 pass CI: the iOS simulator and macOS runs logged it on every capture.
The screenshots are empty in CI on every platform anyway. The test app calls Application.Quit in the frame it captures in, so the end-of-frame screenshot never gets taken and the SDK sends a 0-byte screenshot.jpg ("Screenshot was not taken in time").
What we could do:
- Let
IntegrationTesterwait for the screenshot before quitting in the message and exception tests. - Assert that the event's
screenshot.jpgisn't empty, skipped where no screenshot is taken (batch mode, WebGL). Reading attachments needs a function inapp-runner'ssentry-api-client, which has none.
- Lenguaje dominante
- C#
- Estrellas
- 295
- Forks
- 63
- Merge medio
- 4 d 10 h
- PR fusionados (30 d)
- 29
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
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 getsentry/sentry-unity
-
Drop `Release` and `Environment` from the editor config windowQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. AbiertoBreaking Change Editor Good First Issue Improvement Unity
Dificultad 2/5 1-3 horas Aptitud para principiantes 58/100
getsentry/sentry-unity#2914 · 1 comentario ·
Los mantenedores suelen responder en 5 días
-
Make CI stable and fasterAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
getsentry/sentry-unity#2913 · 1 comentario ·
Los mantenedores suelen responder en 5 días
-
Unity
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
getsentry/sentry-unity#2873 · 2 comentarios ·
Los mantenedores suelen responder en 5 días
-
Feature Unity
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
getsentry/sentry-unity#2796 · 4 comentarios ·
Los mantenedores suelen responder en 5 días
-
Breaking Change Good First Issue Logs Task Unity
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
getsentry/sentry-unity#2659 · 2 comentarios ·
Los mantenedores suelen responder en 5 días
Todos los issues de getsentry/sentry-unity
Issues similares
-
VB -> C#
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
icsharpcode/CodeConverter#1301 ·
Los mantenedores suelen responder en 3 días
-
PettingZoo wrappers send only the first action component to every action dimensionPosiblemente ocupada @RaphaelFakhri la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
Unity-Technologies/ml-agents#6341 ·
Los mantenedores suelen responder en 1 día
-
Area: World-Readiness
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
microsoft/calculator#2522 ·
-
needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Sonarr/Sonarr#9274 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
mailto support for hyperlinkAbiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100