Bug 229311

Summary: [build.webkit.org] Port old unit test with the expected build steps to the new buildbot
Product: WebKit Reporter: Carlos Alberto Lopez Perez <clopez>
Component: Tools / TestsAssignee: Carlos Alberto Lopez Perez <clopez>
Status: RESOLVED FIXED    
Severity: Normal CC: aakash_jain, jbedard, ryanhaddad, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=219364
https://bugs.webkit.org/show_bug.cgi?id=229312
https://bugs.webkit.org/show_bug.cgi?id=229319
Attachments:
Description Flags
Patch
none
Patch none

Carlos Alberto Lopez Perez
Reported 2021-08-19 16:55:41 PDT
With the previous buildbot we had a very useful unit test that described all the expected steps for each builder, so each time you added or modified a builder you had also to add/modify the corresponding entry on the unit test, which served as an extra check that everything was going to work as expected. With the migration to the new buildbot version this test was renamed and disabled because it was not compatible with the new buildbot version (see bug 219364 ) It would be a good idea to port this test to the new version
Attachments
Patch (93.39 KB, patch)
2021-08-19 17:00 PDT, Carlos Alberto Lopez Perez 		no flags
DetailsFormatted DiffDiff
Patch (93.36 KB, patch)
2021-08-19 17:07 PDT, Carlos Alberto Lopez Perez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Carlos Alberto Lopez Perez
Comment 1 2021-08-19 17:00:58 PDT
Created attachment 435923 [details] Patch
Carlos Alberto Lopez Perez
Comment 2 2021-08-19 17:07:01 PDT
Created attachment 435926 [details] Patch
Aakash Jain
Comment 3 2021-08-19 17:25:19 PDT
Comment on attachment 435926 [details] Patch Thanks for taking care of this. We can also re-write https://github.com/WebKit/WebKit/blob/main/Tools/CISupport/ews-build/factories_unittest.py to make it similar to this one.
Carlos Alberto Lopez Perez
Comment 4 2021-08-19 17:39:04 PDT
Comment on attachment 435926 [details] Patch Clearing flags on attachment: 435926 Committed r281286 (240711@main): <https://commits.webkit.org/240711@main>
Carlos Alberto Lopez Perez
Comment 5 2021-08-19 17:39:09 PDT
All reviewed patches have been landed. Closing bug.
Carlos Alberto Lopez Perez
Comment 6 2021-08-19 17:40:43 PDT
(In reply to Aakash Jain from comment #3) > Comment on attachment 435926 [details] > Patch > > Thanks for taking care of this. > We can also re-write > https://github.com/WebKit/WebKit/blob/main/Tools/CISupport/ews-build/ > factories_unittest.py to make it similar to this one. you're welcome, thanks to you also for the quick review :) yes, good idea!
Radar WebKit Bug Importer
Comment 7 2021-08-19 17:40:48 PDT
<rdar://problem/82149509>
Carlos Alberto Lopez Perez
Comment 8 2021-08-19 18:08:00 PDT
(In reply to Carlos Alberto Lopez Perez from comment #6) > (In reply to Aakash Jain from comment #3) > > Comment on attachment 435926 [details] > > Patch > > > > Thanks for taking care of this. > > We can also re-write > > https://github.com/WebKit/WebKit/blob/main/Tools/CISupport/ews-build/ > > factories_unittest.py to make it similar to this one. > > you're welcome, thanks to you also for the quick review :) > > yes, good idea! Proposed patch adding that in bug 229319
Note You need to log in before you can comment on or make changes to this bug.