search

popcorn-nantes / popcorn-machine

🍿La machine à Popcorn permet est un package NPM qui permet de créer un annuaire de dev freelances "Popcorn", tel que celui de Popcorn Nantes.
https://www.npmjs.com/package/popcorn-machine
MIT License
10 stars 8 forks source link

Ajout de test cypress pour vérifier les balises metas pour partager sur réseaux sociaux #35

Closed yann-yinn closed 5 years ago

yann-yinn commented 5 years ago

Pfiou, plus de difficultés que prévues sur la récupération de la config popcorn, mais la fonctionnalité est clef. Pas terminé, j'essaie de m'y remettre quand je peux.

fabienjuif commented 5 years ago

@yann-yinn je pense que je relirais quand les tests passeront sur travis

yann-yinn commented 5 years ago

Oui je dois fixer ça , au temps pour moi.

yann-yinn commented 5 years ago

Ok donc: 1) Si je fais un require sur le fichier .env : il faut enlever le spread operator de popcorn.config.js sinon cypress plante, à cause de sa version de node. Par contre on bénéficie de pouvoir passer côté cypress les variables d'env et config de manière identique à la réalité; donc ça vaut le coup 2) Si je fais en mock en dur, on n'a pas le souci de spread operator (mais on ne prend pas en compte le fichier .env ni le vrai fichier de config de popcorn-machine)

Je vote pour 1) de mon coté, d'autant que le spread operator n'est pas indispensable, c'est plus clair de déclarer explicitement les variables en provenance du popcorn de la localité

Maintenant j'essaie de comprendre pourquoi les tests fonctionnent chez moi et pas sur Travis -_- (edit: c'était le fichier .env qui était ignoré par défaut sur le dossier de projet test )

yann-yinn commented 5 years ago

Les tests passent ! \o/

fabienjuif commented 5 years ago

Bien joué :)

Je me suis pété le bras a vélo, alors je vais etre moins reactif je pense pendant au moins 2 semaines !