Refactored application_helper#event_link_name and added specs.

Since event#type was really used for "data-test" fields, it's better to keep it out of the model and just be explicit in its use when invoking the partials. By doing that, we can remove the event#type method for now.
Code arrangement. Nothing to see here.
Dry some scopes.