test : add unit tests for Database.connection property error path
まだ誰も着手していません。
- #2277 @tmdeveloper007 による — マージされずにクローズ
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 35/100
調査の方向性
Read backend/secuscan/database.py and inspect the Database connection property and initialization state. Then review nearby unit test conventions before creating testing/backend/unit/test_database.py; use --noconftest and an in-memory SQLite path. Done means the tests cover the disconnected error and message, plus the specified state before and after connect().
索引モデルが issue の本文から書いたものです。
説明
Summary of What Needs to be Done:
Add unit tests for the Database.connection property error path in backend/secuscan/database.py. When the database is not connected, accessing the .connection property should raise a RuntimeError with a clear message.
Changes that Need to be Made:
Create testing/backend/unit/test_database.py (standalone unit tests) covering:
- Database(db_path=":memory:").connection raises RuntimeError when not connected
- The RuntimeError message is descriptive and actionable
- Database._connection is None before connect()
- Database._connection is set after connect()
- Database._in_transaction is False initially
Note: Use an in-memory SQLite database (":memory:") path to avoid file system dependencies. The test should NOT require a real database file.
Import from backend.secuscan.database import Database directly. Use --noconftest for local runs.
Impact that it would Provide:
- Reliability: Documents the expected error behavior when database is used before connection
- Testing: Covers an entirely untested module (database.py)
- Maintainability: Provides a clear contract for Database initialization state
Note: This task is being handled by tmdeveloper007 — please assign to that account when picking it up.
- 主要言語
- Python
- スター
- 36
- フォーク
- 234
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
utksh1/SecuScan のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
area:frontend good first issue level:intermediate priority:low type:testing
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
-
area:frontend good first issue level:intermediate priority:low type:testing
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
area:frontend good first issue level:beginner priority:low type:testing
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
area:frontend good first issue level:intermediate priority:low type:testing
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
utksh1/SecuScan の issue をすべて見る
似ている issue
-
[BUG] HTTP lifecycle dispatch treats a NO_PROXY-only environment as corporate egress and skips the resolve-and-pinned session対応中かも @bodapatisaikrishna が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
external
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
langchain-ai/docs#7652 ·
メンテナーはふだん 1 日以内に返信
-
[oblt-aw][autodoc] Document new e2e-secret-leak-artifacts workflow and update e2e-all leaf listオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
elastic/oblt-aw#2513 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug p2
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 1/5 1時間未満 初心者へのやさしさ 75/100
NCAR/CLM-Tutorial#113 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信