Bug 204400 - run-api-tests reports AssertionError os.pathsep not in value in _append_value_colon_separated in Cygwin Python
Summary: run-api-tests reports AssertionError os.pathsep not in value in _append_value...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Fujii Hironori
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-11-20 00:00 PST by Fujii Hironori
Modified: 2019-11-20 15:22 PST (History)
8 users (show)

See Also:


Attachments
Patch (2.91 KB, patch)
2019-11-20 01:45 PST, Fujii Hironori
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fujii Hironori 2019-11-20 00:00:49 PST
[Win][Cygwin][run-api-tests] assert os.pathsep not in value in _append_value_colon_separated

AppleWin run-api-tests is failing.

https://build.webkit.org/builders/Apple%20Win%2010%20Release%20%28Tests%29/builds/4036

> AssertionError raised: 
> Traceback (most recent call last):
>   File "/home/buildbot/worker/win10-release-tests/build/Tools/Scripts/webkitpy/api_tests/run_api_tests.py", line 59, in main
>     return run(port, options, args, stderr)
>   File "/home/buildbot/worker/win10-release-tests/build/Tools/Scripts/webkitpy/api_tests/run_api_tests.py", line 77, in run
>     result = manager.run(args, json_output=options.json_output)
>   File "/home/buildbot/worker/win10-release-tests/build/Tools/Scripts/webkitpy/api_tests/manager.py", line 171, in run
>     test_names = self._collect_tests(args)
>   File "/home/buildbot/worker/win10-release-tests/build/Tools/Scripts/webkitpy/api_tests/manager.py", line 100, in _collect_tests
>     env=self._port.environment_for_api_tests())
>   File "/home/buildbot/worker/win10-release-tests/build/Tools/Scripts/webkitpy/port/win.py", line 141, in environment_for_api_tests
>     env = super(WinPort, self).environment_for_api_tests()
>   File "/home/buildbot/worker/win10-release-tests/build/Tools/Scripts/webkitpy/port/base.py", line 266, in environment_for_api_tests
>     self._append_value_colon_separated(environment, name, build_root_path)
>   File "/home/buildbot/worker/win10-release-tests/build/Tools/Scripts/webkitpy/port/base.py", line 942, in _append_value_colon_separated
>     assert os.pathsep not in value
> AssertionError
Comment 1 Fujii Hironori 2019-11-20 00:02:11 PST
r249500 (Bug 201311) seems the culprit.
Comment 2 Fujii Hironori 2019-11-20 01:17:43 PST
Port._build_path contains ':' because webkit-build-directory returns native paths.

$ perl webkit-build-directory
C:\webkit\gb\WebKitBuild
C:/webkit/gb/WebKitBuild/Release
Comment 3 Fujii Hironori 2019-11-20 01:45:00 PST
Created attachment 383951 [details]
Patch
Comment 4 WebKit Commit Bot 2019-11-20 15:21:17 PST
Comment on attachment 383951 [details]
Patch

Clearing flags on attachment: 383951

Committed r252714: <https://trac.webkit.org/changeset/252714>
Comment 5 WebKit Commit Bot 2019-11-20 15:21:19 PST
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2019-11-20 15:22:32 PST
<rdar://problem/57374957>