coder / coder/internal

flake: e2e externalAuth.spec.ts

Abierto
#356 3 comentarios 0 reacciones 0 asignados Ver en GitHub
flake
Lenguaje dominante
Sin datos de lenguaje
Estrellas
3
Forks
0
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

# Problem

See https://github.com/coder/internal/issues/212 for context.

```
==> Errors
/home/runner/work/coder/coder/site/e2e/tests/externalAuth.spec.ts:15:
13 | import { beforeCoderTest, resetExternalAuthKey } from "../hooks";
14 |
> 15 | test.beforeAll(async ({ baseURL }) => {
| ^
16 | const srv = await createServer(gitAuth.webPort);
17 |
18 | // The GitHub validate endpoint returns the currently authenticated user!
"beforeAll" hook timeout of 30000ms exceeded.
```

It looks like the hardcoded port (50516) can be unavailable for more than 30s.

# Proposed Solution

1) Simply bump the timeout (https://github.com/coder/coder/pull/16528)

2) Refactor the e2e tests to dynamically create a port.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Empieza en site/e2e/tests/externalAuth.spec.ts, en el hook beforeAll de la línea 15, y revisa después la llamada a createServer, el puerto codificado 50516 y el contexto vinculado. Reproduce el timeout del hook y compara el cambio de timeout propuesto con la creación dinámica del puerto; se considera terminado cuando la configuración e2e de external-auth ya no falla intermitentemente cuando ese puerto no está disponible.

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

Evaluación

Stack tecnológico
typescript
Área
authentication, testing
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.