Also fixes the spec so that it actually tests the thing it says it's
testing. Hooray!

Closes #1308

Forgot they were so big!

Should now be much clearer about what each spec is actually testing.
For example, instead of testing stuff like link classes and titles in
every single call, we only test those things once, in their own specs.

Didn't bother with files in db/, config/, or features/

tests references to
* commits
* team members
* issues
* merge requests
* snipptes