diff --git a/awx/main/models/unified_jobs.py b/awx/main/models/unified_jobs.py index c2a74adfc8..16c043a991 100644 --- a/awx/main/models/unified_jobs.py +++ b/awx/main/models/unified_jobs.py @@ -432,7 +432,7 @@ class UnifiedJobTemplate(PolymorphicModel, CommonModelNameNotUnique, Notificatio copy_m2m_relationships(self, unified_jt, fields) return unified_jt - def _accept_or_ignore_job_kwargs(self, _exclude_errors=None, **kwargs): + def _accept_or_ignore_job_kwargs(self, _exclude_errors=(), **kwargs): ''' Override in subclass if template accepts _any_ prompted params ''' diff --git a/awx/main/models/workflow.py b/awx/main/models/workflow.py index d47745e1b8..831f60c21b 100644 --- a/awx/main/models/workflow.py +++ b/awx/main/models/workflow.py @@ -353,7 +353,7 @@ class WorkflowJobTemplate(UnifiedJobTemplate, WorkflowJobOptions, SurveyJobTempl workflow_job.copy_nodes_from_original(original=self) return workflow_job - def _accept_or_ignore_job_kwargs(self, **kwargs): + def _accept_or_ignore_job_kwargs(self, _exclude_errors=(), **kwargs): prompted_fields = {} rejected_fields = {} accepted_vars, rejected_vars, errors_dict = self.accept_or_ignore_variables(kwargs.get('extra_vars', {})) diff --git a/awx/main/tests/functional/api/test_schedules.py b/awx/main/tests/functional/api/test_schedules.py index c6cfc5d91a..bff29f8b07 100644 --- a/awx/main/tests/functional/api/test_schedules.py +++ b/awx/main/tests/functional/api/test_schedules.py @@ -27,6 +27,12 @@ def test_non_job_extra_vars_prohibited(post, project, admin_user): assert 'not allowed on launch' in str(r.data['extra_data'][0]) +@pytest.mark.django_db +def test_wfjt_schedule_accepted(post, workflow_job_template, admin_user): + url = reverse('api:workflow_job_template_schedules_list', kwargs={'pk': workflow_job_template.id}) + post(url, {'name': 'test sch', 'rrule': RRULE_EXAMPLE}, admin_user, expect=201) + + @pytest.mark.django_db def test_valid_survey_answer(post, admin_user, project, inventory, survey_spec_factory): job_template = JobTemplate.objects.create(