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

Add test to ensure 'download_concurrency' config is surfaced. #280

Merged

Conversation

hendersonreed
Copy link
Contributor

This test just verifies that the option exists in the UI - I think that's all the coverage required in the UI tests. Thoughts/opinions/reviews/etcetera from @ZitaNemeckova and @ironfroggy are greatly appreciated.

@ZitaNemeckova
Copy link
Member

ZitaNemeckova commented Dec 15, 2020

@hendersonreed It fails sporadically for me on Chromium
Screenshot 2020-12-15 at 14 35 59

I think it needs to wait for http://localhost:8002/ui/repositories?page_size=10&tab=remote to finish.

@hendersonreed
Copy link
Contributor Author

Good to know - I'll add that wait after standup! Thanks.

@hendersonreed
Copy link
Contributor Author

@ZitaNemeckova I added a wait but that ended up causing a different kind of error. I elected to use cy.visit to navigate directly to the repo page - please let me know if you're still seeing intermittent failures - I haven't seen any yet.

@ZitaNemeckova
Copy link
Member

My remote repos are failed and the test tries to click the failed label instead of the kebab
Screenshot 2020-12-16 at 16 22 43

Kebab has a aria-label="Actions". Maybe that will help.

@hendersonreed
Copy link
Contributor Author

@ZitaNemeckova Updated - don't know why I didn't see that aria-label at first. Thanks for working through this with me!

@ZitaNemeckova
Copy link
Member

LGTM 👍

@ZitaNemeckova ZitaNemeckova merged commit 5683bd6 into ansible:master Dec 17, 2020
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.

None yet

2 participants