Skip to content

Añadir soporte para Punt Frontera en el cálculo del CINI de CentroTransformador#44

Merged
mvilaboa merged 3 commits into
masterfrom
90646_consider_punt_frontera_flag
Jun 17, 2026
Merged

Añadir soporte para Punt Frontera en el cálculo del CINI de CentroTransformador#44
mvilaboa merged 3 commits into
masterfrom
90646_consider_punt_frontera_flag

Conversation

@gpozo-gisce

Copy link
Copy Markdown
Contributor

Objetivos

  • Añadir soporte para Punt Frontera en el cálculo del CINI de CentroTransformador. Cuando un CT es Punt Frontera, la 5ª posición del CINI debe ser '5', independientemente del tipo de CT (categoria_cne).

Comportamiento antiguo

  • La clase CentroTransformador no tenía atributo punt_frontera. La 5ª posición (índice [5]) se fijaba únicamente según el tipo: '1' (Intemperie), '2' (Caseta), '3' (Local), '4' (Subterráneo), '9' (Móvil). No existía el valor '5'.

Comportamiento nuevo

  • Nuevo atributo self.punt_frontera = False en CentroTransformador.init().
  • En la propiedad cini, cuando self.punt_frontera es True, la posición [5] se fija a '5', ignorando el valor de self.tipo.
    Tests
  • Nuevo test "si te flag Punt frontera, ignora tipo, valor es 5" que verifica cini[5] == '5' cuando punt_frontera = True.
  • Tests reorganizados para evitar contaminación de estado compartido entre contexts.

@gpozo-gisce gpozo-gisce self-assigned this Jun 15, 2026
@gpozo-gisce gpozo-gisce changed the title Añadir soporte para _Punt Frontera_ en el cálculo del CINI de CentroTransformador Añadir soporte para Punt Frontera en el cálculo del CINI de CentroTransformador Jun 15, 2026
@gpozo-gisce gpozo-gisce requested review from Copilot and mvilaboa June 15, 2026 14:45

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Este PR añade el flag Punt Frontera al modelo CentroTransformador y ajusta el cálculo del CINI para que la posición 5 valga '5' cuando el CT está marcado como Punt Frontera, independientemente del tipo de CT.

Changes:

  • Se introduce self.punt_frontera = False en CentroTransformador.__init__.
  • En CentroTransformador.cini, si punt_frontera es True, se fuerza c.positions[5] = '5'.
  • Se añade un test que verifica cini[5] == '5' cuando punt_frontera está activo.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
spec/cts_spec.py Añade cobertura de test para la nueva regla de Punt Frontera en la posición 5 del CINI.
cini/models.py Incorpora el nuevo atributo punt_frontera y actualiza la lógica de cálculo del CINI del CT.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread spec/cts_spec.py Outdated
Comment thread cini/models.py Outdated
@gpozo-gisce gpozo-gisce marked this pull request as ready for review June 15, 2026 15:07
@mvilaboa mvilaboa merged commit 307ee5f into master Jun 17, 2026
2 checks passed
@mvilaboa mvilaboa deleted the 90646_consider_punt_frontera_flag branch June 17, 2026 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants