Atividade 1 - contaBancaria - Joyce Mikaella#34
Conversation
|
|
||
| function saqueValor(contaBancaria, valorSaque) { | ||
| if (valorSaque <= contaBancaria.saldo + contaBancaria.limite) { | ||
| contaBancaria.saldo = contaBancaria.saldo - valorSaque |
There was a problem hiding this comment.
Considerando que o valor do saque deve ser menor ou igual ao saldo com o limite, não deveria alterar também o valor do limite aqui? Pq o saque também pode alterar o valor do limite
| if ((valorReajuste) > 0) { | ||
| contaBancaria.limite = contaBancaria.limite + valorReajuste | ||
| } else if ((valorReajuste) < 0) { | ||
| contaBancaria.limite = contaBancaria.limite - Math.abs(valorReajuste) |
There was a problem hiding this comment.
Acho que ficaria mais fácil de entender se somasse um valor negativo
| expect(saqueValor(MockCliente, 400)).toEqual(4000) | ||
| }) | ||
| it("Deve impedir o saque quando o saldo/limite é insuficiente", () => { | ||
| expect(saqueValor(MockCliente, 11500)).toEqual(`Olá ${MockCliente.titular}infelizmente o seu saldo é insuficiente para realizar esta operação!`) |
There was a problem hiding this comment.
Você poderia usar uma variável como output, colocando a mensagem de retorno, pra linha ficar menor. Mas dá pra entender também!! :)
babicfonseca
left a comment
babicfonseca
left a comment
There was a problem hiding this comment.
Gostei bastante da construção das funções e dos testes, ficou fácil de entender. A cobertura dos testes também tá boa, pq você fez várias hipóteses.
Marianaliima
left a comment
Marianaliima
left a comment
There was a problem hiding this comment.
Olá tudo bem? Parabéns pela entrega, uma única dúvida é em relação ao math.abs? entendo que na função não seria necessário utilizar, você tentou sem? Nos testes também poderiam ser cobertos mais cenários, porém você demonstrou ter entendido como funciona construção dos testes e até utilizou beforeEach bom trabalho.
3 participants
Resolução do exercício de Conta Bancária