By convention, should the following be defined as an instance method of my model or a helper method?
# app/models/user.rb class User < ActiveRecord::Base
Everything directly related to your model should stay in your model.
This way, you keep coherent logic and tests.