diff --git a/readthedocs/projects/tasks.py b/readthedocs/projects/tasks.py index 0d22b88cea7..9ab59a08741 100644 --- a/readthedocs/projects/tasks.py +++ b/readthedocs/projects/tasks.py @@ -1917,7 +1917,7 @@ def send_build_status(build_pk, commit, status): for account in user_accounts: # Currently we only support GitHub Status API if account.provider_name == 'GitHub': - success = account.send_build_status(build, status) + success = account.send_build_status(build, commit, status) if success: return True diff --git a/readthedocs/rtd_tests/tests/test_celery.py b/readthedocs/rtd_tests/tests/test_celery.py index 63bb3f21958..7d42e56e6a4 100644 --- a/readthedocs/rtd_tests/tests/test_celery.py +++ b/readthedocs/rtd_tests/tests/test_celery.py @@ -362,9 +362,13 @@ def test_send_build_status_task_with_social_account(self, send_build_status): external_build = get( Build, project=self.project, version=external_version ) - tasks.send_build_status(external_build.id, BUILD_STATUS_SUCCESS) + tasks.send_build_status( + external_build.id, external_build.commit, BUILD_STATUS_SUCCESS + ) - send_build_status.assert_called_once_with(external_build, BUILD_STATUS_SUCCESS) + send_build_status.assert_called_once_with( + external_build, external_build.commit, BUILD_STATUS_SUCCESS + ) @patch('readthedocs.projects.tasks.GitHubService.send_build_status') def test_send_build_status_task_without_remote_repo_or_social_account(self, send_build_status):