should this be a `destroy!` so [RecordNotDestroyed](file:///Users/eprothro/Library/Application%20Support/Dash/DocSets/Ruby_on_Rails_4/Ruby%20on%20Rails.docset/Contents/Resources/Documents/api.rubyonrails.org/classes/ActiveRecord/RecordNotDestroyed.html) would be raised on an error destroying?