Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix-test-acceptance] Fixes broken unit tests #16

Merged
merged 4 commits into from
May 10, 2017

Conversation

AlineLee
Copy link
Contributor

@AlineLee AlineLee commented Apr 17, 2017

👍 @erikatakahara

@AlineLee @diegocesar

Fixes broken unit tests.

@FernandaBernardo
Copy link
Contributor

FernandaBernardo commented Apr 18, 2017

Não entendi esse PR... Os testes não estavam passando e vocês apagaram os asserts para passar? Não seria melhor tentar arrumar o que quebrou esses testes?

--

Quando foi feita aquela história para upload de vários arquivos nós refatoramos para enviar os dados via formData e consequentemente não passamos mais os headers de json e multipart/data por padrão, mas acabamos esquecendo de ajustar o teste de aceitação. O input file foi removido pelo mesmo motivo, neste novo modelo ele só aparece quando há um arquivo anexo.

Com excessão do assert.equal(xhr.headers["Content-Type"][0], "application/x-www-form-urlencoded;charset=UTF-8"); que só precisa remover um espaço ;)

@AlineLee @diegocesar

@tisvasconcelos
Copy link
Contributor

tisvasconcelos commented Apr 18, 2017

Guys, please, remember this is an open source project and use english next time :)

@luiz

@williammizuta williammizuta merged commit 6b0327c into master May 10, 2017
@williammizuta williammizuta deleted the fix-test-acceptance branch May 10, 2017 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants