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

Mise à jour de Selenium, geckodriver et OpenJDK #6258

Merged
merged 3 commits into from
Mar 25, 2022

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Mar 6, 2022

  • Mise à jour de Selenium, geckodriver et OpenJDK 11 à leur dernière version disponible
  • Mise à jour de la documentation concernant les tests avec Selenium
  • AdoptOpenJDK a rejoint la Fondation Eclipse et les binaires sont maintenant disponibles sous le nom de Temurin. J'ai donc modifié le script d'installation pour utiliser les nouveaux dépôts Github, les anciens ne semblent pas être maintenu.
  • OpenJDK 11 sera maintenu au minimum jusqu'en octobre 2024 donc on a encore un peu de temps avant de passer à OpenJDK 17 (d'autant plus que je suppose que notre ancienne version de ElasticSearch ne la supporte pas et je ne suis pas sûr que les paquets soit disponible sur notre version actuelle de Debian)
  • Ajout du paquet Apt xvfb à la liste des dépendances Ubuntu, nécessaire pour lancer make test-back-selenium.

QA :

  • Partir d'une installation neuve : sudo rm -rI zdsenv/ et make wipe-db puismake install-linux-full
  • Activer l'environnement : source zdsenv/bin/activate && make zmd-start
  • Lancer les tests : make test-back-selenium

@coveralls
Copy link

coveralls commented Mar 6, 2022

Coverage Status

Coverage remained the same at 87.32% when pulling 141fda0 on Situphen:selenium into 326a0dd on zestedesavoir:dev.

@Situphen Situphen added this to En développement in Suivi des PR via automation Mar 6, 2022
@Situphen Situphen marked this pull request as ready for review March 6, 2022 16:19
@Situphen Situphen moved this from En développement to En attente de QA in Suivi des PR Mar 6, 2022
@Situphen Situphen added the C-DevelopmentEnv Amélioration de l'environnement de dev label Mar 6, 2022
@philippemilink
Copy link
Member

Rapport de QA

Testé avec Docker, ça fonctionne. Par contre :

  • c'est moi ou les tests fronts ne sont pas lancés sur la CI ?
  • make install-linux-full n'installe pas geckodriver, il faut le faire manuellement, en suivant ce qui est installé dans la doc. Est-ce qu'on ne peut pas ajouter cette étape au Makefile ?

Les deux points peuvent se régler dans des PRs à part.

@philippemilink philippemilink moved this from En attente de QA to En attente de retours in Suivi des PR Mar 25, 2022
@Arnaud-D
Copy link
Contributor

On a un ticket pour automatiser l'installation de geckodriver : #6259.

@Situphen
Copy link
Member Author

c'est moi ou les tests fronts ne sont pas lancés sur la CI ?

Ils ne sont pas lancés. Je crois que le lancement de ces tests prend pas mal de temps donc ça pourrait ralentir la CI. Aussi, il faut avoir Firefox sur la CI et je ne sais pas si c'est faisable aisément. On peut y réfléchir mais ce n'est pas prioritaire à mes yeux.

Est-ce qu'on ne peut pas ajouter cette étape au Makefile ?

Il faudrait, j'ai juste eu la flemme de le faire dans cette PR !

On a un ticket pour automatiser l'installation de geckodriver

Merci de me rappeler l'existence de mon propre ticket, j'avais oublié... facepalm

Suivi des PR automation moved this from En attente de retours to Fusionnable après rebase Mar 25, 2022
@philippemilink philippemilink merged commit 8cd3124 into zestedesavoir:dev Mar 25, 2022
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Mar 25, 2022
@Situphen Situphen deleted the selenium branch March 25, 2022 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-DevelopmentEnv Amélioration de l'environnement de dev
Projects
Archived in project
Suivi des PR
  
Fusionnée
Development

Successfully merging this pull request may close these issues.

None yet

4 participants