Use rspec render_template for layout specification

Improve product spec, too many repeated spec and create(:product) calls

I've noticed this while working on other patches. I think there was too many repeated code on product spec. Also use ``render_template`` to test layout rendering should be better than hacking into the dummy application.html.erb file and then remove it later which was what the previous test did. Please let me know if this sounds wrong.