Code review comment for lp:~vauxoo/addons-vauxoo/7.0-user_story-effective_hours-dev-hbto

Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

@Hbto,
FYI Esta MP está ciclando uno de los test, tengan en cuenta esto antes de mergear.
http://runbot.vauxoo.com/Chuck_Norris.html#erp_vauxoo_com_-mp-218128

El log es el siguiente
Module project_timesheet: loading test/worktask_entry_to_timesheetline_entry.yml
 Create a user 'HR Manager'
 Create a product with type service used to specify employees designation
 Create an analytic journal for employees timesheet
 Create an employee 'HR Manager' for user 'HR Manager'
 Create a timesheet sheet for HR manager
 Create a project 'Timesheet Management'
 Create a task 'Get all timesheet records'
 Open the task
 Make a work task entry 'Get work calendar of all employees' of 10 hours done by HR manager.
En esta última línea se queda pegado, y runbot detecta que ya lleva 40 minutos generándose la instancia, y la frena, etiquetándola como "too long".

@Rafa,
Para aprobar una MP, te recomiendo primero probar en runbot en
http://runbot.vauxoo.com/Chuck_Norris.html
Busca el número de la propuesta de merge, y le das "force build", una vez que se genera, puedes conectarte con usuario admin y password admin, para probar funcionalmente la vaina en desarrollo.
Si no está en verde o no te satisface, dispara alarmas, please.
Si todo trabaja fino funcionalmente, ahora sí apruebas.

review: Needs Fixing

« Back to merge proposal