Skip to content
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

[FEATURE] Utiliser le package ember-testing-library dans Pix App (PIX-4983). #4460

Merged

Conversation

AndreiaPena
Copy link
Member

🦄 Problème

L'application Pix App était la seule à ne pas utiliser le package ember-testing-library.
Une des raisons à cela était un soucis de wrapper sur le visit de /ember/test-helpers (voir PR #4114 ) qui a été corrigé.

🤖 Solution

Utiliser le package ember-testing-library dans les tests

🌈 Remarques

On a juste utilisé TL dans un test d'acceptance et d'intégration pour vérifier que tout fonctionne.

💯 Pour tester

Les tests passent ✅

@pix-service
Copy link
Contributor

@Libouk
Copy link
Member

Libouk commented May 25, 2022

Le package tant attendu ! Bravo !

@Libouk
Copy link
Member

Libouk commented Jun 2, 2022

Salut, on peut merge ?

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-4983-user-ember-testing-library-in-pix-app branch from cde3cda to 7a22c6c Compare June 2, 2022 15:36
@pix-service-auto-merge pix-service-auto-merge merged commit 61ffe7e into dev Jun 2, 2022
@pix-service-auto-merge pix-service-auto-merge deleted the pix-4983-user-ember-testing-library-in-pix-app branch June 2, 2022 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 🚀 Ready to Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants