Bug 78168

Summary: nrwt: run_webkit_tests_integrationtest.py depends on being able to share a port w/ between master and worker
Product: WebKit Reporter: Dirk Pranke <dpranke>
Component: Tools / TestsAssignee: Dirk Pranke <dpranke>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, gtk-ews, ojan, tony, webkit.review.bot, xan.lopez
Priority: P2 Keywords: NRWT
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 103824    
Attachments:
Description Flags
Patch ojan: review+, gtk-ews: commit-queue-

Dirk Pranke
Reported 2012-02-08 15:40:26 PST
Some of the integration tests in run_webkit_tests_integrationtest.py depend on being able to pass data back and forth in shared memory between 'manager' and 'worker'; this only happens to work when we have inline workers, and this causes complications in the manager_worker_broker interface (because we need to pass the shared host and port back and forth). These tests should be rewritten to either be true unit tests or to be proper black-box integration tests that don't rely on custom ports.
Attachments
Patch (39.10 KB, patch)
2012-12-05 15:58 PST, Dirk Pranke
ojan: review+
gtk-ews: commit-queue-
Dirk Pranke
Comment 1 2012-12-05 15:58:05 PST
Dirk Pranke
Comment 2 2012-12-05 15:59:58 PST
Note that the original problem in the description was that we had to jump through some hoops in the message_pool (nee manager_worker_broker) classes in order to pass the shared port across; this was fixed some time ago by proper use of MockHosts and propagating the host across. However, the test code was still ugly as a result of the faking/overriding going on, so this makes things (IMO) quite a bit cleaner and clearer.
WebKit Review Bot
Comment 3 2012-12-05 16:00:29 PST
Attachment 177853 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Tools/ChangeLog', u'Tools/Scripts/webkitpy..." exit_code: 1 Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:155: [get_test_results] Passing unexpected keyword argument 'logging_stream' in function call [pylint/E1123] [5] Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:160: [get_test_results] Instance of 'int' has no 'result_summary' member (but some types could not be inferred) [pylint/E1103] [5] Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:161: [get_test_results] Instance of 'int' has no 'result_summary' member (but some types could not be inferred) [pylint/E1103] [5] Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:163: [get_test_results] Instance of 'int' has no 'retry_summary' member (but some types could not be inferred) [pylint/E1103] [5] Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:164: [get_test_results] Instance of 'int' has no 'retry_summary' member (but some types could not be inferred) [pylint/E1103] [5] Total errors found: 5 in 7 files If any of these errors are false positives, please file a bug against check-webkit-style.
kov's GTK+ EWS bot
Comment 4 2012-12-05 16:12:20 PST
Dirk Pranke
Comment 5 2012-12-05 16:45:39 PST
I think the style failures are due to the style bot being behind; the code is correct and check-webkit-style runs cleanly on my machine. I've filed bug 104190 for that.
Dirk Pranke
Comment 6 2012-12-05 16:54:37 PST
Note You need to log in before you can comment on or make changes to this bug.