Bug 51548 - check-webkit-style doesn't check all of the ports when linting test expectations
Summary: check-webkit-style doesn't check all of the ports when linting test expectations
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords: NRWT
Depends on: 51222
Blocks:
  Show dependency treegraph
 
Reported: 2010-12-23 11:33 PST by Dirk Pranke
Modified: 2012-09-27 12:34 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Pranke 2010-12-23 11:33:31 PST
The code in the TestExpectations style checker only lints the test expectations file against the port it is running on. It really should be linting against all ports, like new-run-webkit-tests --lint-test-files does.

The easiest fix is to just have it call into the lint routines in webkitpy/layout_tests ... I should probably refactor that logic out of TestRunner and into test_expectations.
Comment 1 Dirk Pranke 2010-12-23 11:34:21 PST
There's no point in fixing this before we fix/land 51222, since all of the logic changes in that fix.
Comment 2 Dirk Pranke 2010-12-23 11:42:10 PST
Bug 51500 gave a false positive on the style bot because of this (as an example).
Comment 3 Dirk Pranke 2010-12-23 11:43:53 PST
There's also a lot of testing duplicated between the style checker and the unit tests for the expectations parser that's kind of unnecessary and can lead to confusion (see, e.g. bug 48426 for an example where one set of tests was updated but the other wasn't).
Comment 4 Dirk Pranke 2012-09-27 12:34:48 PDT
I'm pretty sure I fixed this a while ago. It seems to e be working now.