Feature#504

[PabloMartinFluvia]

TO DO: integration test for GET Raisc by Scope id.
Added 2 diagrams: what is new + posibles TODO involved to upgrade the design.
Feature#504.pdf
Feature#504_potential_TODOs_involved.pdf

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

81.9% Coverage
0.0% Duplication

[jonatanvicente]

Parece que se ha hecho mucho trabajo, con resultado confuso.
En primer lugar, se ha intentado hacer una segmentación o implementación de arquitectura que no tiene cabida a este nivel (la implementación; la arquitectura fue definida hace tiempo) ni tampoco en este escenario (en un microservicio con abstracciones claras provistas por el framework).
Este microservicio únicamente se conecta a una fuente de datos externa, y la capa de abstracción necesaria la provee el patrón Repository de Spring, aislando completamente la dependencia.

La task definida tan sólo solicita:

• Conectar con la fuente de datos externa (empleando el Proxy incluido)
• Filtrar por uno o varios parámetros
• Dar formato a la salida

Hay que revisar en el código lo siguiente:

• GencatDataAdapter es un ejemplo del antipatrón 'Objeto Todopoderoso' o 'God Object': él mismo es un repository para acceder a datos, incluye llamadas a un proxy, extiende a JsonMapper para hacer mapeos de objetos JSON e implementa también DataSourceAdapter (un servicio). La funcionalidad que abarca es oscura, el tipo de objeto es inclasificable y es muy difícil de testear bien, ya que abarca demasiada casuística.
• GencatJsonMapper utiliza Log4j de librería lombok. No utilizamos para trazar librerías cuya finalidad es otra.
• La sobreescritura del método equals y hashcode (de utilidad discutible) son completamente ilegibles. Un condicional de más de 3 o 4 elementos (según el escenario) es una mala praxis.
• Se añaden clases y packages 'Utils' con responsabilidad difusa. ¿Son Helpers? ¿Son Adaptadores? ¿Conversores? ¿Proxies?
• Algunos métodos de test incluyen JSON muy extensos 'hard coded' (no intuitivos), que serían fácilmente legibles si se externalizan (como está hecho en otras partes del proyecto).

Revisa esto, por favor ;)