Update days property to an array

Alexander Turinskerequested to merge
526746-fix-weekly-cadence-selection into master

What does this MR do and why?

Update days property to an array

  • the schema says the array of days for the weekly cadence is capitalized
  • update schedule form for invalid cadence
  • update tests

EE: true

References

Screenshots or screen recordings

Invalid cadence Adding a weekly cadence
Screenshot_2025-04-17_at_21.10.38 Screenshot_2025-04-17_at_21.10.04

How to set up and validate locally

  1. Upload a GitLab Ultimate license
  2. Enable the feature flag via terminal or http://gdk.test:3443/rails/features 
echo "Feature.enable(:scheduled_pipeline_execution_policies)" | rails c
  1. Navigate to a project => Secure => Policies => New policy => Pipeline execution policy => Select "schedule a new"
  2. Navigate to yaml mode and change type to something invalid and navigate back to rule mode
  3. Verify the cadence dropdown shows the placeholder
  4. Change the cadence to weekly
  5. Verify the yaml is updated to include days: - Monday
  6. Verify the policy can be saved

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #526746 (closed)

Merge request reports