Skip to content

Add a smoke test for all email previews

The following discussion from !97039 (merged) should be addressed:

  • @splattael started a discussion: (+1 comment)

    Question This reminds me of #368096 (closed) where we also exclude code from having test coverage.

    I think any code should be tested - development, production, or even test code 😉

    What are your thoughts of adding a "smoke test" for all previewed mails? 🤔

    Just a plain specs which iterates over exposed (public) preview method, calls it, and ensures it returns a "message" and doesn't raise.

    We can follow-up on adding this spec and I'll take care of it.

    Suggestion In the meanwhile, can you:

    🙏