-
Notifications
You must be signed in to change notification settings - Fork 10
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
Automated UI E2E Testing #990
Comments
Relates to #419, which had a backend api testing focus up to now, might be a good idea to rename the issue to Also, relates to eclipse-tractusx/portal-backend#617: a similar issue might be needed for portal frontend, and maybe it's better to use LocalDev as that sandbox could be kept more easily up to date with portal-backend changes than the umbrella, at the same time, LocalDev doesn't provide any interfaces other than the portal-backend and the keycloak instances, in comparison the umbrella values.yaml for portal. That decision depends on the kind of test cases and if they are supposed to include also functionalities where interfaces (other than portal-backend, centralidp, sharedidp) are involved. |
@evegufy I won’t be able to join the refinement session today and would like to discuss it in the Open Planning on Wednesday. |
Sana will connect to @charmi-v, as she's also planning to introduce Cypress. |
I have listed some direct references for key subtasks to kickstart Cypress implementation in the repo:
Please share thoughts regarding the CI/CD Integrations & defining testing modules |
Committers: |
Hi @charmi-v, I was informed by @mgarciaLKS and he and @CDiezRodriguez and @gomezbc, would like to contribute. |
Hello all, @gomezbc is not available, but @CDiezRodriguez and I would love to help with this task if possible. |
@mgarciaLKS great! I guess you could start by creating an issue in the portal-frontend for creating a GH action workflow which spins up the Umbrella sandbox in a Kind cluster. |
Agreed, @evegufy , that sounds perfect. |
We have an internal sync scheduled for tomorrow (at 2:15 IST) to plan the test automation alignment. |
Meeting Points Discussed
RoadmapStep 1:
Step 2:
Step 3:
By following this roadmap, we can establish a robust and scalable automation testing strategy that aligns with our project's goals and timelines. Please share your feedback and suggestions. @evegufy |
@charmi-v sounds like a plan 👍 maybe you could share the progress in nexts week open portal meeting Nov 28? There we could talk about open decision items as well. |
Since multiple contributors will work on the Cypress Test Automation, It would be beneficial to adhere to the following best practices to maintain consistency and uniformity in the tests: Test Design Best PracticesTest Structure:
Data Management:
Chai: Test Runner and Assertions:
Error Handling:
Additional Configuration
Browser Compatibility Testing:
Performance and OptimizationTest Run Optimization:
Reporting and Debugging
Logging and Insights:
|
@SanaSalem01 , the Test Design Best Practices you shared are helpful. I plan to incorporate them into the frontend cypress README as a reference for developers. For the initial phase, I suggest excluding Browser Compatibility Testing, Performance and Optimization, and Reporting and Debugging, focusing on core practices to start with. |
Overview
Explain the topic in 2 sentences
What's the benefit?
This implementation improves quality assurance by providing efficient, reliable testing automation. The streamlined CI/CD integration enables faster releases with a reduced risk of errors reaching production.
What are the Risks/Dependencies ?
It requires collaboration and review of the CI/CD pipeline to ensure seamless integration.
Detailed explanation
Current implementation
Proposed improvements
@cypress/code-coverage
plugin will collect and save code coverage metrics, enabling deeper insights into test coverage.Feature Team
Contributor
Committer
User Stories
Acceptance Criteria
Test Cases
Test Case 1
Steps
Expected Result
Architectural Relevance
The following items are ensured (answer: yes) after this issue is implemented:
Justification: (Fill this out, if at least one of the checkboxes above cannot be ticked. Contact the Architecture Management Committee to get an approval for the justification)
Additional information
The text was updated successfully, but these errors were encountered: