test : add unit tests for handle_caret handle_tilde and handle_wildcard helpers

Abierto
  • ¿Algo no está bien?
#2,274 1 comentario 0 reacciones 1 asignado Ver en GitHub

@tmdeveloper007 ya está trabajando en esto.

Desde el 3/8/2026.

  • #2279 de @tmdeveloper007 — cerrado sin fusionar
  • #2283 de @singhanurag0317-bit — cerrado sin fusionar

Evaluación

Este issue todavía no se ha evaluado.

Descripción

Summary of What Needs to be Done:
Add direct unit tests for the handle_caret, handle_tilde, and handle_wildcard helper functions in backend/secuscan/remediation.py. These are internal helpers used by semver_to_pep440. While semver_to_pep440 is tested in test_remediation_safety.py, the individual helpers are not tested directly, leaving edge cases uncovered.

Changes that Need to be Made:
Add tests to testing/backend/unit/test_remediation_safety.py (or create testing/backend/unit/test_remediation_helpers.py) covering:

  1. handle_caret (ver_str: str) -> List[str]:

    • handle_caret("1.2.3") == [">=1.2.3,<2.0.0"]
    • handle_caret("0.2.3") == [">=0.2.3,<0.3.0"]
    • handle_caret("0.0.3") == [">=0.0.3,<0.0.4"]
    • handle_caret("0.1.0") == [">=0.1.0,<0.2.0"]
    • handle_caret("0.0.1") == [">=0.0.1,<0.0.2"]

  2. handle_tilde (ver_str: str) -> List[str]:

    • handle_tilde("1.2.3") == [">=1.2.3,<1.3.0"]
    • handle_tilde("1.2") == [">=1.2.0,<1.3.0"]
    • handle_tilde("1") == [">=1.0.0,<2.0.0"]

  3. handle_wildcard (part: str) -> List[str]:

    • handle_wildcard("1") == [">=1.0.0,<2.0.0"]
    • handle_wildcard("1.2") == [">=1.2.0,<1.3.0"]
    • handle_wildcard("1.2.3") == [">=1.2.3,<1.3.0"]
    • handle_wildcard("x") raises ValueError or returns empty list
    • handle_wildcard("*") raises ValueError or returns empty list

Import from backend.secuscan.remediation directly. Note: These functions are prefixed with underscore in the module (e.g., _handle_caret). Use --noconftest for local runs.

Impact that it would Provide:

  • Reliability: Tests the individual components of semver_to_pep440 in isolation
  • Testing: Fills gaps left by existing semver_to_pep440 tests which only test the top-level function
  • Maintainability: Makes future changes to these helpers safer with direct coverage

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

Lenguaje dominante
Python
Estrellas
36
Forks
234
Sin PR fusionados en 30 d

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de utksh1/SecuScan

Todos los issues de utksh1/SecuScan

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.