Bug 199945

Summary: [webkitpy] Allow the testrunner driver to setup the environment completely without needing to start the server process.
Product: WebKit Reporter: Carlos Alberto Lopez Perez <clopez>
Component: Tools / TestsAssignee: Carlos Alberto Lopez Perez <clopez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, bugs-noreply, cgarcia, ews-watchlist, glenn, mcatanzaro, pnormand, webkit-bug-importer, zan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=199855
Attachments:
Description Flags
Patch
none
Patch mcatanzaro: review+

Carlos Alberto Lopez Perez
Reported 2019-07-19 08:52:19 PDT
Since bug 199855 the GTK and WPE testrunner drivers share more code with the base driver. The scripts run-gtk-tests / run-wpe-tests, run-webdriver-tests and run-web-platform-tests need to setup the testrunner driver environment (to maybe start Xvfb or setup needed environment variables). But those scripts don't need the WebTestRunner server process running. Previous to bug 199855 this scripts where setting only the environment variables that don't depend on the Driver tempdir (because this one is created when the driver is started), but starting the driver also executes the server process. I think its desirable to refactor the code to allow creating the driver tempdir and setup the environment completely without needing to start the server process.
Attachments
Patch (7.55 KB, patch)
2019-07-19 09:11 PDT, Carlos Alberto Lopez Perez 		no flags
DetailsFormatted DiffDiff
Patch (2.27 KB, patch)
2019-07-22 07:10 PDT, Carlos Alberto Lopez Perez 		mcatanzaro: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Carlos Alberto Lopez Perez
Comment 1 2019-07-19 09:11:31 PDT
Created attachment 374468 [details] Patch
Carlos Alberto Lopez Perez
Comment 2 2019-07-19 17:55:56 PDT
Comment on attachment 374468 [details] Patch Clearing flags on attachment: 374468 Committed r247663: <https://trac.webkit.org/changeset/247663>
Carlos Alberto Lopez Perez
Comment 3 2019-07-19 17:56:00 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 4 2019-07-19 17:56:16 PDT
<rdar://problem/53332564>
Carlos Alberto Lopez Perez
Comment 5 2019-07-22 06:52:11 PDT
On further thinking, it seems to me that is also worth also adding an unit test for this.
Carlos Alberto Lopez Perez
Comment 6 2019-07-22 07:10:45 PDT
Created attachment 374597 [details] Patch
Michael Catanzaro
Comment 7 2019-07-22 07:19:34 PDT
Comment on attachment 374597 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=374597&action=review > Tools/Scripts/webkitpy/port/driver_unittest.py:396 > + self.assertNotEqual(environ_driver[var], None) > + > + > def test_create_temporal_home_dir(self): All the other surrounding functions have only one line between them (though I thought two was supposed to be standard for python).
Carlos Alberto Lopez Perez
Comment 8 2019-07-22 07:25:07 PDT
Committed r247682: <https://trac.webkit.org/changeset/247682>
Note You need to log in before you can comment on or make changes to this bug.