first

I wrote rspecs for mailer_spec.rb
I appreciate that you took the time to write specs for this file but… Is this in scope of your current story? As tempting as it may be to do more, try to keep within scope of the current task.

That said, since it is already written, I might as well give some feedback :grin: This block isn’t a real expectation, it will not fail if you use `user.persisted? == true`! The test description could be more helpful, what is `it` expected to do? What is `it` in this case? When you read it out as a sentence, “it persisted?” does not make much sense. This is where `describe` or `context` comes in handy. The tests in this file are not very well written, so don’t feel you have to follow the current example. I find the guidelines [here](http://betterspecs.org/) useful to writing tests.