WebDriver: run-webdriver-tests doesn't handle test names with a '/' correctly

[Carlos Garcia Campos]

We use os.path.basename() to get the subtest name. If the test name contains a slash we get part of the subtest name as the name. For example:

find_element.py::test_xhtml_namespace[xpath-//*[name()='a']] -> *[name()='a']]

[Carlos Garcia Campos]

Created attachment 330892 [details]
Patch

[Charlie Turner]

Comment on attachment 330892 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=330892&action=review

> Tools/Scripts/webkitpy/webdriver_tests/webdriver_test_runner_w3c.py:82
> +    def _subtest_name(self, subtest):

Looks like this could be a free function, self isn't referenced in the body.

[Carlos Alberto Lopez Perez]

Comment on attachment 330892 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=330892&action=review

>> Tools/Scripts/webkitpy/webdriver_tests/webdriver_test_runner_w3c.py:82
>> +    def _subtest_name(self, subtest):
> 
> Looks like this could be a free function, self isn't referenced in the body.

I think its ok to have it as part of the object if it is related to it.

[Carlos Garcia Campos]

Committed r226711: <https://trac.webkit.org/changeset/226711>

[Radar WebKit Bug Importer]

<rdar://problem/36405305>