Skip to content

Conversation

@MrHinsh
Copy link
Member

@MrHinsh MrHinsh commented Feb 5, 2025

🐛 (TfsTestPlansAndSuitesMigrationProcessor.cs): handle duplicate test plan names

Enhance the FindTestPlan method to handle scenarios where multiple test plans have the same name. Previously, the code assumed that test plan names were unique, which could lead to unexpected behavior or errors if duplicates existed. The updated logic checks for multiple test plans with the same name, logs a warning with their IDs, and throws an exception to prevent further execution. This ensures that the migration process does not proceed with ambiguous test plan identification, maintaining data integrity.

… plan names

Enhance the `FindTestPlan` method to handle scenarios where multiple
test plans have the same name. Previously, the code assumed that test
plan names were unique, which could lead to unexpected behavior or
errors if duplicates existed. The updated logic checks for multiple
test plans with the same name, logs a warning with their IDs, and
throws an exception to prevent further execution. This ensures that
the migration process does not proceed with ambiguous test plan
identification, maintaining data integrity.
@MrHinsh MrHinsh enabled auto-merge February 5, 2025 10:58
@MrHinsh MrHinsh merged commit c4f3801 into main Feb 5, 2025
9 checks passed
@MrHinsh MrHinsh deleted the duplicate-test-plan-names branch February 5, 2025 11:07
@nkdagility-actions-bot
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://blue-river-093197403-2616.westeurope.5.azurestaticapps.net

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