Bug 239788

Summary: test-webkitpy TestInstallGitLFS fails
Product: WebKit Reporter: Michael Catanzaro <mcatanzaro>
Component: Tools / TestsAssignee: Jonathan Bedard <jbedard>
Status: RESOLVED FIXED    
Severity: Normal CC: jbedard, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: PC   
OS: Linux   

Description Michael Catanzaro 2022-04-26 13:53:05 PDT
I see two failures when running test-webkitpy:

[1571/2000] webkitscmpy.test.install_git_lfs_unittest.TestInstallGitLFS.test_no_repo failed:              
  Traceback (most recent call last):
    File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/install_git_lfs_unittest.py", line 141, in test_no_repo
      self.assertEqual(0, program.main(
  AssertionError: 0 != 1
  
[1841/2000] webkitscmpy.test.install_git_lfs_unittest.TestInstallGitLFS.test_install failed:
  Traceback (most recent call last):
    File "/home/mcatanzaro/Projects/WebKit/Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/install_git_lfs_unittest.py", line 83, in test_install
      self.assertEqual(0, program.main(
  AssertionError: 0 != 1

It's pretty suspicious:

    def test_install(self):
        with OutputCapture() as captured, mocks.remote.GitHub('github.com/git-lfs/git-lfs', releases={
            'v3.1.2/git-lfs-darwin-arm64-v3.1.2.zip': wkmocks.Response(status_code=200, content=self.ZIP_CONTENTS),
            'v3.1.2/git-lfs-darwin-amd64-v3.1.2.zip': wkmocks.Response(status_code=200, content=self.ZIP_CONTENTS),
        }), mocks.local.Git(self.path), mocks.local.Svn():
            self.assertIsNone(local.Git(self.path).config().get('lfs.repositoryformatversion'))
            self.assertEqual(0, program.main(
                args=('install-git-lfs',),
                path=self.path,
            ))
            self.assertEqual(local.Git(self.path).config()['lfs.repositoryformatversion'], '0')

Looks like it's trying to install macOS binaries?
Comment 1 Radar WebKit Bug Importer 2022-04-26 15:55:28 PDT
<rdar://problem/92363359>
Comment 2 Jonathan Bedard 2022-04-26 15:58:42 PDT
Pull request: https://github.com/WebKit/WebKit/pull/404
Comment 3 EWS 2022-04-27 10:16:41 PDT
Committed r293515 (250046@main): <https://commits.webkit.org/250046@main>

Reviewed commits have been landed. Closing PR #404 and removing active labels.