Skip to content

Updating presenter specs to include routing helper by default

Buck O'Leary requested to merge 499855-follow-up-enterprise-url-helpers-fix into master

What does this MR do and why?

  • Reduces boilerplate code in individual presenter specs by removing the need to explicitly include routing helpers
  • Ensures consistent inclusion of routing helpers across all presenter specs
  • Automatically set the correct spec type for presenter specs based on their file location

Co-authored by @dstull

MR acceptance checklist

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

Screenshots or screen recordings

N/A

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Checkout branch
  2. Run presenter specs

bin/rspec ee/spec/presenters/gitlab_subscriptions/trials/duo_enterprise_status_widget_presenter_spec.rb

or

bin/rspec ee/spec/presenters/gitlab_subscriptions/trials/duo_pro_status_widget_presenter_spec.rb

Related to #499855

Edited by Buck O'Leary

Merge request reports

Loading