Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Test invalid positive-integer environment settings

オープン 初心者向け
  • 何か問題がありますか？
#56 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
72/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
fastapi, python
領域
api, backend

調査の方向性

リポジトリで MAX_FILE_SIZE を検索し、まずその設定のパース処理と近くにあるテストを読んでください。有効な値、整数でない値、ゼロ、負の値を対象とした集中的なカバレッジを追加してください。無効な設定で明確な設定エラーが発生し、有効な設定が引き続き機能すれば完了です。

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

説明

area:auth contributor-roadmap difficulty:intermediate help wanted status:ready

AUT-02 · Authentication and configuration

Difficulty: intermediate · Status: ready

This is a focused contribution against current main. No unmerged feature PR is required.

Why this is useful

positive_int validates values but parsing errors are not covered

Start here

Source audit: 9d8ac3e01df1. Read these files on current main too, because paths may change after the audit:

Scope and intended output

Add the focused test file tests/roadmap/auth/test_numeric_settings.py. Change runtime code only if the acceptance criteria explicitly request a behavior fix; otherwise report newly discovered behavior separately.

Acceptance criteria
  • Cover zero, negative, noninteger, and valid values for all numeric settings
  • Run import-sensitive cases in isolated subprocesses
  • Assert no model or provider work occurs
Verification
uv run pytest tests/roadmap/auth/test_numeric_settings.py -q
uv run pylint app tests

Create the named focused test file before running its command. Tests must be deterministic, mock external services, and avoid paid calls. Run bash scripts/check.sh before submitting a Python change.

Dependencies

None. Keep the PR based on current upstream main.

Out of scope
  • Unrelated refactors, broad formatting, endpoint renames, or extra dependencies without agreement.
  • Live cloud provisioning, paid model calls, real private messages, or credentials in fixtures/logs.
  • Implementing neighboring roadmap issues in the same PR.
Getting help and submitting

Comment with your intended approach and ask when expected behavior is unclear. Check for an existing PR before starting; there is no deadline for a volunteer contribution. A useful reproduction or partial finding is welcome.

Use the issue's criteria as the review checklist. Include the commands actually run and any limitations. Link the issue; use Fixes only when its entire agreed scope is complete. If CONTRIBUTING.md is not yet on main, the contributor-roadmap PR provides the onboarding guide.

Original issue before the source audit (historical context)

Problem

MAX_FILE_SIZE is parsed directly with int(), so invalid, zero, or negative values produce confusing startup or runtime behavior.

Acceptance criteria

  • Reject non-integer values with a clear configuration error.
  • Reject zero and negative values.
  • Add focused tests for valid and invalid configuration values.
主要言語
Python
スター
3
フォーク
10
平均マージ
8時間 46分
マージ済み PR（30日）
16

環境構築

このプロジェクトの環境構築ファイルはまだ確認していません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

ashrafee-dev/scamshield-api のほかの issue

ashrafee-dev/scamshield-api の issue をすべて見る

似ている issue

Python の issue をもっと見る

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

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