matrix-org / matrix-org/complement

Password change tests depend on oneshot UIA

Abierto
#897 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
Go
Estrellas
99
Forks
72
Merge medio
4 d 1 h
PR fusionados (30 d)
8

Descripción

Related: #893

https://github.com/matrix-org/complement/blob/1a2baac5dbf45ead46409793fe625b3c253eb4f6/tests/csapi/account_change_password_test.go#L110-L122

The password change helper requires being capable of skipping UIA, which is not spec compliant and causes flaky failures in all tests that use it.

The test also depends on being able to do oneshot UIA as it does not use the helper: https://github.com/matrix-org/complement/blob/1a2baac5dbf45ead46409793fe625b3c253eb4f6/tests/csapi/account_change_password_test.go#L82-L95

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Empieza leyendo tests/csapi/account_change_password_test.go en las líneas 82-95 y 110-122; después, revisa el issue #893 relacionado y el helper de cambio de contraseña al que hacen referencia. Confirma cómo omiten o ejecutan actualmente UIA los tests, y haz que la cobertura del cambio de contraseña use un manejo de UIA conforme con la especificación y sin dependencias de oneshot. La tarea estará terminada cuando los tests afectados ya no dependan de esos comportamientos ni fallen intermitentemente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
go
Área
testing-qa
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Tranquilo
Claridad
Bastante claro
Aptitud para principiantes
62/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.