Skip to content
  • P
    Projects
  • G
    Groups
  • S
    Snippets
  • Help

yotta / pictogram

  • This project
    • Loading...
  • Sign in
Go to a project
  • Project
  • Repository
  • Issues 60
  • Merge Requests 0
  • Pipelines
  • Wiki
  • Settings
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Closed
Open
Issue #225 opened May 18, 2017 by Sebastián Collado Montañez@scollado 
  • New issue
New issue

Pruebas: 1.1.4 Comportamiento no esperado al crear cuenta tutor (Sin centro)

tutor

  • ID APARTADO: 1.1
  • ID PRUEBA: 4

Esperado: Tras registrar el usuario como tutor sin centro, se espera que no pueda hacer login hasta que el administrador de un gabinete le asigne un estudiante.

Resultado: El tutor recién registrado puede hacer login sin ningún problema tras confirmar su email, como Administrador de gabinete.

  • Arturo Montejo Ráez @amontejo

    added To Do label

    May 22, 2017

    added To Do label

    added ~29 label
    Toggle commit list
  • Jose Antonio @jlrufian

    closed

    May 23, 2017

    closed

    closed
    Toggle commit list
  • Jose Antonio @jlrufian

    reopened

    May 23, 2017

    reopened

    reopened
    Toggle commit list
  • Jose Antonio @jlrufian

    changed title from Pruebas: 1.1.3 Comportamiento no esperado al crear cuenta tutor to Pruebas: 1.1.4 Comportamiento no esperado al crear cuenta tutor (Sin centro)

    May 23, 2017

    changed title from Pruebas: 1.1.3 Comportamiento no esperado al crear cuenta tutor to Pruebas: 1.1.4 Comportamiento no esperado al crear cuenta tutor (Sin centro)

    changed title from **Pruebas: 1.1.{-3 Comportamiento no esperado al crear cuenta tutor-}** to **Pruebas: 1.1.{+4 Comportamiento no esperado al crear cuenta tutor (Sin centro)+}**
    Toggle commit list
  • Jose Antonio @jlrufian commented May 23, 2017

    El problema está en el siguiente código de SupervisorController.js:

    if (params.role === 'therapist_office' || params.role === 'tutor_office') {
            sendConfirmationMail((err) => {
              if (err) throw err;
              return res.ok();
            });
          } else if (params.role === 'therapist_nooffice' || params.role === 'tutor_nooffice') {
    
            Office.create(params.office)
            .then((off) => {
    
              // link supervisor with office
              console.log("supervisor: \n" + JSON.stringify(sup));
              sup.id_off = off.id;
              delete sup.password;
              sup.save();
              console.log("supervisor: \n" + JSON.stringify(sup));
    
              // set supervisor as admin in the office
              off.admin = sup.id;
              off.save();
    
          supervisor = sup;
              sendConfirmationMail((err) => {
                if (err) throw err;
                return res.ok();
              });
            })
            .catch(err => {throw err});
          } else
            return res.badRequest("Invalid role");

    Si le indicamos que NO tenga gabinete, crea una nueva office y asigna al tutor como administrador.

    @amontejo ¿En caso de tener rol "tutor_nooffice" debería simplemente no crear la office?

    Edited May 23, 2017 by Jose Antonio
    El problema está en el siguiente código de SupervisorController.js: ``` if (params.role === 'therapist_office' || params.role === 'tutor_office') { sendConfirmationMail((err) => { if (err) throw err; return res.ok(); }); } else if (params.role === 'therapist_nooffice' || params.role === 'tutor_nooffice') { Office.create(params.office) .then((off) => { // link supervisor with office console.log("supervisor: \n" + JSON.stringify(sup)); sup.id_off = off.id; delete sup.password; sup.save(); console.log("supervisor: \n" + JSON.stringify(sup)); // set supervisor as admin in the office off.admin = sup.id; off.save(); supervisor = sup; sendConfirmationMail((err) => { if (err) throw err; return res.ok(); }); }) .catch(err => {throw err}); } else return res.badRequest("Invalid role"); ``` Si le indicamos que NO tenga gabinete, crea una nueva office y asigna al tutor como administrador. @amontejo ¿En caso de tener rol "tutor_nooffice" debería simplemente no crear la office?
  • Jose Antonio @jlrufian

    assigned to @jalr0005

    May 23, 2017

    assigned to @jalr0005

    assigned to @jalr0005
    Toggle commit list
  • Arturo Montejo Ráez @amontejo commented May 25, 2017
    Owner

    Este comportamiento es el correcto. Un padre debe poder dar de alta a su hijo sin depender de su gabinete. Mírate el resultado esperado de la prueba @jalr0005 y verás que todo está correcto.

    Edited May 25, 2017 by Arturo Montejo Ráez
    Este comportamiento es el correcto. Un padre debe poder dar de alta a su hijo sin depender de su gabinete. Mírate el resultado esperado de la prueba @jalr0005 y verás que todo está correcto.
  • Arturo Montejo Ráez @amontejo

    closed

    May 25, 2017

    closed

    closed
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
No due date
1
Labels
To Do
Assign labels
  • View labels
3
3 participants
Reference: yotta/pictogram#225