[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.