Skip to content

Initial E2E tests for Kimchi - #1293

Open
ramonmedeiros wants to merge 6 commits into
kimchi-project:masterfrom
ramonmedeiros:add_e2e_tests
Open

ramonmedeiros wants to merge 6 commits into
kimchi-project:masterfrom
ramonmedeiros:add_e2e_tests

Conversation

@ramonmedeiros

@ramonmedeiros ramonmedeiros commented Dec 28, 2019

Copy link
Copy Markdown

First proposal to create E2E tests for Kimchi

The tests were created in a separated folder uiTests, and there is a README.md with all instructions to trigger them.

The tests are using Chrome and expect version 78.0.3904.105.0

gif

Comment thread .gitignore Outdated
Comment thread uiTests/README.md Outdated
Comment thread uiTests/pages/kimchi_project/login.py Outdated
Comment thread uiTests/README.md Outdated
Comment thread uiTests/pages/kimchi_project/login.py Outdated

@alinefm alinefm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@ramonmedeiros I tried to run this and seems like it is stuck in something. When I run the command it does not do anything, apparently. I am on Ubuntu 19.10

read -s pass; KIMCHI_USERNAME=XXX KIMCHI_PASSWORD=YYY DEBUG=true make

I think it may be because the PR does not include the Makefile. Shouldn't it have one inside uiTestsdir?

Comment thread tests/ui/pages/templates.py Outdated
Comment thread .gitignore
*.rej
*.pem
ui/pages/help/*/*.html
tests/ui/.env

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we still need .env?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Its optional

Comment thread tests/ui/README.md Outdated
Comment thread tests/ui/README.md Outdated
Comment thread tests/ui/pages/templates.py Outdated
@alinefm

alinefm commented Jan 21, 2020

Copy link
Copy Markdown
Member

The comments/questions I made related to dependency on Chrome and browser support on Wok PR (kimchi-project/wok#292) also make sense in here

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants