Clean up auth and user tests

Pass a `User` into the setup instead of a plain object.
Also, fix one typo in a test name.

Author: dmurvihill

test/unit/auth.js

@@ -23,7 +23,7 @@ describe('Auth', function () {
   describe('#changeAuthState', function () {
 
     it('sets the auth data', function () {
-      var user = {};
+      var user = new User(ref, {});
       ref.changeAuthState(user);
       ref.flush();
       expect(ref.getAuth()).to.equal(user);
@@ -266,18 +266,18 @@ describe('Auth', function () {
       var context = {};
       ref.onAuth(spy);
       ref.onAuth(spy, context);
-      ref.changeAuthState({
+      ref.changeAuthState(new User(ref, {
         uid: 'kato1'
-      });
+      }));
       ref.flush();
       expect(spy.callCount).to.equal(4);
       spy.reset();
       // will not match any context
       ref.offAuth(spy, {});
       ref.offAuth(spy, context);
-      ref.changeAuthState({
+      ref.changeAuthState(new User(ref, {
         uid: 'kato2'
-      });
+      }));
       ref.flush();
       expect(spy.callCount).to.equal(1);
     });
@@ -331,7 +331,7 @@ describe('Auth', function () {
       });
     });
 
-    it('is not be triggered if auth state does not change', function () {
+    it('is not triggered if auth state does not change', function () {
       ref.changeAuthState({
         uid: 'kato'
       });

test/unit/user.js

@@ -217,7 +217,7 @@ describe('User', function() {
 
   describe('#getIdToken', function() {
     it('should get token', function() {
-      var user = new User(auth, {});
+      const user = new User(auth, {});
       return expect(user.getIdToken()).to.eventually.not.be.empty;
     });