Skip to content

Conversation

@cuococarlos
Copy link

Se adapto la solución para hacer la misma validaciones que se realizan en los gastos que en los pagos.

Creo que se podria refactorizar , ya que hay un poco de reutilización de codigo pero creo que como primera solución esta funcional.

Hice las pruebas sugeridas en el documento de testing funcional.

The solution has been customized to do the same validations that are done in expenses as in payments.

I think it could be refactored since there is some code reuse, but I think it is functional as a first solution.

I did the tests suggested in the functional testing document.

Related to #32

bot/utils.py Outdated
Comment on lines 221 to 228
"desea computar el gasto, con el formato dd/mm/yy."
return text
except User.DoesNotExist:
text = "El usuario espcificado ({}) no existe dentro de este grupo. \n".format(to_user)
text += "Los posibles usuarios a los que les podes cargar un pago son: \n"
async for member in group.users.exclude(pk=user.pk):
text += "- {}\n".format(member.username)
return text
Copy link
Owner

Choose a reason for hiding this comment

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

El resultado de la función debería ser un dict, entiendo que para estos dos casos, en lugar de hacer return text se debería raisear el error esperado:

Suggested change
"desea computar el gasto, con el formato dd/mm/yy."
return text
except User.DoesNotExist:
text = "El usuario espcificado ({}) no existe dentro de este grupo. \n".format(to_user)
text += "Los posibles usuarios a los que les podes cargar un pago son: \n"
async for member in group.users.exclude(pk=user.pk):
text += "- {}\n".format(member.username)
return text
"desea computar el gasto, con el formato dd/mm/yy."
raise ParameterError(text)
except User.DoesNotExist:
text = "El usuario espcificado ({}) no existe dentro de este grupo. \n".format(to_user)
text += "Los posibles usuarios a los que les podes cargar un pago son: \n"
async for member in group.users.exclude(pk=user.pk):
text += "- {}\n".format(member.username)
raise ParameterError(text)

Copy link
Author

Choose a reason for hiding this comment

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

Si, se ve que hice un cambio de ultima momento y quedo mal esa parte. Ahí se agrega el lanzamiento de excepciones como bien sugeriste. Gracias @sofide !

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.

2 participants