Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Add a small reusable valid assessment factory

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#291 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
78/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
testing-qa, tooling

Hướng nghiên cứu

Đọc tests/conftest.py và pyproject.toml trên main hiện tại, sau đó kiểm tra scripts/check.sh và .github/workflows/checks.yml để nắm các yêu cầu về test và lint. Thêm tests/roadmap/tooling/test_assessment_factory.py bao quát các response tổng hợp đầy đủ và hợp lệ, các override tường minh, việc xác thực response và tính độc lập với các thành phần nội bộ của provider. Xác minh bằng lệnh pytest tập trung, pylint và bash scripts/check.sh.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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

TST-02 · Test infrastructure

Difficulty: intermediate · Status: ready

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

Why this is useful

Tests duplicate response dictionaries with inconsistent values

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/tooling/test_assessment_factory.py. Change runtime code only if the acceptance criteria explicitly request a behavior fix; otherwise report newly discovered behavior separately.

Acceptance criteria
  • Return complete valid synthetic responses with explicit overrides
  • Avoid a MagicMock that bypasses response validation
  • Keep it independent of provider internals
Verification
uv run pytest tests/roadmap/tooling/test_assessment_factory.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.

Ngôn ngữ chính
Python
Star
3
Fork
10
Merge trung bình
8 giờ 11 phút
Pull request đã merge (30 ngày)
18

Chuẩn bị môi trường

Chúng tôi chưa kiểm tra các tệp thiết lập môi trường của dự án này. Hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của ashrafee-dev/scamshield-api

Tất cả issue của ashrafee-dev/scamshield-api

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.