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

Cypress Automation - Item, playlist , login , media object specs #5961

Merged
merged 8 commits into from
Aug 7, 2024

Conversation

charumitraravi
Copy link
Contributor

@charumitraravi charumitraravi commented Jul 29, 2024

Changelog:

1.Implemented the page object model for login page and homepage to use in Login tests and browse tests as a POC.
2.Added item creation via UI into cypress commands for reusability
3.Added some fixture data.
4.Added tests to item spec, playlists, media object.

Note:
There might be some failing tests and some flakiness in the media player and playlist tests due to recent changes, which I am currently working on fixing. I am also working on removing static data wherever possible, so I will be updating the environment files soon, probably in a follow up pr tomorrow.

@charumitraravi charumitraravi requested a review from cjcolvar July 29, 2024 20:12
@charumitraravi charumitraravi changed the title Cypress charumitra 3 Cypress Automation - Item, playlist , login , media object specs Jul 30, 2024
Copy link
Member

@cjcolvar cjcolvar left a comment

Choose a reason for hiding this comment

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

Great work on resurrecting the cypress suite and advancing it forward! This will be a good basis for it to start replacing manual QA tests in a systematic way.

I'm not approving and merging it yet since we're so close to release of 7.8. Once the release is cut then this can be merged in.

@charumitraravi
Copy link
Contributor Author

@cjcolvar Thank you. Hope this helps as a starting point to build the automation setup. There are some intermittent failures in 2-3 cases which needs to be addressed a bit more deeply. I have added some comments and details next to them, to help with debugging. I have added a bit more code , mainly in the env files after your last comment. They can be merged together once 7.8 is complete.

@cjcolvar cjcolvar merged commit 07b47e1 into develop Aug 7, 2024
1 check failed
@cjcolvar cjcolvar deleted the cypress_charumitra__3 branch August 7, 2024 19:54
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