Skip to content

Replace the dummy theme spec with overlay and journey tests - #41

Open
MustafaGood wants to merge 1 commit into
new-design-systemfrom
feat/theme-specs
Open

MustafaGood wants to merge 1 commit into
new-design-systemfrom
feat/theme-specs

Conversation

@MustafaGood

@MustafaGood MustafaGood commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replaces the dummy testing the truth spec with a real theme suite: Stripe and reCAPTCHA overlay contracts, plus controller examples for frontpage, sign-in and /body.
  • Specs load handlingar-theme via ALAVETELI_TEST_THEME (same hook as other Alaveteli themes). Run from the Alaveteli app: docker compose exec -e RAILS_ENV=test app bundle exec rspec lib/themes/handlingar-theme/spec.
  • GitHub Actions runs ruby script/check-overlay-contracts.rb so PRs catch Stripe/captcha/chrome regressions without booting Rails.

Test plan

  • CI job overlay-contracts is green on this PR
  • ruby script/check-overlay-contracts.rb
  • docker compose exec -e RAILS_ENV=test app bundle exec rspec lib/themes/handlingar-theme/spec
  • Stripe / captcha markup still present in the overlays

Do not merge to main.

@MustafaGood MustafaGood moved this to 👀 In review in Handlingar Theme Aug 25, 2026
@MustafaGood MustafaGood moved this from In review to Done in Handlingar Theme Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant