Skip to content

Conversation

@aurelijusb
Copy link
Owner

Rebased: #199

aurelijusb and others added 13 commits November 10, 2019 18:34
So it would be clear, that all parts are working correctly,
leaving more time to Code review.

Adding screenshots and debugging intermediate results,
so it would be faster to evaluate and/or fix issues.

Using Cypress as one of Headless browser based test framework.
Cypress is running browser as a separate process,
so it was needed for custom debug function to see intermediate results
in GitHub actions as well.

Using e2e-tests.sh to reach docker network used by docker-compose.yml.

Initial idea was to use GitHub Checks Annotations,
but Cypress already formats output with colors.
So using just different indentation.

Video recording disabled to not abuse GitHub –
it already does tremendous job.

Using GitHub Action "if: always()" to store screenshots for both
successful builds and errors.

Documentation:
 * https://www.cypress.io/
 * cypress-io/cypress#3199
 * https://help.github.com/en/actions/automating-your-workflow-with-github-actions/development-tools-for-github-actions#set-a-warning-message-warning
 * https://help.github.com/en/actions/automating-your-workflow-with-github-actions/contexts-and-expression-syntax-for-github-actions#job-status-check-functions
Intention is to create similar project in symfony,
as is deployed on https://hw1.nfq2019.online

Note: https://hw1.nfq2019.online/students.json can be useful.
To cover other edge cases.
There was an error in https://hw1.nfq2019.online/
There should be 51 real students instead of 46 unique names.
@aurelijusb
Copy link
Owner Author

Čia tik Symfony 1 namų darbų kodas 😞

@aurelijusb aurelijusb closed this Dec 14, 2019
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.

4 participants