Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMP] account_invoice_line_stock_move_info: Change test. #110

Conversation

alfredoavanzosc
Copy link
Contributor

Se ha tenido que modificar el test, porque en la condición de búsqueda de test se hacía referencia al campo "sale_id". Antes funcionaba, ahora no porque dicho campo, es decir "sale_id", ya no es un campo de tabla, y por lo tanto no se puede meter en una condición para realizar una búsqueda en albaranes.

@pedrobaeza
Copy link
Member

Ese cambio se ha hecho en Odoo core?

En cualquier caso, documenta con un comentario por qué se tiene que hacer así para luego cuando se mire de nuevo se sepa.

@alfredoavanzosc alfredoavanzosc force-pushed the odoomrp-utilsV8-account-invoice_line_stock_move_info branch from 1a4ae79 to 4d46a39 Compare December 11, 2015 07:48
@alfredoavanzosc
Copy link
Contributor Author

Buenos días @pedrobaeza . Pienso que el cambio de no tener disponible el campo "sale_id" vendrá del core, pienso yo vamos, no le veo otra explicación si no. En su día este módulo pasó el test, y ahora ha saltado la liebre, y ha fallado el test de este módulo, al hacer un PR de otro módulo que he modificado.
He comentado la línea como he has solicitado. Un saludo.

@alfredoavanzosc
Copy link
Contributor Author

Buenos días @pedrobaeza , ¿Este error de runbot supongo que es lo mismo de ayer que no se ha arreglado verdad?. Lo que no entiendo es porque ayer con este módulo me fué bien, y ahora mal. Un saludo.

@oihane
Copy link
Contributor

oihane commented Dec 11, 2015

@alfredoavanzosc Odoo no ha cambiado el sale_id ese error que "arreglas" ya daba antes también:
https://travis-ci.org/odoomrp/odoomrp-utils/jobs/76587858#L313

picking = self.picking_model.search(cond, limit=1)
picking = False
pickings = self.picking_model.search([])
# "For" is used because the "sale_id" field of pickings is not written
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pon "is not stored"

@pedrobaeza
Copy link
Member

@alfredoavanzosc, prueba a hacer un filtered a ver si funciona con campos no almacenados.

@alfredoavanzosc alfredoavanzosc force-pushed the odoomrp-utilsV8-account-invoice_line_stock_move_info branch from 4d46a39 to e043b71 Compare December 14, 2015 08:34
@alfredoavanzosc
Copy link
Contributor Author

Buenos días @pedrobaeza , he usado la instrucción "filtered",y al menos en mi local va bien, a ver que pasa aqui en "travis". Gracias.

@alfredoavanzosc
Copy link
Contributor Author

@pedrobaeza , "runbot" me sigue fallando, ¿puedo hacer algo?. Me quiero quitar todos los PRs que tengo del "procurement plan", y de "stock planning".

@pedrobaeza
Copy link
Member

Bueno, en este caso no es necesario runbot, así que mergeo. Prueba a hacer rebuild en los otros PRs

pedrobaeza added a commit that referenced this pull request Dec 14, 2015
…invoice_line_stock_move_info

[IMP] account_invoice_line_stock_move_info: Change test.
@pedrobaeza pedrobaeza merged commit c7f78fd into odoomrp:8.0 Dec 14, 2015
@alfredoavanzosc alfredoavanzosc deleted the odoomrp-utilsV8-account-invoice_line_stock_move_info branch December 14, 2015 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants