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   

Michael Catanzaro
Reported 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?
Attachments
Radar WebKit Bug Importer
Comment 1 2022-04-26 15:55:28 PDT
Jonathan Bedard
Comment 2 2022-04-26 15:58:42 PDT
EWS
Comment 3 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.
Note You need to log in before you can comment on or make changes to this bug.