Summary: | [GTK] Improve the way unit test are run and the results reported | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bugs-noreply, clopez, lforschler, Ms2ger, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | Gtk, InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Carlos Garcia Campos
2017-08-30 08:29:38 PDT
Created attachment 319362 [details]
Patch
Current test results is something like this:
Tests failed (4): /WebKit2Gtk/TestWebKitWebContext, /WebKit2Gtk/TestWebExtensions, /WebKit2Gtk/TestWebsiteData, /WebKit2Gtk/TestBackForwardList
Tests that timed out (3): /WebKit2Gtk/TestWebViewEditor, /WebKit2Gtk/TestWebKitWebView, /WebKit2/TestWebKit2
With the patch the output is like this:
Unexpected failures (9)
/WebKit2Gtk/TestSSL
/webkit2/WebKitWebView/tls-subresource
/webkit2/WebKitWebView/load-failed-with-tls-errors
/webkit2/WebKitWebView/ssl
/WebKit2Gtk/TestAutomationSession
/webkit2/WebKitAutomationSession/request-session
/WTF/TestWTF
WTF_DateMath.calculateLocalTimeOffset
/WebKit2Gtk/TestWebKitAccessibility
/webkit2/WebKitAccessibility/atspi-basic-hierarchy
/WebKit2Gtk/TestLoaderClient
/webkit2/WebKitWebView/title
/WebKit2Gtk/TestWebExtensions
/webkit2/WebKitWebExtension/form-controls-associated-signal
/WebKit2Gtk/TestUIClient
/webkit2/WebKitWebView/mouse-target
Unexpected crashes (4)
/WebKit2/TestWebKit2
WebKit2.InputMethodFilterContextFocusOutDuringOngoingComposition
WebKit2.InputMethodFilterContextMouseClickDuringOngoingComposition
WebKit2.MouseMoveAfterCrash
WebKit2.InputMethodFilterComposeKey
Unexpected timeouts (2)
/WebKit2Gtk/TestLoaderClient
/webkit2/WebKitWebView/reload
/webkit2/WebKitURIRequest/http-headers
Created attachment 319364 [details]
Patch
Just fixed some typos in the ChangeLog.
Comment on attachment 319364 [details]
Patch
The information about the test run is much cleaner now. Thanks!
BTW.. I think the buildmaster is not automatically restarted, so you may have to send a mail to admin <at> webkit.org requesting it to get restarted
(In reply to Carlos Alberto Lopez Perez from comment #3) > Comment on attachment 319364 [details] > Patch > > The information about the test run is much cleaner now. Thanks! > BTW.. I think the buildmaster is not automatically restarted, so you may > have to send a mail to admin <at> webkit.org requesting it to get restarted Thanks! Lucas, do you need to manually restart the master when the config file changes? Maybe it's easier if you just cq+ this when you can, and restart the master right after the patch lands? (In reply to Carlos Garcia Campos from comment #4) > (In reply to Carlos Alberto Lopez Perez from comment #3) > > Comment on attachment 319364 [details] > > Patch > > > > The information about the test run is much cleaner now. Thanks! > > BTW.. I think the buildmaster is not automatically restarted, so you may > > have to send a mail to admin <at> webkit.org requesting it to get restarted > > Thanks! > > Lucas, do you need to manually restart the master when the config file > changes? Maybe it's easier if you just cq+ this when you can, and restart > the master right after the patch lands? I think the sync/copy from the webkit repo to the server where the config runs is done manually. So what I do when doing changes to the master is this: - Land the patch - Send an email to admin at webkit.org telling that I landing the patch at rxxxx and asking if they can schedule a restart of the master to apply the new config. My 2 cents Committed r221474: <http://trac.webkit.org/changeset/221474> |