coder / coder/internal

Remove DisableForeignKeysAndTriggers from our test suite in coder/coder

Offen
#856 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
tech-debt
Vorherrschende Sprache
Keine Sprachdaten
Sterne
3
Forks
0
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

`DisableForeignKeysAndTriggers` was added to punt some of the work getting our tests running with Postgres instead of the in-memory database (which didn't support foreign key constraints and triggers).

Some of the uses, like in `dbauthz` should get replaced by the mock database, others should get fixed up to satisfay Postgres.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, jede Verwendung von DisableForeignKeysAndTriggers in der Testsuite zu lokalisieren, mit besonderem Augenmerk auf dbauthz, und überprüfe, wie die Mock-Datenbank verwendet wird. Die Aufgabe ist erledigt, wenn der Helper entfernt, die entsprechenden Verwendungen durch die Mock-Datenbank ersetzt und die verbleibenden Tests so angepasst wurden, dass sie die Postgres-Constraints und -Trigger erfüllen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
postgresql
Bereich
databases, testing
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

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