[Finishes #173176790] Finish modifying Social Authentication functionality to redirect to client

Author: devel605

server/controllers/authController.js

@@ -172,23 +172,7 @@ class AuthController {
         isVerified
       });
 
-      res.status(200).json({
-        status: 200,
-        message: `User successfully signed in with ${authType}`,
-        data: {
-          token,
-          user: {
-            username,
-            firstname,
-            lastname,
-            email,
-            role,
-            lineManager,
-            isVerified,
-            authType
-          }
-        }
-      });
+      res.redirect(`https://champs-frontend.herokuapp.com/home?${token}`);
     } catch (error) {
       return res.status(500).json({
         status: 500,

server/middleware/user.js

@@ -135,7 +135,7 @@ class User {
    */
   static async verifyGoogleSignIn(req, res, next) {
     try {
-      passport.authenticate('google', { failureRedirect: '/google', session: false }, (err, user) => {
+      passport.authenticate('google', { failureRedirect: 'https://champs-frontend.herokuapp.com/signin', session: false }, (err, user) => {
         if (err) { return next(err); }
         req.user = user;
         return next();
@@ -154,7 +154,7 @@ class User {
      */
   static async verifyFacebookSignIn(req, res, next) {
     try {
-      passport.authenticate('facebook', { failureRedirect: '/facebook', session: false }, (err, user) => {
+      passport.authenticate('facebook', { failureRedirect: 'https://champs-frontend.herokuapp.com/signin', session: false }, (err, user) => {
         if (err) { return next(err); }
         req.user = user;
         return next();

server/tests/oauth.test.js

@@ -62,12 +62,9 @@ describe('Google Social Sign In Test', () => {
   it('should successfully sign in the user with Google', (done) => {
     request(app).get('/api/v1/auth/google/return')
       .send(mockData[2])
-      .expect(200)
+      .expect(302)
       .end((err, res) => {
-        expect(res).to.have.status(200);
-        expect(res.body).to.be.a('object');
-        expect(res.body).to.have.property('status');
-        expect(res.body).to.have.property('message');
+        expect(res).to.have.status(302);
         done();
       });
   });
@@ -87,12 +84,9 @@ describe('Facebook Social Sign In Test', () => {
   it('should successfully sign in the user with Facebook', (done) => {
     request(app).get('/api/v1/auth/facebook/return')
       .send(mockData[2])
-      .expect(200)
+      .expect(302)
       .end((err, res) => {
-        expect(res).to.have.status(200);
-        expect(res.body).to.be.a('object');
-        expect(res.body).to.have.property('status');
-        expect(res.body).to.have.property('message');
+        expect(res).to.have.status(302);
         done();
       });
   });