Skip to content

Conversation

IDzyre
Copy link
Contributor

@IDzyre IDzyre commented Aug 17, 2023

No description provided.

@IDzyre IDzyre changed the title Integrate Cypress Cloud Docs [Documentation:Developer] Integrate Cypress Cloud Docs Aug 21, 2023
@IDzyre IDzyre marked this pull request as ready for review August 21, 2023 17:23
Copy link
Contributor

@nia-gaeaf nia-gaeaf left a comment

Choose a reason for hiding this comment

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

I don't see the header for Testing:

image

@IDzyre
Copy link
Contributor Author

IDzyre commented Aug 22, 2023

image

@IDzyre IDzyre requested a review from nia-gaeaf August 22, 2023 17:24
@nia-gaeaf
Copy link
Contributor

Gotcha!

Copy link
Contributor

@nia-gaeaf nia-gaeaf left a comment

Choose a reason for hiding this comment

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

Looks good! Just have a couple comments.

---
title: Cypress Cloud
---
Submitty has an open source Cypress license to record and run tests via Cypress Cloud. Cypress records tests to [Cypress Cloud](https://cloud.cypress.io/projects/es51qa), where you can view a video of the tests running, and if they failed, a screenshot of the website at the point of failure. This also allows Cypress to automatically parallelize the cypress tests into four separate machines.
Copy link
Contributor

Choose a reason for hiding this comment

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

Might want to say "HTML screenshot" since you can inspect/click things on the screenshot.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can you inspect the screenshot on cypress cloud ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, that's how I got error messages for fixing the accessibility test.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

how, is it not a png?

Copy link
Contributor

Choose a reason for hiding this comment

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

Never mind, it is only not a png on local.

Submitty has an open source Cypress license to record and run tests via Cypress Cloud. Cypress records tests to [Cypress Cloud](https://cloud.cypress.io/projects/es51qa), where you can view a video of the tests running, and if they failed, a screenshot of the website at the point of failure. This also allows Cypress to automatically parallelize the cypress tests into four separate machines.

## Parallelization
Cypress automatically parallelizes the tests by starting four separate VMs to run the tests. The first machine to finish the 'setup' stage takes the test that Cypress estimates will take the longest (Usually accessibility.spec.js). The second machine to finish takes the second longest test... etc. After each machine finishes its current test, it will take the next longest test, until all tests have finished.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe want to mention that this is on Github. I know that this page is on Cypress Cloud, but I think we should be very clear that Cypress Cloud is different than when we run Cypress tests locally.

@IDzyre IDzyre requested a review from nia-gaeaf August 23, 2023 17:17
Copy link
Contributor

@nia-gaeaf nia-gaeaf left a comment

Choose a reason for hiding this comment

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

Approved!

@bmcutler bmcutler merged commit aa96866 into main Aug 24, 2023
@bmcutler bmcutler deleted the integrate_cloud branch August 24, 2023 21:17
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.

3 participants