Bug 236043 - REGRESSION(r288878) webkitpy.layout_tests.controllers.layout_test_finder_legacy_unittest.LayoutTestFinderTests tests are failing with Windows Python
Summary: REGRESSION(r288878) webkitpy.layout_tests.controllers.layout_test_finder_lega...
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: 236149
Blocks:
  Show dependency treegraph
 
Reported: 2022-02-02 15:39 PST by Fujii Hironori
Modified: 2022-02-06 20:43 PST (History)
7 users (show)

See Also:


Attachments
Patch (1.81 KB, patch)
2022-02-02 16:14 PST, Fujii Hironori
no flags Details | Formatted Diff | Diff
Patch (1.81 KB, patch)
2022-02-06 18:18 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 2022-02-02 15:39:06 PST
REGRESSION(r288878) webkitpy.layout_tests.controllers.layout_test_finder_legacy_unittest.LayoutTestFinderTests tests are failing with Windows Python

Since r288878 (Bug 235370)
WinCairo buildbot is using Windows Python.
https://build.webkit.org/#/builders/60/builds/4301


[496/1640] webkitpy.layout_tests.controllers.layout_test_finder_legacy_unittest.LayoutTestFinderTests.test_find_all_no_paths
[497/1640] webkitpy.layout_tests.controllers.layout_test_finder_legacy_unittest.LayoutTestFinderTests.test_find_all_no_paths passed
[497/1640] webkitpy.layout_tests.controllers.layout_test_finder_legacy_unittest.LayoutTestFinderTests.test_find_directory
[498/1640] webkitpy.layout_tests.controllers.layout_test_finder_legacy_unittest.LayoutTestFinderTests.test_find_directory failed:
  Traceback (most recent call last):
    File "C:\BW\wincairo-wkl-release-tests\build\Tools\Scripts\webkitpy\layout_tests\controllers\layout_test_finder_legacy_unittest.py", line 263, in test_find_directory
      self.assertEqual(tests, ['websocket/tests/passes/text.html'])
  AssertionError: Lists differ: ['\\test.checkout\\LayoutTests\\websocket\\tests\\passes\\text.html'] != ['websocket/tests/passes/text.html']
  
  First differing element 0:
  '\\test.checkout\\LayoutTests\\websocket\\tests\\passes\\text.html'
  'websocket/tests/passes/text.html'
  
  - ['\\test.checkout\\LayoutTests\\websocket\\tests\\passes\\text.html']
  + ['websocket/tests/passes/text.html']
  
[498/1640] webkitpy.layout_tests.controllers.layout_test_finder_legacy_unittest.LayoutTestFinderTests.test_find_directory_includes_platform_specific
[499/1640] webkitpy.layout_tests.controllers.layout_test_finder_legacy_unittest.LayoutTestFinderTests.test_find_directory_includes_platform_specific failed:
  Traceback (most recent call last):
    File "C:\BW\wincairo-wkl-release-tests\build\Tools\Scripts\webkitpy\layout_tests\controllers\layout_test_finder_legacy_unittest.py", line 173, in test_find_directory_includes_platform_specific
      self.assertEqual(
  AssertionError: Lists differ: ['\\test.checkout\\LayoutTests\\http\\tests[142 chars]tml'] != ['http/tests/passes/image.html', 'http/test[87 chars]tml']
  
  First differing element 0:
  '\\test.checkout\\LayoutTests\\http\\tests\\passes\\image.html'
  'http/tests/passes/image.html'
  
  Second list contains 1 additional elements.
  First extra element 3:
  'platform/test-mac-leopard/http/test.html'
  
  - ['\\test.checkout\\LayoutTests\\http\\tests\\passes\\image.html',
  -  '\\test.checkout\\LayoutTests\\http\\tests\\passes\\text.html',
  -  '\\test.checkout\\LayoutTests\\http\\tests\\ssl\\text.html']
  + ['http/tests/passes/image.html',
  +  'http/tests/passes/text.html',
  +  'http/tests/ssl/text.html',
  +  'platform/test-mac-leopard/http/test.html']
  
[499/1640] webkitpy.layout_tests.controllers.layout_test_finder_legacy_unittest.LayoutTestFinderTests.test_find_directory_multiple_times
Comment 1 Fujii Hironori 2022-02-02 16:14:53 PST
Created attachment 450715 [details]
Patch
Comment 2 Sam Sneddon [:gsnedders] 2022-02-03 10:24:39 PST
Ah, gah, sorry about this regression! I'd actually realised I was breaking the tests in this way and had worked around it locally (essentially by maintaining the existing semantics of always running the tests under an emulated UNIX-like filesystem), but clearly I uploaded an outdated patch. This is, however, undoubtedly better, given you're fixing the actual reason for the failure!
Comment 3 Fujii Hironori 2022-02-03 11:53:25 PST
Comment on attachment 450715 [details]
Patch

Thank you for the review.
Comment 4 EWS 2022-02-03 12:16:27 PST
Committed r289067 (246774@main): <https://commits.webkit.org/246774@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 450715 [details].
Comment 5 Radar WebKit Bug Importer 2022-02-03 12:17:18 PST
<rdar://problem/88449289>
Comment 6 WebKit Commit Bot 2022-02-04 11:44:22 PST
Re-opened since this is blocked by bug 236149
Comment 7 Fujii Hironori 2022-02-06 18:18:59 PST
Created attachment 451058 [details]
Patch
Comment 8 EWS 2022-02-06 20:43:12 PST
Committed r289197 (246883@main): <https://commits.webkit.org/246883@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 451058 [details].