The method lstrip is incorrectly used to get the name of failed tests from the full path.
Created attachment 135058 [details] Patch
Comment on attachment 135058 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=135058&action=review > Tools/Scripts/run-gtk-tests:268 > + names = [test.replace(self._programs_path, '', 1) for test in failed_tests] What about os.path.basename(test) ?
(In reply to comment #2) > (From update of attachment 135058 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=135058&action=review > > > Tools/Scripts/run-gtk-tests:268 > > + names = [test.replace(self._programs_path, '', 1) for test in failed_tests] > > What about os.path.basename(test) ? that would give you the name, but we also want the directory unittests/WebKit2APITests
(In reply to comment #3) > (In reply to comment #2) > > (From update of attachment 135058 [details] [details]) > > View in context: https://bugs.webkit.org/attachment.cgi?id=135058&action=review > > > > > Tools/Scripts/run-gtk-tests:268 > > > + names = [test.replace(self._programs_path, '', 1) for test in failed_tests] > > > > What about os.path.basename(test) ? > > that would give you the name, but we also want the directory unittests/WebKit2APITests Is it really useful? The test naming conventions differ between WK1 and WK2
(In reply to comment #4) > (In reply to comment #3) > > (In reply to comment #2) > > > (From update of attachment 135058 [details] [details] [details]) > > > View in context: https://bugs.webkit.org/attachment.cgi?id=135058&action=review > > > > > > > Tools/Scripts/run-gtk-tests:268 > > > > + names = [test.replace(self._programs_path, '', 1) for test in failed_tests] > > > > > > What about os.path.basename(test) ? > > > > that would give you the name, but we also want the directory unittests/WebKit2APITests > > Is it really useful? > The test naming conventions differ between WK1 and WK2 That's how we refer to tests, see SkippedTest class
Committed r112853: <http://trac.webkit.org/changeset/112853>