Bug 90579

Summary:

[Tools] webkit_unittest.py got assertion

Product:

WebKit

Reporter:

yosin

Component:

Tools / Tests

Assignee:

yosin

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, dpranke, morrita, ojan, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch 1	none

yosin

Reported 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

Attachments
Patch 1 (2.07 KB, patch) 2012-07-04 19:25 PDT, yosin	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

yosin

Comment 1 2012-07-04 19:25:10 PDT

Created attachment 150859 [details] Patch 1

yosin

Comment 2 2012-07-04 19:25:34 PDT

Comment on attachment 150859 [details] Patch 1 Could you review this patch? Thanks in advance.

WebKit Review Bot

Comment 3 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>

WebKit Review Bot

Comment 4 2012-07-04 20:48:27 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.