adobe / aem-guides-wknd

Tutorial Code companion for Getting Started Developing with AEM Sites WKND Tutorial
https://experienceleague.adobe.com/docs/experience-manager-learn/getting-started-wknd-tutorial-develop/overview.html
MIT License
288 stars 569 forks source link

Cypress UI tests #433

Closed andresbott closed 11 months ago

andresbott commented 1 year ago

Migrate UI test to cypress

Description

Remove the default selenium based UI tests, and replace them with cypress

The new tests a) follow the author flow verifying:

b) verify the public page

Related Issue

jira.corp.adobe.com/browse/CQ-4354669

Recordings of the verification

https://github.com/adobe/aem-guides-wknd/assets/1543913/646e96ea-be75-47dd-a692-5492c45311b1

https://github.com/adobe/aem-guides-wknd/assets/1543913/b58e1eeb-88e6-4988-9fb8-00f8111a7408

davidjgonzalez commented 1 year ago

@andresbott should the PR be moved out of Draft yet?

andresbott commented 11 months ago

@davidjgonzalez sorry, i missed the notification; I checked again this tests on a local environment and all is green; this can be merged

SachinMali commented 11 months ago

@andresbott Can you please resolve the remaining conversations, we can merge it soon then, thanks!

andresbott commented 11 months ago

@SachinMali done; the points in the conversations were fixed; i just closed the conversations

andresbott commented 11 months ago

@SachinMali as discussed I verified again the Local execution of all the steps to execute the UI tests against an AEMCS instance. as well I updated the Readme accordingly

SachinMali commented 11 months ago

@andresbott Thanks, appreciated calling out it's AEMCS specific.