-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat : Version basique de l'assistant PAM micro-fiscal #3354
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for nostalgic-mahavira-52b682 canceled.
|
🚀 La branche est déployée !
|
@@ -0,0 +1,180 @@ | |||
déclaration revenus PAMC: | |||
experimental: oui | |||
valeur: non |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pourquoi la mettre à non
par défaut ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pour la désactiver par défaut et ne l'activer que dans l'assistant en définissant une situation de base
'déclaration revenus PAMC': oui
}) | ||
|
||
it("ne devrait pas montrer les champs avant qu'une profession soit sélectionnée", function () { | ||
cy.get(`#${idPrefix}_recettes_brutes_totales-title`).should('not.exist') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Idéalement, les tests d’UI se basent sur ce qui est affiché, sémantiquement, plutôt que sur des ids (mais c’est parfois plus facile à dire)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Certains titres sont communs à la partie question et la partie résultats. Je vais donc laisser ainsi pour ceux-là. Pour les autres je vais utiliser le texte.
… régime micro-fiscal
e4fb26d
to
956bf02
Compare
#3338