Skip to content

feat: new rule AAR054 classification valid values - #19

Merged
SebastianDT1 merged 5 commits into
developfrom
feat/2448/aar054-classification-valid-values
Aug 12, 2026
Merged

feat: new rule AAR054 classification valid values#19
SebastianDT1 merged 5 commits into
developfrom
feat/2448/aar054-classification-valid-values

Conversation

@mels-h

@mels-h mels-h commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

@SebastianDT1

Copy link
Copy Markdown
Contributor

AAR054 – Casos de prueba faltantes

  1. v3/v31 sin cdc ni sys válidos — solo se prueba cmd como valor válido
    en estas versiones (v2 sí cubre los 3).
  2. v31 con cobertura pobre — le faltan tests de no-address,
    null-address y address numérico/booleano sin comillas (v3 sí los
    tiene).
  3. Mayúsculas en el segmento (ej. Cmd en vez de cmd) — sin test.
  4. Parámetro validValues custom vía RuleProperty — sin test, ni
    positivo ni negativo.
  5. Documento con canales mezclados (válido + inválido en el mismo
    archivo) — sin test.
  6. Segmento con espacios o vacío (ej. beca..alumnos) — sin test.
  7. Raíz sin campo channels — el guard existe en el código pero no hay
    test que lo ejercite.

@sonarqubecloud

Copy link
Copy Markdown

@SebastianDT1
SebastianDT1 merged commit 64ed400 into develop Aug 12, 2026
1 check passed
@SebastianDT1
SebastianDT1 deleted the feat/2448/aar054-classification-valid-values branch August 12, 2026 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants