Added snapshot tests for welcome emails#26965
Conversation
closes https://linear.app/ghost/issue/NY-1178 towards https://linear.app/ghost/issue/NY-1163 This was written almost entirely by Claude Opus 4.6 against the following prompt: > I want to create a snapshot test for member welcome emails. We > currently have _some_ tests for this, like in > `ghost/core/test/e2e-api/admin/automated-emails.test.js` and > `ghost/core/test/integration/services/member-welcome-emails.test.js`, > but these don't assert on the final rendered output. > > Build a (test-only) change that renders the full HTML of a member > welcome email, and asserts on the snapshot. We have > `assertMatchSnapshot` for this purpose, which I reckon you should use. I reviewed the code thoroughly and made a couple of small tweaks.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
WalkthroughA new Jest integration snapshot test suite was added for 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
cmraible
left a comment
There was a problem hiding this comment.
LGTM, thanks for covering this!
closes https://linear.app/ghost/issue/NY-1178
towards https://linear.app/ghost/issue/NY-1163
This was written almost entirely by Claude Opus 4.6 against the following prompt:
I reviewed the code thoroughly and made a couple of small tweaks.