199941 2019-07-19 07:03:27 -0700 [ews-build] Display pre-existing Layout test failure names in the build summary 2019-07-19 08:55:04 -0700 1 1 1 Unclassified WebKit Tools / Tests Other Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=199525 P2 Normal --- 1 aakash_jain aakash_jain aakash_jain ap jbedard ryanhaddad webkit-bug-importer oldest_to_newest 1554194 0 aakash_jain 2019-07-19 07:03:27 -0700 Currently when there is a failure on clean tree (ToT), layout-test build summary shows: "Found x pre-existing test failure" (e.g.: https://ews-build.webkit.org/#/builders/18/builds/102). This is fine, but not very informative. It would be better to include the names of the pre-existing failures in the build summary string. Those failing test names would show up on the builder page, and it would be very convenient to notice the test failure pattern over multiple builds. 1554195 1 374462 aakash_jain 2019-07-19 07:04:16 -0700 Created attachment 374462 Patch 1554218 2 374462 aakash_jain 2019-07-19 08:54:31 -0700 Comment on attachment 374462 Patch Clearing flags on attachment: 374462 Committed r247646: <https://trac.webkit.org/changeset/247646> 1554219 3 aakash_jain 2019-07-19 08:54:35 -0700 All reviewed patches have been landed. Closing bug. 374462 2019-07-19 07:04:16 -0700 2019-07-19 08:54:31 -0700 Patch bug-199941-20190719100416.patch text/plain 1578 aakash_jain SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl dmlzaW9uIDI0NzY0NCkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz ICsxLDEzIEBACisyMDE5LTA3LTE5ICBBYWthc2ggSmFpbiAgPGFha2FzaF9qYWluQGFwcGxlLmNv bT4KKworICAgICAgICBbZXdzLWJ1aWxkXSBEaXNwbGF5IHByZS1leGlzdGluZyBMYXlvdXQgdGVz dCBmYWlsdXJlIG5hbWVzIGluIHRoZSBidWlsZCBzdW1tYXJ5CisgICAgICAgIGh0dHBzOi8vYnVn cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xOTk5NDEKKworICAgICAgICBSZXZpZXdlZCBi eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIEJ1aWxkU2xhdmVTdXBwb3J0L2V3cy1idWls ZC9zdGVwcy5weToKKyAgICAgICAgKEFuYWx5emVMYXlvdXRUZXN0c1Jlc3VsdHMucmVwb3J0X3By ZV9leGlzdGluZ19mYWlsdXJlcyk6CisKIDIwMTktMDctMTkgIENhcmxvcyBBbGJlcnRvIExvcGV6 IFBlcmV6ICA8Y2xvcGV6QGlnYWxpYS5jb20+CiAKICAgICAgICAgW0dUS11bV1BFXSBydW4td2Vi a2l0LXRlc3RzIHNob3VsZCBydW4gd2l0aCBhIHRlbXBvcmFsIGhvbWVkaXIKSW5kZXg6IFRvb2xz L0J1aWxkU2xhdmVTdXBwb3J0L2V3cy1idWlsZC9zdGVwcy5weQo9PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29s cy9CdWlsZFNsYXZlU3VwcG9ydC9ld3MtYnVpbGQvc3RlcHMucHkJKHJldmlzaW9uIDI0NzY0NCkK KysrIFRvb2xzL0J1aWxkU2xhdmVTdXBwb3J0L2V3cy1idWlsZC9zdGVwcy5weQkod29ya2luZyBj b3B5KQpAQCAtMTAyNyw3ICsxMDI3LDggQEAgY2xhc3MgQW5hbHl6ZUxheW91dFRlc3RzUmVzdWx0 cyhidWlsZHN0ZQogICAgICAgICBzZWxmLmJ1aWxkLnJlc3VsdHMgPSBTVUNDRVNTCiAgICAgICAg IHNlbGYuZGVzY3JpcHRpb25Eb25lID0gJ1Bhc3NlZCBsYXlvdXQgdGVzdHMnCiAgICAgICAgIHBs dXJhbFN1ZmZpeCA9ICdzJyBpZiBsZW4oY2xlYW5fdHJlZV9mYWlsdXJlcykgPiAxIGVsc2UgJycK LSAgICAgICAgbWVzc2FnZSA9ICdGb3VuZCB7fSBwcmUtZXhpc3RpbmcgdGVzdCBmYWlsdXJle30n LmZvcm1hdChsZW4oY2xlYW5fdHJlZV9mYWlsdXJlcyksIHBsdXJhbFN1ZmZpeCkKKyAgICAgICAg Y2xlYW5fdHJlZV9mYWlsdXJlc19zdHJpbmcgPSAnLCAnLmpvaW4oW2ZhaWx1cmVfbmFtZSBmb3Ig ZmFpbHVyZV9uYW1lIGluIGNsZWFuX3RyZWVfZmFpbHVyZXNdKQorICAgICAgICBtZXNzYWdlID0g J0ZvdW5kIHt9IHByZS1leGlzdGluZyB0ZXN0IGZhaWx1cmV7fToge30nLmZvcm1hdChsZW4oY2xl YW5fdHJlZV9mYWlsdXJlcyksIHBsdXJhbFN1ZmZpeCwgY2xlYW5fdHJlZV9mYWlsdXJlc19zdHJp bmcpCiAgICAgICAgIHNlbGYuYnVpbGQuYnVpbGRGaW5pc2hlZChbbWVzc2FnZV0sIFNVQ0NFU1Mp CiAgICAgICAgIHJldHVybiBkZWZlci5zdWNjZWVkKE5vbmUpCiAK