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

RP E2E Test Execution #3856

Draft
wants to merge 51 commits into
base: master
Choose a base branch
from

Conversation

shubhadapaithankar
Copy link
Collaborator

Which issue this PR addresses:

Fixes: ARO-9501

What this PR does / why we need it:

This PR introduces an E2E testing stage using Docker Compose to orchestrate the RP, Selenium, and E2E test services. The pipeline pulls the required RP image from Azure Container Registry (ACR) and ensures the correct environment is set up for the E2E tests. It helps validate the end-to-end functionality of the system and ensures that tests are running using containerized RP images.

  • Why we need it: The E2E stage ensures that the entire system is tested in a environment, using containerized components (including RP) and validating workflows through Selenium for browser-based testing.
  • This change helps automate the test execution process for the complete system, ensuring that every aspect of the application functions correctly in a containerized environment before deployment.

Test plan for issue:

  1. E2E tests are executed within the pipeline's new E2E stage using Docker Compose.
  2. The tests are triggered automatically after the Containerized stage, ensuring seamless orchestration.

Is there any documentation that needs to be updated for this PR?

N/A

How do you know this will function as expected in production?

N/A

@kevinobriendotca kevinobriendotca force-pushed the shubhadapaithankar/ARO-9501-RP-E2E-Test branch from 3d795ef to deeca9a Compare November 1, 2024 22:48
@shubhadapaithankar shubhadapaithankar force-pushed the shubhadapaithankar/ARO-9501-RP-E2E-Test branch from 153769a to e6ba412 Compare November 5, 2024 15:58
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.

3 participants