Skip to content

Skip reporting synthetic failures#11259

Draft
mhdatie wants to merge 7 commits intomasterfrom
mo.atie/apmlp-1296-skip-synthetic-failures
Draft

Skip reporting synthetic failures#11259
mhdatie wants to merge 7 commits intomasterfrom
mo.atie/apmlp-1296-skip-synthetic-failures

Conversation

@mhdatie
Copy link
Copy Markdown

@mhdatie mhdatie commented May 1, 2026

What Does This Do

Extends TagInitializationErrors.java to also tag executionError and test exception synthetic testcases
with dd_tags[test.final_status]=skip, unconditionally. Previously, only intermediate
initializationError retries were tagged.

The tagging logic is refactored into a shared tagSkip helper that uses direct child element lookups
(instead of getElementsByTagName, which searches all descendants) and is idempotent - testcases already
carrying the property are left unchanged.

Motivation

executionError and test exception are framework-level synthetic entries that never represent a real
test result. Without this change, Test Optimization treats them as genuine failures, producing noise in
the test results dashboard. Like initializationError intermediates, they should be skipped so only
real test outcomes are surfaced.

Additional Notes

  • Future PR - Rename from TagInitializationErrors.java to TagSyntheticFailures.java

Contributor Checklist

Jira ticket: APMLP-1296

Note: Once your PR is ready to merge, add it to the merge queue by commenting /merge. /merge -c cancels the queue request. /merge -f --reason "reason" skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.

@mhdatie mhdatie changed the title Mo.atie/apmlp 1296 skip synthetic failures Skip reporting synthetic failures May 1, 2026
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.

1 participant