File "./Tools/Scripts/test-webkitpy", line 64, in <module> sys.exit(not tester.run()) File "/mnt/git/webkit-sheriff-bot/Tools/Scripts/webkitpy/test/main.py", line 138, in run return self._run_tests(names) File "/mnt/git/webkit-sheriff-bot/Tools/Scripts/webkitpy/test/main.py", line 169, in _run_tests suites.append(loader.loadTestsFromName(name, None)) File "/usr/lib/python2.6/unittest.py", line 587, in loadTestsFromName return self.loadTestsFromModule(obj) File "/usr/lib/python2.6/unittest.py", line 559, in loadTestsFromModule tests.append(self.loadTestsFromTestCase(obj)) File "/usr/lib/python2.6/unittest.py", line 550, in loadTestsFromTestCase return self.suiteClass(map(testCaseClass, testCaseNames)) File "/mnt/git/webkit-sheriff-bot/Tools/Scripts/webkitpy/layout_tests/port/http_lock_unittest.py", line 44, in __init__ self.clean_all_lockfile() File "/mnt/git/webkit-sheriff-bot/Tools/Scripts/webkitpy/layout_tests/port/http_lock_unittest.py", line 52, in clean_all_lockfile self.filesystem.unlink(file_name) AttributeError: 'FileSystem' object has no attribute 'unlink' Unclear when it started.
Created attachment 138400 [details] Patch
Looks like that code triggers only in error cases; it's probably been broken for a long time. This is a quick fix, but all of this test code should probably be rewritten to use a MockFileSystem (there's already a FIXME for that in the code; I'll file a separate bug).
Comment on attachment 138400 [details] Patch LGTM. Thanks.
Committed r114933: <http://trac.webkit.org/changeset/114933>
bug 84617 filed for that.