coder / coder/internal

Remove DisableForeignKeysAndTriggers from our test suite in coder/coder

オープン
#856 コメント 0 件 リアクション 1 件 担当者 0 名 GitHub で見る
tech-debt
主要言語
言語のデータがありません
スター
3
フォーク
0
PR マージ指標
30日以内にマージされた PR はありません

説明

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

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

まず、テストスイート内の DisableForeignKeysAndTriggers の使用箇所をすべて特定し、特に dbauthz に注意を払いながら、モックデータベースがどのように使用されているかを確認します。helper を削除し、該当する使用箇所をモックデータベースに置き換え、残りのテストを Postgres の制約とトリガーを満たすように修正できれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
postgresql
領域
databases, testing
issue の種類
リファクタリング
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。