Skip to content

DRY up admin, supervisor and volunteer spec using shared_examples#6877

Merged
compwron merged 1 commit intorubyforgood:mainfrom
alanparmenter:clean-refactor
Apr 24, 2026
Merged

DRY up admin, supervisor and volunteer spec using shared_examples#6877
compwron merged 1 commit intorubyforgood:mainfrom
alanparmenter:clean-refactor

Conversation

@alanparmenter
Copy link
Copy Markdown
Contributor

@alanparmenter alanparmenter commented Apr 23, 2026

What github issue is this PR for, if any?

Related to #6874

What changed, and why?

Using shared_examples to remove duplication in admin, supervisor and volunteer tests.

How is this tested? (please write rspec and jest tests!) 💖💪

bundle exec rspec spec/mailers/previews/devise_mailer_preview_spec.rb

Feelings gif (optional)

Cactus with umbrella in the rain

DRY up admin, supervisor and volunteer specs using shared_examples
@github-actions github-actions Bot added ruby Pull requests that update Ruby code Tests! 🎉💖👏 labels Apr 23, 2026
@alanparmenter alanparmenter changed the title Update devise_mailer_preview_spec.rb DRY up admin, supervisor and volunteer specs using shared_examples Apr 23, 2026
@alanparmenter alanparmenter changed the title DRY up admin, supervisor and volunteer specs using shared_examples DRY up admin, supervisor and volunteer spec using shared_examples Apr 23, 2026
Copy link
Copy Markdown
Collaborator

@compwron compwron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice :)

@compwron compwron merged commit 2b1b9af into rubyforgood:main Apr 24, 2026
11 checks passed
@alanparmenter alanparmenter deleted the clean-refactor branch April 24, 2026 12:10
alanparmenter added a commit to alanparmenter/casa that referenced this pull request Apr 25, 2026
alanparmenter added a commit to alanparmenter/casa that referenced this pull request Apr 25, 2026
…byforgood#6877)

Update devise_mailer_preview_spec.rb

DRY up admin, supervisor and volunteer specs using shared_examples
alanparmenter added a commit to alanparmenter/casa that referenced this pull request Apr 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ruby Pull requests that update Ruby code Tests! 🎉💖👏

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants