[auth] Update providers to match web spec

lib/modules/auth/index.js

@@ -4,12 +4,12 @@ import ModuleBase from './../../utils/ModuleBase';
 import ConfirmationResult from './ConfirmationResult';
 
 // providers
-import EmailAuthProvider from './providers/Email';
-import GoogleAuthProvider from './providers/Google';
-import GithubAuthProvider from './providers/Github';
-import TwitterAuthProvider from './providers/Twitter';
-import FacebookAuthProvider from './providers/Facebook';
-import PhoneAuthProvider from './providers/Phone';
+import EmailAuthProvider from './providers/EmailAuthProvider';
+import GoogleAuthProvider from './providers/GoogleAuthProvider';
+import GithubAuthProvider from './providers/GithubAuthProvider';
+import TwitterAuthProvider from './providers/TwitterAuthProvider';
+import FacebookAuthProvider from './providers/FacebookAuthProvider';
+import PhoneAuthProvider from './providers/PhoneAuthProvider';
 
 export default class Auth extends ModuleBase {
   static _NAMESPACE = 'auth';
@@ -185,7 +185,7 @@ export default class Auth extends ModuleBase {
   signInWithCredential(credential: CredentialType): Promise<Object> {
     return this._interceptUserValue(
       this._native.signInWithCredential(
-        credential.provider, credential.token, credential.secret,
+        credential.providerId, credential.token, credential.secret,
       ),
     );
   }

lib/modules/auth/providers/Email.js

@@ -1,10 +0,0 @@
-export default {
-  credential(email, password) {
-    return {
-      token: email,
-      secret: password,
-      provider: 'password',
-      providerId: 'password',
-    };
-  },
-};

lib/modules/auth/providers/EmailAuthProvider.js

@@ -0,0 +1,19 @@
+const providerId = 'password';
+
+export default class EmailAuthProvider {
+  static get PROVIDER_ID() {
+    return providerId;
+  }
+
+  static credential(email, password) {
+    return {
+      token: email,
+      secret: password,
+      providerId,
+    };
+  }
+
+  get providerId() {
+    return providerId;
+  }
+};

lib/modules/auth/providers/Facebook.js

@@ -1,10 +0,0 @@
-export default {
-  credential(token) {
-    return {
-      token,
-      secret: '',
-      provider: 'facebook',
-      providerId: 'facebook',
-    };
-  },
-};

lib/modules/auth/providers/FacebookAuthProvider.js

@@ -0,0 +1,19 @@
+const providerId = 'facebook';
+
+export default class FacebookAuthProvider {
+  static get PROVIDER_ID() {
+    return providerId;
+  }
+
+  static credential(token) {
+    return {
+      token,
+      secret: '',
+      providerId,
+    };
+  }
+
+  get providerId() {
+    return providerId;
+  }
+};

lib/modules/auth/providers/Github.js

@@ -1,10 +0,0 @@
-export default {
-  credential(token) {
-    return {
-      token,
-      secret: '',
-      provider: 'github',
-      providerId: 'github',
-    };
-  },
-};

lib/modules/auth/providers/GithubAuthProvider.js

@@ -0,0 +1,19 @@
+const providerId = 'github';
+
+export default class GithubAuthProvider {
+  static get PROVIDER_ID() {
+    return providerId;
+  }
+
+  static credential(token) {
+    return {
+      token,
+      secret: '',
+      providerId,
+    };
+  }
+
+  get providerId() {
+    return providerId;
+  }
+};

lib/modules/auth/providers/Google.js

@@ -1,10 +0,0 @@
-export default {
-  credential(token, secret) {
-    return {
-      token,
-      secret,
-      provider: 'google',
-      providerId: 'google',
-    };
-  },
-};

lib/modules/auth/providers/GoogleAuthProvider.js

@@ -0,0 +1,19 @@
+const providerId = 'google';
+
+export default class GoogleAuthProvider {
+  static get PROVIDER_ID() {
+    return providerId;
+  }
+
+  static credential(token, secret) {
+    return {
+      token,
+      secret,
+      providerId,
+    };
+  }
+
+  get providerId() {
+    return providerId;
+  }
+};

lib/modules/auth/providers/Phone.js

@@ -1,10 +0,0 @@
-export default {
-  credential(verificationId, code) {
-    return {
-      token: verificationId,
-      secret: code,
-      provider: 'phone',
-      providerId: 'phone',
-    };
-  },
-};

lib/modules/auth/providers/PhoneAuthProvider.js

@@ -0,0 +1,19 @@
+const providerId = 'phone';
+
+export default class PhoneAuthProvider {
+  static get PROVIDER_ID() {
+    return providerId;
+  }
+
+  static credential(verificationId, code) {
+    return {
+      token: verificationId,
+      secret: code,
+      providerId,
+    };
+  }
+
+  get providerId() {
+    return providerId;
+  }
+};

lib/modules/auth/providers/Twitter.js

@@ -1,10 +0,0 @@
-export default {
-  credential(token, secret) {
-    return {
-      token,
-      secret,
-      provider: 'twitter',
-      providerId: 'twitter',
-    };
-  },
-};

lib/modules/auth/providers/TwitterAuthProvider.js

@@ -0,0 +1,19 @@
+const providerId = 'twitter';
+
+export default class TwitterAuthProvider {
+  static get PROVIDER_ID() {
+    return providerId;
+  }
+
+  static credential(token, secret) {
+    return {
+      token,
+      secret,
+      providerId,
+    };
+  }
+
+  get providerId() {
+    return providerId;
+  }
+};

lib/modules/auth/user.js

@@ -109,7 +109,7 @@ export default class User {
    * @param credential
    */
   linkWithCredential(credential: CredentialType) {
-    return this._auth._interceptUserValue(this._auth._native.link(credential.provider, credential.token, credential.secret));
+    return this._auth._interceptUserValue(this._auth._native.link(credential.providerId, credential.token, credential.secret));
   }
 
   /**
@@ -126,7 +126,7 @@ export default class User {
    * @return {Promise}         A promise resolved upon completion
    */
   reauthenticateWithCredential(credential: CredentialType) {
-    return this._auth._interceptUserValue(this._auth._native.reauthenticate(credential.provider, credential.token, credential.secret));
+    return this._auth._interceptUserValue(this._auth._native.reauthenticate(credential.providerId, credential.token, credential.secret));
   }
 
   /**