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.

Sign up for GitHub

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

Jump to bottom

[WIP] add selenium test integration #952

Closed
wants to merge 28 commits into from
Closed

[WIP] add selenium test integration #952

wants to merge 28 commits into from

Conversation

ptitloup
Copy link
Contributor

@ptitloup ptitloup commented Sep 20, 2023 •
edited
Loading

Before sending your pull request, make sure the following are done :

  • You have read our contribution guidelines.
  • Your PR targets the develop branch.
  • The title of your PR starts with [WIP] or [DONE].

@ptitloup
add selenium test in pod main app, add selenium, geckodriver and fire…
b7f0719 
…fox in docker file, create integration github worklow
@ptitloup ptitloup marked this pull request as draft September 20, 2023 14:35
@ptitloup ptitloup self-assigned this Sep 20, 2023
@ptitloup ptitloup added the enhancement New feature or request label Sep 20, 2023
Badatos
Badatos requested changes Sep 21, 2023
View reviewed changes
Copy link
Collaborator

@Badatos Badatos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci pour cette PR. Hate de voir ce que ca donne :)

Je me demande juste si on aurait pas pu mettre le "Run Tests" dans le workflow existant, histoire de pas faire l'install de Pod 2 fois

pod/main/integration_tests/selenium_pod_integration_tests.py
# pod.main.integration_tests.selenium_pod_integration_tests


class PodSeleniumTests(StaticLiveServerTestCase):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manque des pydocs un peu partout ^^

@ptitloup
Copy link
Contributor Author

ptitloup commented Sep 21, 2023 •
edited
Loading

Merci pour cette PR. Hate de voir ce que ca donne :)

Ca focntionne plutot bien, j'ai vu avec Laurine et Aymeric ce matin, Laurine va créer des sides selenium et les fournir à Aymeric

Je me demande juste si on aurait pas pu mettre le "Run Tests" dans le workflow existant, histoire de pas faire l'install de Pod 2 fois

Les sides sont créés au format json via selenium IDE. Aymeric va créé un script python qui sera lancé par les tests. Ce script va parcourir les sides json et lancera les tests des sides.
Vu le fonctionnement impaginé, j'ai préféré séparé les tests unitaires des tests fonctionnels. On prévoit un side par "type" d'application (video, meeting, authentification, gestion des fichiers etc.)

@Badatos
Copy link
Collaborator

Badatos commented Sep 21, 2023

Je me demande juste si on aurait pas pu mettre le "Run Tests" dans le workflow existant, histoire de pas faire l'install de Pod 2 fois

Les sides sont créés au format json via selenium IDE. Aymeric va créé un script python qui sera lancé par les tests. Ce script va parcourir les sides json et lancera les tests des sides. Vu le fonctionnement impaginé, j'ai préféré séparé les tests unitaires des tests fonctionnels. On prévoit un side par "type" d'application (video, meeting, authentification, gestion des fichiers etc.)

Et ca empeche de merger les 2 workflows ca ?

@ptitloup
Copy link
Contributor Author

Je me demande juste si on aurait pas pu mettre le "Run Tests" dans le workflow existant, histoire de pas faire l'install de Pod 2 fois

Les sides sont créés au format json via selenium IDE. Aymeric va créé un script python qui sera lancé par les tests. Ce script va parcourir les sides json et lancera les tests des sides. Vu le fonctionnement impaginé, j'ai préféré séparé les tests unitaires des tests fonctionnels. On prévoit un side par "type" d'application (video, meeting, authentification, gestion des fichiers etc.)

Et ca empeche de merger les 2 workflows ca ?

Non pas ca, je t'expliquais juste le fonciotnnement et pourquoi je n'ia pas mis ces tests dans le répertoire tests de main. Après concernant les workflow git hub, on pourrait imaginer les merger mais autant, faire les tests unitaires sur 3 versions de python a du sens, autant sur les tests fonctionnels, je ne le fais que sur une seule version. il faudrait voir pour les mettre éventuellement à la suite, je vais voir

@ptitloup
Copy link
Contributor Author

test commentaire

@ptitloup ptitloup closed this Sep 21, 2023
@ptitloup ptitloup reopened this Sep 21, 2023
@ptitloup ptitloup closed this Sep 21, 2023
@Badatos
Copy link
Collaborator

Badatos commented Sep 21, 2023

test commentaire

Celui-la est automatique ?

@ptitloup
Copy link
Contributor Author

test commentaire

Celui-la est automatique ?

Non pas du tout lol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Badatos Badatos Badatos requested changes

@aitkarrafarid aitkarrafarid Awaiting requested review from aitkarrafarid

@AymericJak AymericJak Awaiting requested review from AymericJak

Assignees

@ptitloup ptitloup

@Badatos Badatos

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ptitloup @Badatos