Code cleaning : Remove campaignId (except LunaticJsonRawData)#437
Code cleaning : Remove campaignId (except LunaticJsonRawData)#437alexisszmundy wants to merge 92 commits intomainfrom
Conversation
Also deprecated DataType
alexisszmundy
requested review from
Hajarel-moukh,
alicela and
loichenninger
and removed request for
loichenninger
nsenave
left a comment
nsenave
left a comment
There was a problem hiding this comment.
relecture en diagonale surtout sur la partie main, looks good to me
| import java.time.LocalDateTime; | ||
| import java.util.Map; | ||
|
|
||
| @Builder |
| } catch (IllegalArgumentException iae){ | ||
| log.warn("'{}' is not a valid questionnaire state according to filiere model", questionnaireStateString); | ||
| } | ||
| } catch (NullPointerException ignored){} //WARN already done in getStringFieldInPayload |
There was a problem hiding this comment.
ici certes ça va être refacto avec les exception handler, mais la bonne pratique depuis java 21 (ou 25 je sais plus) c'est d'utiliser le unnamed pattern + commentaire à mettre dans le bloc de code pour pas générer une issue sonar
catch (NullPointerException _) {
// WARN already done in getStringFieldInPayload
}There was a problem hiding this comment.
j'ai pas été voir mais à vérifier que le maven failsafe plugin est bien configuré pour run ces tests **IT.java
| "logging.level.=DEBUG" | ||
| }) | ||
| @EnableAutoConfiguration(exclude = {MongoAutoConfiguration.class, MongoDataAutoConfiguration.class}) | ||
| public abstract class IntegrationTestAbstract { |
There was a problem hiding this comment.
un peu mauvais signe d'avoir besoin d'une classe abstraite (donc de l'héritage j'imagine) côté tests
ça veut sans doute dire qu'on a des classes de service qui font trop de chose et qui tirent ont besoin de trop de choses
Juste une remarque au passage mais en l'état ok
There was a problem hiding this comment.
c'est surtout pour éviter la duplication de code entre les tests d'intégration
| </executions> | ||
| <configuration> | ||
| <includes> | ||
| <include>**/*IT.java</include> |
There was a problem hiding this comment.
cf. mon commentaire plus haut : looks good 😄
|
3 participants
Uh oh!
There was an error while loading. Please reload this page.