test : add unit tests for _check_field helper in validation
まだ誰も着手していません。
- #2342 @tmdeveloper007 による — マージされずにクローズ
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 35/100
調査の方向性
Read backend/secuscan/validation.py and follow _check_field's string, list, and other-value paths. Add testing/backend/unit/test_validation_check_field.py with the listed boundary, pass-through, and error-message cases, using settings for limits. Run the new unit tests and confirm they cover each case; issue #2342 has a closed, unmerged pull request, so check its changes before starting.
索引モデルが issue の本文から書いたものです。
説明
Summary of What Needs to be Done:
Add unit tests for the _check_field helper function in backend/secuscan/validation.py. This private function validates individual input field values (strings and lists) against length limits. It is the core of the validate_task_start_payload function and has distinct logic paths for string vs list values.
Changes that Need to be Made:
- Add a new file
testing/backend/unit/test_validation_check_field.py - Import
_check_fieldandsettingsfrombackend.secuscan.validation - For string values:
- Test string at exactly max length passes
- Test string exceeding max length returns 400 with appropriate message
- Test string well under limit passes
- For list values:
- Test list at exactly max array length passes
- Test list exceeding max array length returns 400
- Test list with items exceeding max field length returns 400
- Test list with all items within limits passes
- For non-string/non-list values:
- Test returns (True, 0, '') for unexpected types (pass-through)
- Test that error messages do not echo back field values
- Test with edge case: empty string ("") passes
- Test with edge case: empty list ([]) passes
Impact that it would Provide:
- Provides granular coverage for the field-level validation logic
- Ensures string length and array length limits are enforced correctly
- Guards against regression as settings limits are adjusted
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 日以内に返信