Skip to content

Commit 8aeafcd

Browse files
cfsmp3claude
andcommitted
test: Add verify_artifacts_exist mock to webhook tests
Tests that exercise the successful workflow completion path now mock verify_artifacts_exist to return {'linux': True, 'windows': True}. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 4ec417a commit 8aeafcd

File tree

1 file changed

+14
-5
lines changed

1 file changed

+14
-5
lines changed

tests/test_ci/test_controllers.py

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -877,10 +877,12 @@ def test_webhook_workflow_run_requested_valid_workflow_name(self, mock_requests,
877877
self.assertEqual(response.data, b'{"msg": "EOL"}')
878878
mock_schedule_test.assert_called_once()
879879

880+
@mock.patch('mod_ci.controllers.verify_artifacts_exist', return_value={'linux': True, 'windows': True})
880881
@mock.patch('github.Github.get_repo')
881882
@mock.patch('mod_ci.controllers.queue_test')
882883
@mock.patch('requests.get', side_effect=mock_api_request_github)
883-
def test_webhook_workflow_run_completed_successful_linux(self, mock_request, mock_queue_test, mock_repo):
884+
def test_webhook_workflow_run_completed_successful_linux(self, mock_request, mock_queue_test, mock_repo,
885+
mock_verify):
884886
"""Test webhook triggered with workflow run event with action completed and status success on linux."""
885887
data = {'action': 'completed',
886888
'workflow_run': {'event': 'push',
@@ -911,10 +913,12 @@ def test_webhook_workflow_run_completed_successful_linux(self, mock_request, moc
911913
data=json.dumps(data), headers=self.generate_header(data, 'workflow_run'))
912914
mock_queue_test.assert_called_once()
913915

916+
@mock.patch('mod_ci.controllers.verify_artifacts_exist', return_value={'linux': True, 'windows': True})
914917
@mock.patch('github.Github.get_repo')
915918
@mock.patch('mod_ci.controllers.queue_test')
916919
@mock.patch('requests.get', side_effect=mock_api_request_github)
917-
def test_webhook_workflow_run_completed_successful_windows(self, mock_request, mock_queue_test, mock_repo):
920+
def test_webhook_workflow_run_completed_successful_windows(self, mock_request, mock_queue_test, mock_repo,
921+
mock_verify):
918922
"""Test webhook triggered with workflow run event with action completed and status success on windows."""
919923
data = {'action': 'completed',
920924
'workflow_run': {'event': 'push',
@@ -1053,12 +1057,13 @@ def test_webhook_with_invalid_ci_signature(self, mock_github, mock_warning):
10531057
data=json.dumps({}), headers=self.generate_header({}, 'workflow_run', "1"))
10541058
mock_warning.assert_called_once()
10551059

1060+
@mock.patch('mod_ci.controllers.verify_artifacts_exist', return_value={'linux': True, 'windows': True})
10561061
@mock.patch('github.Github.get_repo')
10571062
@mock.patch('mod_ci.controllers.BlockedUsers')
10581063
@mock.patch('mod_ci.controllers.queue_test')
10591064
@mock.patch('requests.get', side_effect=mock_api_request_github)
10601065
def test_webhook_workflow_run_completed_successful_pr_linux(self, mock_request, mock_queue_test,
1061-
mock_blocked, mock_repo):
1066+
mock_blocked, mock_repo, mock_verify):
10621067
"""Test webhook triggered - workflow run event, action completed, status success for pull request on linux."""
10631068
data = {'action': 'completed',
10641069
'workflow_run': {'event': 'pull_request',
@@ -1100,12 +1105,14 @@ def test_webhook_workflow_run_completed_successful_pr_linux(self, mock_request,
11001105
mock_queue_test.assert_not_called()
11011106
self.assertEqual(response.data, b'ERROR')
11021107

1108+
@mock.patch('mod_ci.controllers.verify_artifacts_exist', return_value={'linux': True, 'windows': True})
11031109
@mock.patch('github.Github.get_repo')
11041110
@mock.patch('mod_ci.controllers.BlockedUsers')
11051111
@mock.patch('mod_ci.controllers.queue_test')
11061112
@mock.patch('requests.get', side_effect=mock_api_request_github)
11071113
def test_webhook_workflow_run_completed_successful_pr_windows(self, mock_request,
1108-
mock_queue_test, mock_blocked, mock_repo):
1114+
mock_queue_test, mock_blocked, mock_repo,
1115+
mock_verify):
11091116
"""Test webhook triggered - workflow run event, action completed, status success for pull request on windows."""
11101117
data = {'action': 'completed',
11111118
'workflow_run': {'event': 'pull_request',
@@ -1148,13 +1155,15 @@ def test_webhook_workflow_run_completed_successful_pr_windows(self, mock_request
11481155
mock_queue_test.assert_not_called()
11491156
self.assertEqual(response.data, b'ERROR')
11501157

1158+
@mock.patch('mod_ci.controllers.verify_artifacts_exist', return_value={'linux': True, 'windows': True})
11511159
@mock.patch('github.Github.get_repo')
11521160
@mock.patch('mod_ci.controllers.deschedule_test')
11531161
@mock.patch('mod_ci.controllers.BlockedUsers')
11541162
@mock.patch('mod_ci.controllers.queue_test')
11551163
@mock.patch('requests.get', side_effect=mock_api_request_github)
11561164
def test_webhook_workflow_run_completed_successful_pr_updated(self, mock_request, mock_queue_test,
1157-
mock_blocked, mock_deschedule_test, mock_repo):
1165+
mock_blocked, mock_deschedule_test, mock_repo,
1166+
mock_verify):
11581167
"""Test webhook triggered - workflow run event, action completed, for a pull request whose head was updated."""
11591168
data = {'action': 'completed',
11601169
'workflow_run': {'event': 'pull_request',

0 commit comments

Comments
 (0)