On Jan 30, 2014, at 3:23 PM, pedrootorres <notifications@github.com> wrote:

> In test/functional/steps/NewsTestDataAndOperations.groovy:
> 
> > @@ -30,4 +30,44 @@ class NewsTestDataAndOperations {
> >          def news = News.findByDescriptionAndDateAndResearchGroup(description, dateAsDateObj, researchGroup)
> >          return news != null
> >      }
> > +
> > +    static public boolean checkExistingNewsByDescription(String description) {
> > +        def news = News.findByDescription(description)
> > +        return news != null
> Professor, não entendemos muito bem o comentário.
> O objetivo desse método é verificar se existe uma news a partir de uma certa descrição. Na linha 35, a feita a busca. Se a busca retornou uma news, é porque ela existe.
> 
OK, mas a news pode ter sido armazenada com valores de atributos diferentes do que deveria ser armazenado. Imagina um método que armazena a news mas zera todos os campos dela, por exemplo. O teste funcionaria com ele, certo? Mas o sistema não estaria correto. Entendeu?
> Se retornou null, é porque não existe. Por isso estamos comparando news != null. Se formos verificar o valor de algum atributo dessa news, e ela fosse null, daria erro.
> 
> —
> Reply to this email directly or view it on GitHub.
>