Ronneesley / redesocial

Projeto do Curso de Sistemas de Informação sobre Rede Social
https://ronneesley.github.io/redesocial
MIT License
41 stars 11 forks source link

Criação da unidade de testes para Album método excluir() #164

Closed Ronneesley closed 7 years ago

iankatalita commented 7 years ago

Confere pra ver se está tudo certo.

iankatalita commented 7 years ago

Ronee, não to entendendo como tenho que fazer o método excluir do álbum. Seguinte, tá pedindo data, ai lá no dto temos os campos Id, Nome, Data e Usuário, então quando eu for fazer o método excluir tenho que seguir o que tem lá? Tenho que mandar excluir todos aqueles campos? Ou os mesmos campos que tem em inserir? To mega confusa pq de todo jeito que faço ta pedindo a data só que eu coloquei a data. ME SOCORRE, PLEASE.

iankatalita commented 7 years ago

Ronne, eu de novo kkkkkkkkk tomei base o código da Thalia, acho que entende o que tinha que ser feito, se você inseriu as informações no método inserir, eram aquelas informações que eu iria pegar pra remover o álbum, bom meio que entendi assim, só não entendi o pq desse código ( bo.excluir(id); - Album albumSelecionadoPosExclusao = bo.selecionar(id); ) ele serve pra escolher no caso? Realmente não entende. No final nos comandos de assertNotNull não deu pra colocar o fail tava dando erro e com a mensagem de Falha ao inserir Album: null, então pensei que as vezes o fail não se classificaria ali, e achei melhor inverter, coloquei outro assertNotNull, não sei se está correto, realmente não sei. Tenho muita dificuldade em entender algumas coisas. O código compilou sem erro, agora bom, me explica pra que serve esse código, please. :)

Ronneesley commented 7 years ago

Esses comandos servem para verificar se está excluído mesmo. Tirar o fail() que deveria estar lá foi que nem varrer sujeira para debaixo da cama, se estava chegando lá era porque estava falhando, o que você fez foi tirar a mensagem de falha. Execute passo a passo para ver onde falha.

iankatalita commented 7 years ago

Pronto, menino Ronne.

Ronneesley commented 7 years ago

A indentação não está boa. Aguardo a correção.

iankatalita commented 7 years ago

Indentação pronta, confere se atende aos seus critérios, menino Ronne kkkkkkk

Ronneesley commented 7 years ago

Ainda não está bom no trecho:

} catch (Exception ex){ fail("Falha ao inserir um album: " + ex.getMessage()); }