Skip to content

Commit 1117caa

Browse files
committed
remove log test
1 parent edab9b3 commit 1117caa

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

readthedocs/rtd_tests/tests/test_celery.py

+1-6
Original file line numberDiff line numberDiff line change
@@ -347,17 +347,12 @@ def test_send_build_status_task(self, send_build_status):
347347

348348
send_build_status.assert_called_once_with(external_build, BUILD_STATUS_SUCCESS)
349349

350-
@patch('readthedocs.projects.tasks.log')
351350
@patch('readthedocs.projects.tasks.GitHubService.send_build_status')
352-
def test_send_build_status_task_without_remote_repo(self, send_build_status, mock_logger):
351+
def test_send_build_status_task_without_remote_repo(self, send_build_status):
353352
external_version = get(Version, project=self.project, type=EXTERNAL)
354353
external_build = get(
355354
Build, project=self.project, version=external_version
356355
)
357356
tasks.send_build_status(external_build, BUILD_STATUS_SUCCESS)
358357

359358
send_build_status.assert_not_called()
360-
mock_logger.info.assert_called_with(
361-
'Remote repository does not exist for %s',
362-
self.project,
363-
)

readthedocs/rtd_tests/tests/test_oauth.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -191,8 +191,8 @@ def test_send_build_status_value_error(self, session, mock_logger):
191191

192192
self.assertFalse(success)
193193
mock_logger.exception.assert_called_with(
194-
'GitHub commit status creation failed for project: %s',
195-
self.project,
194+
'GitHub commit status creation failed for project: %s',
195+
self.project,
196196
)
197197

198198
@override_settings(DEFAULT_PRIVACY_LEVEL='private')

0 commit comments

Comments
 (0)