Bug 215337

Summary: [ews] emails for test failures should include link to test history
Product: WebKit Reporter: Aakash Jain <aakash_jain>
Component: Tools / TestsAssignee: Aakash Jain <aakash_jain>
Status: RESOLVED FIXED    
Severity: Normal CC: aakash_jain, ap, darin, dewei_zhu, jbedard, ryanhaddad, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=215425
Bug Depends on: 215331    
Bug Blocks: 215220    
Attachments:
Description Flags
Patch
jbedard: review+
Sample email none

Aakash Jain
Reported 2020-08-10 14:16:28 PDT
r265368 added emails notification for layout test failures (although not enabled completely yet). These ews emails (for layout test failures) should also include link to test history (next to the failing test name). It would make it easy to directly click on the history url and see the history of that test. This might help patch authors in determining whether the test failure is real or false positive (because of flaky test).
Attachments
Patch (2.23 KB, patch)
2020-08-10 14:19 PDT, Aakash Jain 		jbedard: review+
DetailsFormatted DiffDiff
Sample email (159.45 KB, image/png)
2020-08-10 14:22 PDT, Aakash Jain 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Aakash Jain
Comment 1 2020-08-10 14:19:45 PDT
Created attachment 406329 [details] Patch
Aakash Jain
Comment 2 2020-08-10 14:22:08 PDT
Created attachment 406330 [details] Sample email
Jonathan Bedard
Comment 3 2020-08-12 10:00:38 PDT
Comment on attachment 406329 [details] Patch Seems like this would require https://bugs.webkit.org/show_bug.cgi?id=215331 to land first
Aakash Jain
Comment 4 2020-08-12 10:57:31 PDT
Committed r265555: <https://trac.webkit.org/changeset/265555>
Radar WebKit Bug Importer
Comment 5 2020-08-12 10:59:09 PDT
<rdar://problem/66919702>
Note You need to log in before you can comment on or make changes to this bug.