diff --git a/src/appengine/handlers/jobs.py b/src/appengine/handlers/jobs.py index 0c70ac86c40..cdd957a532f 100644 --- a/src/appengine/handlers/jobs.py +++ b/src/appengine/handlers/jobs.py @@ -147,7 +147,8 @@ def post(self): 'Job name can only contain letters, numbers, dashes and underscores.', 400) - fuzzers = request.form.get('fuzzers', []).split(',') + fuzzers_string = request.form.get('fuzzers') + fuzzers = fuzzers_string.split(',') if fuzzers_string else [] templates = request.form.get('templates', '').splitlines() for template in templates: if not data_types.JobTemplate.query( diff --git a/src/clusterfuzz/_internal/tests/appengine/handlers/jobs_test.py b/src/clusterfuzz/_internal/tests/appengine/handlers/jobs_test.py index 5c4a15a0b5e..402339d5eab 100644 --- a/src/clusterfuzz/_internal/tests/appengine/handlers/jobs_test.py +++ b/src/clusterfuzz/_internal/tests/appengine/handlers/jobs_test.py @@ -222,3 +222,21 @@ def test_post(self): self.assertEqual(200, resp.status_int) self.mock.update_mappings_for_job.assert_called_with( mock.ANY, ['test_fuzzer']) + + def test_post_empty_fuzzers(self): + """Test post method with no fuzzers provided.""" + self.mock.has_access.return_value = True + job = self._create_job('test_job', 'PROJECT_NAME = proj\n') + + resp = self.app.post( + '/', { + 'csrf_token': form.generate_csrf_token(), + 'name': job.name, + 'desciption': job.description, + 'platform': job.platform, + 'fuzzers': '' + }, + expect_errors=True) + + self.assertEqual(200, resp.status_int) + self.mock.update_mappings_for_job.assert_called_with(mock.ANY, [])