[element-query] Update css-element-queries dependency

Atiomi commented 4 years ago

Description

Update element-queries dependency

Types de changements

[ ] Correction de bug (sans breaking change)
[ ] Amélioration (ajout par example une nouvelle propriété, évènement, slot ou méthode à un composant existant sans breaking change)
[ ] Nouvelle fonctionalité (nouveau composant, directive, filtre ou service)
[ ] Breaking change (modification à une fonctionnalités existante qui nécessite une migration remplir la section release note)
[ ] Refactoring/ménage (sans breaking change)
[ ] Documentation/storybook (changement à la documentation ou aux storybooks qui n'affecte aucun package)
[x] Autre

Comment cela peut-il être testé?

[ ] Test unitaire (un nouveau test unitaire à été fait)
[ ] Storybook
[ ] Test manuel / Sandboxes
[ ] Autre

Inclure cette section dans les release notes

Liens internes

https://jira.dti.ulaval.ca/browse/ENA2-11026

chuckmah commented 4 years ago

@Atiomi svp regarder le build

Atiomi commented 4 years ago

@chuckmah L'update de la lib brise les tests jest qui appelle la mixin ElementQueries. Malheureusement je n'y comprends pas grand chose, j'imagine qu'il faudrait mock la mixin d'une façon ou une autre...

vidal7 commented 4 years ago

Salut @chuckmah et @Atiomi, j'ai mocké partiellement ElementQueries. Les tests fonctionnent à nouveau mais je trippe pas beaucoup sur ma modification. À moins que @chuckmah ai une meilleure idée, on va laisser ça comme ça.

chuckmah commented 4 years ago

Salut @chuckmah et @Atiomi, j'ai mocké partiellement ElementQueries. Les tests fonctionnent à nouveau mais je trippe pas beaucoup sur ma modification. À moins que @chuckmah ai une meilleure idée, on va laisser ça comme ça.

Ca me va! Mais au lieu de repeter le code dans chaque test est-ce qu'on pourrait déplacer le mocking vers le fichier /tests/polyfills.js ou sinon creeer un nouveau fichier qui est appeler dans le jest config?

vidal7 commented 4 years ago

Hum, on ne veut pas affecter tous les tests unitaires je crois. Je pourrais essayer un import avec effet de bord. Si ça marche, on fera ça de même sinon, je pense qu'il vaut mieux laisser ainsi.

vidal7 commented 4 years ago

OK, l'import avec effet de bord fonctionne! Le mock est écrit dans un seul fichier que l'on import au besoin.

ulaval / modul