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

AzureTestPlan@V0 Code Refactoring. #20816

Merged
merged 15 commits into from
Feb 3, 2025
Merged

Conversation

adityashahms
Copy link
Contributor

@adityashahms adityashahms commented Jan 30, 2025

Task name: AzureTestPlanV0

Description:

  • Changed the code flow for both manual and automated flow.
  • Moved common helpers and utils to common files
  • Introduced an interface IOperationResult and changed function signature for each stage outcome for implementing proper error handling and exit status propagation.
    • For automated flow, introduced an interface ITestExecutor to decouple concrete classes dependency.
    • The new flow is under an env variable {MyEnvVar} which act as a knob for the old and new automated flow for the time being
    • Moved the old files to OldAutomated Folder
    • Tested the flow for Maven Gradle and Python automated and manual flow as well

Documentation changes required: (Y/N) N

Added unit tests: (Y/N) N

Attached related issue: (Y/N) N

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

Validation Screenshot:
Python:
image
image

Maven:
image
image

Gradle:
image
image

Tasks/AzureTestPlanV0/runTestPlan.ts Outdated Show resolved Hide resolved
Tasks/AzureTestPlanV0/runTestPlan.ts Outdated Show resolved Hide resolved
Tasks/AzureTestPlanV0/runTestPlan.ts Outdated Show resolved Hide resolved
Tasks/AzureTestPlanV0/runTestPlan.ts Show resolved Hide resolved
Tasks/AzureTestPlanV0/Automated Flow/automatedFlow.ts Outdated Show resolved Hide resolved
Tasks/AzureTestPlanV0/Automated Flow/automatedFlow.ts Outdated Show resolved Hide resolved
@adityashahms adityashahms merged commit f0eff63 into master Feb 3, 2025
11 checks passed
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.

2 participants