GroupController.js

GroupController.addStudent = (id, student_id) => {
    return Group.findById(id)
        .then((g) => new Promise((resolve, reject) => {
            if(g){
                resolve(g);
            }else{
                reject('??');
            }
        }));
};

I would write ir like:

GroupController.addStudent = (id, student_id) => {
  return Group.findById(id).then((g) => new Promise((resolve, reject) => {
    if (g) {
      resolve(g);
    } else {
      reject('??');
    }
  }));
}

So consumers of controller may call as GroupController.addStudent(params).then(....)

Also take a 👀 at this promises examples

Pode encadear as promises através do then():

GroupController.addStudent = (id, student_id) => {
    return Group.findById(id)
        .then(g => {
            if (!g) {
                throw new Error('??');
            }

            return Model.create(...);
        });
};

Ou melhor ainda, faça findById rejeitar a promise se não encontrar o objeto com a id:

GroupController.addStudent = (id, student_id) => {
    return Group.findById(id)
        .then(g => {
        // g é um objeto
        });
};