test : add unit tests for make_scan_rate_limiter factory and RateLimitExceeded

Aperta
  • Qualcosa non va?
#2,271 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

  • #2276 di @tmdeveloper007 — chiusa senza merge

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
25/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
fastapi, python
Ambito
backend, testing

Direzione di ricerca

Read backend/secuscan/rate_limiter.py and the existing tests under testing/backend/unit to follow the project’s test conventions. Create testing/backend/unit/test_rate_limiter.py and import the helpers directly as specified; use --noconftest for local runs. Cover the factory behavior and RateLimitExceeded cases listed in the issue, then confirm the standalone unit tests pass. The issue says it is being handled by tmdeveloper007 and links a closed, unmerged pull request, so check the current work before starting.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Summary of What Needs to be Done:
Add unit tests for the make_scan_rate_limiter factory function and RateLimitExceeded exception class in backend/secuscan/rate_limiter.py. These are standalone helpers that do not require a running Redis instance or FastAPI app state.

Changes that Need to be Made:
Create testing/backend/unit/test_rate_limiter.py (standalone unit tests) covering:

  1. make_scan_rate_limiter with redis_client=None returns a ScanRateLimiter instance with correct attributes:

    • _rate_limit matches the input
    • _rate_window matches the input
    • _burst_limit matches the input
    • _burst_window matches the input
    • _redis_failed is False
    • _fallback_history is empty dict

  2. make_scan_rate_limiter with valid redis client returns ScanRateLimiter with redis set

  3. RateLimitExceeded exception:

    • Is a subclass of HTTPException
    • status_code is 429
    • Has a Retry-After header
    • Has a descriptive detail message
    • Can be instantiated with custom detail

Import from backend.secuscan.rate_limiter directly. Use --noconftest for local runs.

Impact that it would Provide:

  • Reliability: Ensures the rate limiter factory and exception are correctly wired
  • Testing: Covers an entirely untested module
  • Maintainability: Documents the expected factory behavior

Note: This task is being handled by tmdeveloper007 — please assign to that account when picking it up.

Lingua principale
Python
Stelle
36
Fork
234
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di utksh1/SecuScan

Tutte le issue di utksh1/SecuScan

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.