Bug 229311 – [build.webkit.org] Port old unit test with the expected build steps to the new buildbot

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Carlos Alberto Lopez Perez]

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

[Carlos Alberto Lopez Perez]

Created attachment 435923 [details]
Patch

[Carlos Alberto Lopez Perez]

Created attachment 435926 [details]
Patch

[Aakash Jain]

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 on attachment 435926 [details]
Patch

Clearing flags on attachment: 435926

Committed r281286 (240711@main): <https://commits.webkit.org/240711@main>

[Carlos Alberto Lopez Perez]

All reviewed patches have been landed.  Closing bug.

[Carlos Alberto Lopez Perez]

(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]

<rdar://problem/82149509>

[Carlos Alberto Lopez Perez]

(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