Descricão da transação e valor estão inconsistentes

[giggio]

Há casos em que o campo descrição está sendo enviado com mais de 10 caracteres ou vazio. Peguei estes dois logo no começo do teste.

rinha-de-backend-2024-q1/load-test/user-files/simulations/rinhabackend/RinhaBackendCrebitosSimulation.scala

Line 196 in e1729f3

.body(StringBody(s"""{"valor": 1, "tipo": "c", "descricao": "123456789 e mais um pouco"}"""))

e

rinha-de-backend-2024-q1/load-test/user-files/simulations/rinhabackend/RinhaBackendCrebitosSimulation.scala

Line 203 in e1729f3

.body(StringBody(s"""{"valor": 1, "tipo": "c", "descricao": ""}"""))

e

rinha-de-backend-2024-q1/load-test/user-files/simulations/rinhabackend/RinhaBackendCrebitosSimulation.scala

Line 210 in e1729f3

.body(StringBody(s"""{"valor": 1, "tipo": "c", "descricao": null}"""))

A especificação no README diz que o campo é obrigatório, e que deve ter entre 1 e dez caracteres:

rinha-de-backend-2024-q1/README.md

Line 32 in e1729f3

- `descricao` deve ser uma string de 1 a 10 caractéres.

Seria bom esclarecer e corrigir, afinal o campo é mesmo obrigatório? Tem mesmo esse limite de dez caracteres?

Percebi que o mesmo está acontecendo para o valor. O README diz que deve ser um inteiro:

rinha-de-backend-2024-q1/README.md

Line 30 in e1729f3

- `valor` deve um número inteiro positivo que representa centavos (não vamos trabalhar com frações de centavos). Por exemplo, R$ 10 são 1000 centavos.

Mas nos testes vemos um decimal:

rinha-de-backend-2024-q1/load-test/user-files/simulations/rinhabackend/RinhaBackendCrebitosSimulation.scala

Line 182 in e1729f3

.body(StringBody(s"""{"valor": 1.2, "tipo": "d", "descricao": "devolve"}"""))

[leoralph]

Eu acho que isso é para simular envio errado por parte do client, a api tem que estar preparada para retornar 422 caso seja menor que 1, maior que 10 ou nulo, pelo menos foi o que eu entendi

[zebaroni]

Esses testes são justamente pra testar se sua API ta tratando o erro e retornando o 422

[giggio]

Galera, prestem atenção na descrição do issue: o README diz uma coisa e os testes fazem outra. Esse é o problema.

[zebaroni]

Não entendi, os testes que você citou estão esperando o status 422 na resposta.

• Manda um payload errado
• Espera o 422

Se retornar 200. o teste quebra

[zanfranceschi]

@giggio é como o pessoal falou, esses testes estão validando se o retorno é um 422 para esses casos de payload inválido.

Note o .check(status.in(422)) – a API precisa retornar um 422 nesses requests para passar nos testes.

[giggio]

@zanfranceschi Sem problemas, mas então é necessário atualizar o readme, a especificação está confusa. O único local onde aparece que o retorno de um 422 no readme é:

Se uma requisição para débito for deixar o saldo inconsistente, a API deve retornar HTTP Status Code 422 sem completar a transação! O corpo da resposta nesse caso não será testado e você pode escolher como o representar.