Remove AbstractEvent and AbstractVenue

Been wanting to do this for while. The `SourceParser` subclasses were creating an `AbstractEvent` and an `AbstractVenue`, but then immediately invoking complex methods to convert them to a normal `Event` and `Venue`. The abstract versions were transient, and never used outside of this process, and thus unnecessary. This refactoring removes all that complexity, and now `SourceParser` subclasses create `Event`s and `Venue`s directly.

This exposes more low-hanging refactoring fruit among the `SourceParser` subclasses, but I'll save that for later.
https://github.com/botandrose/calagator/blob/remove_abstract_event_and_venue/spec/models/source_parser_spec.rb#L56

Just speculating (haha puns), but this spec might be coming from a performance perspective (haha). Running all the parsers may be expensive, especially the ones that hit the network. So, personally, I think the spec makes sense. What do you think?