refactor(client): FxaClient no longer takes a relier in its constructor.

@ckarlof - This is the FxaClient without the relier passed in to the constructor. The core of this update was pretty easy, updating the tests was not. Since so many fxaClient.signUp calls has to be updated in the unit tests, I replaced most calls to the auth-server with sinon stubs.

* Instead of creating the FxaClient with a relier in its constructor, pass a relier instance to the functions that need it.
* Since we had to update any unit test that used fxaClient.signUp to pass in a relier, it was easy to update the tests to no longer touch the server.

Note, no functional tests were modified with this!

This is not a full fix for the issues listed in #1777, but contains probably the most difficult part.

issue #1777 


It was easier to mock in responses to the channel notification if this error clause was here instead of below.