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.

Sign up for GitHub

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 cypress cloud to see videos of cypress runs on CI #401

Draft
wants to merge 10 commits into
base: develop_tchap
Choose a base branch
from
Draft

Add cypress cloud to see videos of cypress runs on CI #401

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
94edd88
Use our own cloud.cypress.io, maybe it will be useful
estellecomment Jan 30, 2023
8550191
Record CI jobs in cloud.cypress.io (attempt!)
estellecomment Jan 30, 2023
819ea3d
Try again connecting CI runs to cypress cloud
estellecomment Jan 31, 2023
337958e
upgrade github actions, just in case
estellecomment Jan 31, 2023
dcadd4a
Use the --record flag (stull trying to connect to cypress cloud)
estellecomment Jan 31, 2023
ae5ba3b
Run cypress on pushes. All pushes belong to a pull request in the end…
estellecomment Jan 31, 2023
f6bbbf0
Try storing artifacts in the github actions, for another way to acces…
estellecomment Jan 31, 2023
9db3925
Remove import that is now obsolete and breaks the build
estellecomment Jan 30, 2023
952d009
Merge branch 'develop_tchap' into cypress-cloud
estellecomment Sep 21, 2023
638d93e
Merge branch 'develop_tchap' into cypress-cloud
estellecomment Oct 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 18 additions & 1 deletion .github/workflows/cypress.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,28 @@ jobs:
browser: chrome
# headed: true # tchap uses headless, since we don't record videos.

command: "yarn test:cypress"
command: "yarn test:cypress:record"
record: true
# Note : 'working-directory' changes pwd, for everything. If you use it you need to change relative paths for everything, it's tricky.
env:
E2E_TEST_USER_EMAIL: ${{ secrets.E2E_TEST_USER_EMAIL }}
E2E_TEST_USER_PASSWORD: ${{ secrets.E2E_TEST_USER_PASSWORD }}
E2E_TEST_USER_SECURITY_KEY: ${{ secrets.E2E_TEST_USER_SECURITY_KEY }}
E2E_TEST_USER_HOMESERVER_URL: "https://matrix.agent1.tchap.incubateur.net"
E2E_TEST_USER_HOMESERVER_SHORT: "agent1.tchap.incubateur.net"
# pass GitHub token to allow accurately detecting a build vs a re-run build
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# pass the Dashboard record key as an environment variable
CYPRESS_RECORD_KEY: ${{ secrets.CYPRESS_RECORD_KEY }}
# pass the project ID
CYPRESS_PROJECT_ID: 'x8og2r'

- name: Upload Artifact
if: failure()
uses: actions/upload-artifact@v3
with:
name: cypress-results
path: |
cypress/screenshots
cypress/videos
cypress/synapselogs
2 changes: 1 addition & 1 deletion cypress.config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ getVar("E2E_TEST_USER_HOMESERVER_SHORT");
export default defineConfig({
watchForFileChanges: false,
videoUploadOnPasses: false,
projectId: "ppvnzg",
projectId: 'x8og2r',
experimentalInteractiveRunEvents: true,
defaultCommandTimeout: 10000,
chromeWebSecurity: false,
Expand Down
1 change: 1 addition & 0 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@
"test": "jest",
"test:cypress": "cypress run",
"test:cypress:open": "cypress open",
"test:cypress:record": "cypress run --record",
"coverage": "yarn test --coverage",
"analyse:unused-exports": "ts-node ./scripts/analyse_unused_exports.ts",
"postinstall": "./scripts/tchap/apply_patches.sh",
Expand Down
Loading