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

Handle manual test cases associated with same automated tests #5064

Merged
merged 2 commits into from
Dec 24, 2024

Conversation

KathanS
Copy link
Contributor

@KathanS KathanS commented Dec 17, 2024

Changes in PR are specifically for Test Result publish flow from ATP task.

We see that when same AutomatedTest is associated with different manual test cases - Tc1 and Tc2, in the output test tab, we only see Tc1 as per below, (Mapping: Tc1 -> AutomatedTest1, Tc2 -> AutomatedTest1, Tc3 -> AutomatedTest2)

image

We are adding changes to ensure that test is published if present in list of automated test points passed to publish command in agent and result xml file has result corresponding to that automatedTestName

After fix,

image

We are now publishing test results based on list of input tests given by user rather than relying on result xml file that will have results as per one automated test executed only once even if specified multiple times by user for execution.

@KathanS KathanS requested review from a team as code owners December 17, 2024 12:11
@KathanS KathanS changed the title Handle manual test cases with same automated tests associated Handle manual test cases associated with same automated tests Dec 17, 2024
@KathanS KathanS added the bug label Dec 17, 2024
Copy link
Contributor

@adityashahms adityashahms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@KathanS KathanS merged commit 08edb6e into master Dec 24, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants