Bug 90579

Summary: [Tools] webkit_unittest.py got assertion
Product: WebKit Reporter: yosin
Component: Tools / TestsAssignee: yosin
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dpranke, morrita, ojan, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch 1 none

Description yosin 2012-07-04 19:14:57 PDT
FAILURE: webkitpy.layout_tests.port.webkit_unittest.WebKitPortTest.test_skipped_directories_for_symbols
--------------------------------------------------------------------------------
Traceback (most recent call last):
  File "/mnt/data/b/WebKit-BuildSlave/chromium-linux-release-tests/build/Tools/Scripts/webkitpy/layout_tests/port/webkit_unittest.py", line 118, in test_skipped_directories_for_symbols
    self.assertEqual(result_directories, expected_directories)
AssertionError: set(['mathml', 'transforms/3d', 'inspector/styles/variables', 'compositing/webgl', 'fast/canvas/webgl', 'animations/3d', 'mhtml', 'http/tests/canvas/webgl', 'fast/css/variables']) != set(['mathml', 'transforms/3d', 'compositing/webgl', 'fast/canvas/webgl', 'animations/3d', 'mhtml', 'http/tests/canvas/webgl', 'fast/css/variables'])


Following changes in r121874 causes this.
http://trac.webkit.org/changeset/121874/trunk/Tools/Scripts/webkitpy/layout_tests/port/webkit_unittest.py
Comment 1 yosin 2012-07-04 19:25:10 PDT
Created attachment 150859 [details]
Patch 1
Comment 2 yosin 2012-07-04 19:25:34 PDT
Comment on attachment 150859 [details]
Patch 1

Could you review this patch?
Thanks in advance.
Comment 3 WebKit Review Bot 2012-07-04 20:48:22 PDT
Comment on attachment 150859 [details]
Patch 1

Clearing flags on attachment: 150859

Committed r121878: <http://trac.webkit.org/changeset/121878>
Comment 4 WebKit Review Bot 2012-07-04 20:48:27 PDT
All reviewed patches have been landed.  Closing bug.