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

feat: installations e2e tests with playwright #936

Draft
wants to merge 1 commit into
base: develop_tchap
Choose a base branch
from

Conversation

MarcWadai
Copy link
Contributor

What has been done

cf issues

  • Modify element-web-test file to remove docker homeserver specificities
  • Troubleshoot and try to see how it can work on our configuration

Issues

  • Since we have yarn-linked-dependencies inside the project, playwright cannot be runned. (cannot instantiate two times the playwright instance)

TODO

  • Modify install yarn dep scripts to move it to parents files
  • Modify all relatives paths
  • Adapt e2e tests files

Copy link

gitguardian bot commented Mar 18, 2024

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
4304252 Triggered Generic Password 81d460e playwright/e2e/login.spec.ts View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Revoke and rotate the secret.

  2. If possible, rewrite git history with git commit --amend and git push --force.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants