Por vacilo meu, eu aceitei sem rodar o teste eu mesmo, com o pensamento de que "se tem um teste, então é porque o pessoal rodou ele antes de me mandar o merge request; tá tranquilo". Ledo engano.
Só que tem um erro de digitação bem sutil no teste, que faz com que ele falhe assim:
1) Failure: test_should_create_a_new_user_with_image(AccountControllerTest) [/vagrant/test/functional/account_controller_test.rb:651]: Expected response to be a <:successs>, but was <200>
Eu já dei push de um commit consertando o teste, mas no futuro, por favor *rodem os testes* que vocês estão escrevendo antes de fazer merge request.
A equipe, desde algum tempo, sempre escreve e roda todos os testes antes de enviar um patch para o Noosfero. Como o Fábio disse, não fizeram diferente desta vez.
Muito provavelmente foi um erro devido falta de atenção em algum momento antes de enviar, de fato um erro!
Mas fica ai o aprendizado para termos mais atenção das próximas vezes.
att Arthur
Em 24 de setembro de 2014 16:35, Fábio Teixeira escreveu:
> Desculpe o transtorno. Mas estamos realmente perplexos com o ocorrido. > > Durante o desenvolvimento, alem deste, também rodamos todos os testes > unitários e funcionais do noosfero, para garantir que nada havia quebrado. > > Como todos os testes haviam passado, fizemos o merge request. > > Futuramente, iremos rodar os testes em diferentes PCs para ter total > certeza. > Obrigado pela atenção e desculpe pelo ocorrido. > > _______________________________________________ > spb-dev mailing list > spb-dev@listas.softwarepublico.gov.br >http://listas.softwarepublico.gov.br/mailman/cgi-bin/listinfo/spb-dev > >
Antonio Terceiro
Fabio Teixeira
Arthur Esposte