coder / coder/internal

flake: e2e externalAuth.spec.ts

Offen
#356 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
flake
Vorherrschende Sprache
Keine Sprachdaten
Sterne
3
Forks
0
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

# 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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne in site/e2e/tests/externalAuth.spec.ts beim beforeAll-Hook in Zeile 15 und überprüfe anschließend den createServer-Aufruf, den fest codierten Port 50516 und den verknüpften Kontext. Reproduziere das Hook-Timeout und vergleiche die vorgeschlagene Timeout-Änderung mit der dynamischen Porterstellung; als erledigt gilt, wenn das external-auth e2e-Setup nicht mehr flakey ist, wenn dieser Port nicht verfügbar ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
authentication, testing
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.