FabricaDeSoftwareINF/Mural-UFG — pull request #127 by brunorfranco (2014-11-20 01:27:23+00:00)
Title: Implementacao IMPNEG Issue 91

Implementação IMPNEG #Issue 91.

Seguem algumas considerações:

O construtor de dispositivo esta retornando um objeto Notificação, é o que esta no diagrama de sequencia, sequencia Típica, está correto?

Aliás, na sequencia t[ipica não possui a classe Notificacao, que esta presente nos diagramas de sequencia com exceções.

A classe Dispositivo deveria ter um atributo Usuario para armazenar essa informação após criação e passagem pelo construtor.

Qual a motivação para os m[etodos validarDispositivo, verificarUsuarioNotificacaoDirigida e validarCredencial serem públicos no diagrama de classe
se só são utilizados dentro da própria classe? Estamos perdendo encapsulamento dessa forma.

Se o usuário e o identificador são passados no construtor, para que passar novamente no método dessassociarDispositivoAoUsuario? Vou escolher passar apenas no construtor.

A classe UsuarioDao não deveria ter o método boolean verificarUsuarioNotificacaoDirigida(Usuario usuario)?

Eu suponho que o método validaDispositivo(String identificador) no diagrama de classe seja referente ao passo 2 da sequencia tipica: CentralUFG localiza o dispositivo informado.
Vou criar um StubDispositivoDao para futura avaliação desse passo e método.

Se o dessassociarDispositivoAoUsuario é void no diagrama de sequencia, como será devolvido o código de sucesso ou exceção para o dispositivo móvel?
Vou alterar para um Enumerado com os valores de retorno, mas também poderia ser uma String.

Os casos de teste gerados não atendem os 70% de cobertura informados porque grande parte do código foi feito por classes de Entidades, Stubs e classes de Exceção. A classe relevante de negócio possui seus métodos testados de forma adequada.