annaswims/GrinnellPlans — pull request #47 by aggarwal (2011-11-04 02:59:46+00:00)
Title: User registration

Hey, I finally got around to finishing the tests for the user registration. I am pleased with what I have; it is well tested.

I didn't think changing the behavior at all because of our feature freeze. We could work on making user registration more secure in the future. I really don't like sending out passwords in plain text.

I ended up using RAILS testing framework instead of rspec because (1) I wasn't motivated to learn rspec, and (2) the two frameworks can coexist.

Let me know if you want me to change something. Particularly, I settled on a style where I display stuff in a view depending on what flag was set in the controller. I probably should be using something like flash notices.