[TRNF-6565] Add delete method to AccountNumbers v2 - #117
Conversation
|
|
||
| def test_v2_account_number_delete(self): | ||
| """Test deleting an account number using v2 API.""" | ||
| account_number_id = "test_account_number_id" |
|
|
||
| result = self.fintoc.v2.account_numbers.delete(account_number_id) | ||
|
|
||
| assert result == account_number_id |
There was a problem hiding this comment.
wait. el delete devuelve el id?
tb veo que los otros tests assertean el m茅todo y la url a la que se pega. tar铆a bueno a帽adirlo en esta pr y la otra que aprob茅
There was a problem hiding this comment.
les estuve dando una vuelta y por na naturaleza de como funciona el metodo delete en sdk no se testea el tema de la url y metodo en todo el repo. Encontr茅 una forma de hacerlo pero rompe la concistencia con todos los tests de delete entonces nose si me convence
There was a problem hiding this comment.
lo dejar铆a cm est谩 en los otros.
y no es raro que te est茅 retornando el acc num id si el result en teor铆a es el objeto completo seg煤n lo que hicistre tu?
o tb son cositas del sdk
There was a problem hiding this comment.
cositas del sdk, para los metodos delete asume que no hay un response y retorna el mismo id que se le pasa al momento de llamar al metodo
2 participants
Description
Agrega m茅todo
deleteal managerAccountNumbersManagerv2, habilitandoDELETE /v2/account_numbers/{id}.Requirements
None.
Additional changes
None.