merge request com teste quebrado
-
24 de Setembro de 2014 às 18:06Recentemente eu aprovei um merge request que continha esse commit aqui:
https://gitlab.com/noosfero/noosfero/commit/8ee125a83859872afde7d73049c19b17e0c1ff73
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.
--
Antonio Terceiro
http://softwarelivre.org/terceiro -
24 de Setembro de 2014 às 19:35Desculpe 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. -
26 de Setembro de 2014 às 02:34Muito estranho o ocorrido mesmo!
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
>
>
Ordenar por:
Relacionado:
- e-sic-livre Nova versão do e-SIC Livre!
- spb-dev Revisões de Merge - Terceiro
- spb-dev Paginação e testes da API
- e-sic-livre Anexo de arquivos
- spb-dev Merge Request não está funcionando
- i-educar Versão do php do ieducar
- spb-dev rodando noosfero num /subdiretorio
- e-sic-livre Merge request
- e-sic-livre =?ISO-8859-1?Q?_Remo=E7=E3o_da_enquet?=e
- spb-dev Status do dia
Estatísticas:
-
iniciada em
10 anos, 1 mês atrás
-
vizualizada
1085 vezes
-
respondida
3 vezes
-
votada
0 vezes