Clean Up Your Project with Modules - Quality Assurance / Advanced Node and Express

server.js

 'use strict';
require('dotenv').config();
const express = require('express');
const myDB = require('./connection');
const fccTesting = require('./freeCodeCamp/fcctesting.js');
const session = require('express-session');
const passport = require('passport');
const routes = require('./routes.js');
const auth = require('./auth.js');

const app = express();

const rv = Math.random();
process.env.SESSION_SECRET = rv;

app.use('/public', express.static(process.cwd() + '/public'));
app.use(express.json());
app.use(express.urlencoded({ extended: true }));

app.use(session({
  secret: process.env.SESSION_SECRET,
  resave: true,
  saveUninitialized: true,
  cookie: { secure: false } // Change to true in production with HTTPS
}));

app.use(passport.initialize());
app.use(passport.session());

app.set('view engine', 'pug');
app.set('views', './views/pug');

myDB(async client => {
  const myDataBase = await client.db('advancednode').collection('node');

  routes(app, myDataBase);
  auth(app, myDataBase);

  fccTesting(app); // For FCC testing purposes

}).catch(e => {
  app.route('/').get((req, res) => {
    res.render('index', { title: e.message, message: 'Unable to connect to database' });
  });
});

const PORT = process.env.PORT || 3000;
app.listen(PORT, () => {
  console.log('Listening on port ' + PORT);
});

auth.js

const passport = require('passport');
const LocalStrategy = require('passport-local');
const bcrypt = require('bcrypt');
const { ObjectID } = require('mongodb');

module.exports = function (app, myDataBase) {
    passport.serializeUser((user, done) => {
        done(null, user._id);
      });
    
    passport.deserializeUser((id, done) => {
        myDataBase.findOne({ _id: new ObjectID(id) }, (err, doc) => {
            if (err) {
                return done(err);
            }
            done(null, doc);
        });
    });

    passport.use(new LocalStrategy((username, password, done) => {
        myDataBase.findOne({ username: username }, (err, user) => {
          console.log(`User ${username} attempted to log in.`);
          if (err) {
            return done(err);
          }
          if (!user) {
            return done(null, false);
          }
          if (!bcrypt.compareSync(password, user.password)) {
            return done(null, false);
          }
          return done(null, user);
        });
      }));
}

routes.js

const passport = require('passport');
const bcrypt = require('bcrypt');

module.exports = function (app, myDataBase) {
    app.route('/').get((req, res) => {
        res.render('index', {
          title: 'Connected to Database',
          message: 'Please login',
          showLogin: true,
          showRegistration: true
        });
      });

    app.route('/login').post(passport.authenticate('local', { failureRedirect: '/' }), (req, res) => {
        res.redirect('/profile');
      });

    app.route('/profile').get(ensureAuthenticated, (req, res) => {
        res.render('profile', { username: req.user.username });
      });

    app.route('/logout').get((req, res, next) => {
        req.logout((err) => {
          if (err) { return next(err); }
          res.redirect('/');
        });
      });

    app.route('/register').post((req, res, next) => {
        const hash = bcrypt.hashSync(req.body.password, 12);
        myDataBase.findOne({ username: req.body.username }, (err, user) => {
          if (err) {
            return next(err);
          }
          if (user) {
            return res.redirect('/');
          } else {
            myDataBase.insertOne({
              username: req.body.username,
              password: hash
            },
            (err, doc) => {
              if (err) {
                return res.redirect('/');
              }
              next(null, doc.ops[0]);
            });
          }
        });
      },
        passport.authenticate('local', { failureRedirect: '/' }),
        (req, res) => {
          res.redirect('/profile');
        }
      );

    app.use((req, res) => {
        res.status(404)
          .type('text')
          .send('Not Found');
      });
}

function ensureAuthenticated(req, res, next) {
    if (req.isAuthenticated()) {
      return next();
    }
    res.redirect('/');
}

I get these his error : // running tests
Modules should be present.
// tests completed
// console output
[TypeError: Failed to fetch]

Challenge: Clean Up Your Project with Modules

Link to the challenge:

I should also add the server running when i try to register a new user !

Never Mind, I just used FCC’s solutions and corrected the deprecated lines ! the tests pass <3