[wip] Adds specs around existing functionality

[chalkos]

• fix existing specs
• add VCR
• model specs (to test creations, relations and accessible info)
• session controller specs
• reviewership specs

github controller specs:

• handle ping event
• do not process requests with invalid signatures
• handle issue comment: patronus: :+1:, test, retry, :-1:, fork
• process issue comment: only on creation, only for collaborators, only for comments starting with patronus:
• handle status: combined/patronus success/failure/pending
• process status: ignore commits not made by patronus
• interact using patronus account when it has permissions, otherwise use user's account
• merging a PR creates N new pull requests, porting the changes to N port-branches

Note: rebase this on master after merging #40

[chalkos]

@indirect @segiddins do you have any comments on spec/github_web_hook_request.rb ?

It is a PORO that I'm using to build the mocked web hook request info and (optionally) add it to an existing HTTP request.

Github web hooks are json, so in github_web_hook_request.rb there are methods that create an hash and use parameters to fill in the blanks (like username, repository name, etc). The hash is then converted to json and sent in the request body.

The current methods work for ping (which is mostly just setting the headers right). There are methods to create the status web hook event json, but those are not being used yet.

The next step would be to mock status web hook requests, saving patronus' interactions with github in a VCR cassette, and removing the tokens (and repeating these steps for each test: pending status, success status, failure status, forced failure, etc).

Does this seems like a step in the right direction? Or am I overcomplicating something?