Bug 184595 - [WPE] [webkitpy] The driver requirements should be checked before starting the tests
Summary: [WPE] [webkitpy] The driver requirements should be checked before starting th...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WPE WebKit (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Carlos Alberto Lopez Perez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-04-13 09:47 PDT by Carlos Alberto Lopez Perez
Modified: 2018-04-13 19:14 PDT (History)
8 users (show)

See Also:


Attachments
Patch (1.49 KB, patch)
2018-04-13 09:52 PDT, Carlos Alberto Lopez Perez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2018-04-13 09:47:42 PDT
Currently the WPE port doesn't call the driver check_driver() method.

So, what happens when the user tries to run layout tests with the wayland driver, from a non wayland environment you get all crashes.

Instead, the user should get a meaningful error saying that she should use wayland
Comment 1 Carlos Alberto Lopez Perez 2018-04-13 09:52:05 PDT
Created attachment 337901 [details]
Patch
Comment 2 Carlos Alberto Lopez Perez 2018-04-13 09:53:06 PDT
(In reply to Carlos Alberto Lopez Perez from comment #1)
> Created attachment 337901 [details]
> Patch

after this patch, trying to run layout tests on wpe from a non-wayland environment will result in:

18:52:19.121 3274 wptwk successfully started (pid = 3372)
18:52:19.121 3274 
18:52:19.121 3274 Running 29948 tests
18:52:19.121 3274 
18:52:19.121 3274 Checking build ...
18:52:19.215 3274 "Tools/Scripts/build-dumprendertree --release --wpe" took 0.09s
18:52:19.215 3274 Output of ['Tools/Scripts/build-dumprendertree', '--release', '--wpe']:
18:52:19.287 3274 "Tools/Scripts/build-webkittestrunner --release --wpe" took 0.07s
18:52:19.287 3274 Output of ['Tools/Scripts/build-webkittestrunner', '--release', '--wpe']:
18:52:19.288 3274 Starting helper ...
18:52:19.548 3274 Checking system dependencies ...
18:52:19.581 3274 "/usr/sbin/apache2 -v" took 0.03s
18:52:19.581 3274 WAYLAND_DISPLAY or WAYLAND_SOCKET not found in the environment. Cannot run tests.
18:52:19.582 3274 Testing completed, Exit status: -1
18:52:19.583 3274 Stopping Web Platform Test server ...
No handlers could be found for logger "webkitpy.layout_tests.servers.http_server_base"



... instead of thousands of crashes.
Comment 3 Carlos Alberto Lopez Perez 2018-04-13 19:14:08 PDT
Comment on attachment 337901 [details]
Patch

Clearing flags on attachment: 337901

Committed r230653: <https://trac.webkit.org/changeset/230653>
Comment 4 Carlos Alberto Lopez Perez 2018-04-13 19:14:12 PDT
All reviewed patches have been landed.  Closing bug.