Refactor and improve `GIDEMMSupport` parameters and tests
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- objective-c
- Área
- authentication, mobile-dev, testing
Línea de trabajo
Comienza leyendo la implementación de GIDEMMSupport y sus pruebas unitarias existentes, especialmente parametersWithParameters y las pruebas que hacen swizzle de UIDevice. Comprueba cómo la API pública expone este comportamiento e identifica qué pruebas dependen del estado real del dispositivo. Se considera terminado cuando los parámetros con claves de arrays anidados se descartan de forma segura, el mocking de UIDevice es coherente y hermético, y la cobertura ejercita la API pública cuando corresponde.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
This issue tracks follow-up improvements identified during the review of PR #608. We need to address the following technical debt and testing improvements:
- Handle nested data structures: Ensure that the
parametersWithParametersfunction safely drops key parameters that contain values like a nested array. - Standardize
UIDevicemocking: Fix the tests that are swizzlingUIDeviceto "Not Do That," or ensure all tests are doing it the same way. Relying on the realUIDevicewill make tests non-hermetic. - Uplevel tests to the public API: Consider upleveling the new
GIDEMMSupportunit tests to test against the public API instead of testing internal classes in isolation.
- Lenguaje dominante
- Objective-C
- Estrellas
- 751
- Forks
- 282
- Merge medio
- 4 d 20 h
- PR fusionados (30 d)
- 11
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 google/GoogleSignIn-iOS
-
enhancement triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
google/GoogleSignIn-iOS#621 ·
-
enhancement triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
google/GoogleSignIn-iOS#620 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
google/GoogleSignIn-iOS#440 · 4 comentarios · 2 reacciones ·
-
Remove Cocoapods Support Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
google/GoogleSignIn-iOS#607 ·
-
Add App Check support for macOS Abiertoenhancement triage
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
google/GoogleSignIn-iOS#602 ·
Todos los issues de google/GoogleSignIn-iOS
Issues similares
-
Suzhou subway logo AbiertoIcons Metro Public Transport
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
organicmaps/organicmaps#13617 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
objective-see/LuLu#925 ·
-
macOS下查找brew::ncurse失败 Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
noah-nuebling/mac-mouse-fix#2054 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
obsproject/obs-studio#13918 ·