Bug 90894 – [Chromium-Android] Use setup_test_runner() instead of start_helper() to setup test environment

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Xianzhu Wang]

start_helper() is actually start_pixel_test_helper() since http://trac.webkit.org/changeset/115601 (bug 81729).
Should use setup_test_runner() to setup test environment for chromium-android.

[Xianzhu Wang]

Created attachment 151492 [details]
Patch

[WebKit Review Bot]

Comment on attachment 151492 [details]
Patch

Clearing flags on attachment: 151492

Committed r122251: <http://trac.webkit.org/changeset/122251>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Thiago Marcos P. Santos]

This patch broke EFL test runner:

Traceback (most recent call last):
  File "./Tools/Scripts/test-webkitpy", line 64, in <module>
    sys.exit(not tester.run())
  File "/home/buildslave-1/webkit-buildslave/efl-linux-64-debug/build/Tools/Scripts/webkitpy/test/main.py", line 146, in run
    return self._run_tests(names)
  File "/home/buildslave-1/webkit-buildslave/efl-linux-64-debug/build/Tools/Scripts/webkitpy/test/main.py", line 177, in _run_tests
    suites.append(loader.loadTestsFromName(name, None))
  File "/usr/lib/python2.7/unittest/loader.py", line 103, in loadTestsFromName
    return self.loadTestsFromModule(obj)
  File "/usr/lib/python2.7/unittest/loader.py", line 65, in loadTestsFromModule
    tests.append(self.loadTestsFromTestCase(obj))
  File "/usr/lib/python2.7/unittest/loader.py", line 56, in loadTestsFromTestCase
    loaded_suite = self.suiteClass(map(testCaseClass, testCaseNames))
  File "/home/buildslave-1/webkit-buildslave/efl-linux-64-debug/build/Tools/Scripts/webkitpy/layout_tests/controllers/test_expectations_editor_unittest.py", line 74, in __init__
    self.test_port = host.port_factory.get('test-win-xp', None)
  File "/home/buildslave-1/webkit-buildslave/efl-linux-64-debug/build/Tools/Scripts/webkitpy/layout_tests/port/factory.py", line 110, in get
    module = __import__(module_name, globals(), locals(), [], -1)
  File "/home/buildslave-1/webkit-buildslave/efl-linux-64-debug/build/Tools/Scripts/webkitpy/layout_tests/port/chromium_android.py", line 62, in <module>
    DRT_APP_CACHE_DIR = DIR_APP_DIR + 'cache/'
NameError: name 'DIR_APP_DIR' is not defined

[Xianzhu Wang]

Sorry for the break. The patch (bug 90904) has been in CQ.

[Thiago Marcos P. Santos]

(In reply to comment #5)
> Sorry for the break. The patch (bug 90904) has been in CQ.

No problem. Thanks for fixing.