Merge pull request #6022 from stsewd/fix-missin-arg-tests

Fix missing arg in tests

Author: stsewd

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
 

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):