Bug 92673

Summary: nrwt: --verbose logging does not work right on windows
Product: WebKit Reporter: Dirk Pranke <dpranke>
Component: New BugsAssignee: Dirk Pranke <dpranke>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, ojan, tmpsantos, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 92845    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch
ojan: review+
remove wontfix from add_skipped_tests, expectation_for_skipped_tests none

Description Dirk Pranke 2012-07-30 13:22:06 PDT
nrwt: --verbose logging does not work right on windows
Comment 1 Dirk Pranke 2012-07-30 14:05:30 PDT
Created attachment 155357 [details]
Patch
Comment 2 Dirk Pranke 2012-07-30 14:06:39 PDT
Comment on attachment 155357 [details]
Patch

landing as an unreviewed build fix so I can get output from the bot ... I will land a revised patch w/ a test later.
Comment 3 Dirk Pranke 2012-07-30 14:07:59 PDT
Committed r124090: <http://trac.webkit.org/changeset/124090>
Comment 4 Dirk Pranke 2012-07-30 14:51:42 PDT
Reopening to attach new patch.
Comment 5 Dirk Pranke 2012-07-30 14:51:44 PDT
Created attachment 155370 [details]
Patch
Comment 6 Dirk Pranke 2012-07-30 14:55:37 PDT
you'll probably want to look at both patches ... the second is just the test for the first. it turns out it was tricky to write because outputcapture doesn't play nicely with multiprocess tests using the test port.
Comment 7 Dirk Pranke 2012-07-30 16:45:58 PDT
Created attachment 155398 [details]
remove wontfix from add_skipped_tests, expectation_for_skipped_tests
Comment 8 Dirk Pranke 2012-07-30 16:49:39 PDT
Comment on attachment 155398 [details]
remove wontfix from add_skipped_tests, expectation_for_skipped_tests

ignore, wrong bug.
Comment 9 Dirk Pranke 2012-07-31 12:10:02 PDT
Committed r124235: <http://trac.webkit.org/changeset/124235>
Comment 10 Thiago Marcos P. Santos 2012-07-31 23:40:59 PDT
This patch caused a regression on the test-webkitpy. I'm going to open a bug for fixing it.

[945/1552] webkitpy.layout_tests.run_webkit_tests_integrationtest.MainTest.test_verbose erred:
  Traceback (most recent call last):
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py", line 934, in test_verbose
      _, _, err, _ = logging_run(['--verbose', '--fully-parallel', '--child-processes', '2', 'passes/text.html', 'passes/image.html'], tests_included=True)
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py", line 110, in logging_run
      res, buildbot_output, regular_output = run_and_capture(port_obj, options, parsed_args, shared_port)
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py", line 124, in run_and_capture
      regular_output=regular_output)
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py", line 115, in run
      unexpected_result_count = manager.run(args)
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py", line 725, in run
      interrupted, keyboard_interrupted, thread_timings, test_timings, individual_test_timings = self._run_tests(self._test_names, result_summary, int(self._options.child_processes))
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py", line 638, in _run_tests
      pool.run(('test_list', shard.name, shard.test_inputs) for shard in all_shards)
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/common/message_pool.py", line 97, in run
      self.wait()
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/common/message_pool.py", line 130, in wait
      self._loop(block=True)
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/common/message_pool.py", line 182, in _loop
      method(message.src, *message.args)
    File "/mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/Tools/Scripts/webkitpy/common/message_pool.py", line 161, in _handle_worker_exception
      raise WorkerException(str(exception_value))
  WorkerException: [Errno 2] No such file or directory: '/tmp/layout-test-results/tests_run1.txt'
  
Ran 1552 tests in 6.899s
FAILED (failures=0, errors=1)
Comment 11 Thiago Marcos P. Santos 2012-08-01 03:14:26 PDT
Fix pending review at bug 92845.