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

Testes de integração no PHPSP #43

Open
nawarian opened this issue May 21, 2019 · 5 comments
Open

Testes de integração no PHPSP #43

nawarian opened this issue May 21, 2019 · 5 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@nawarian
Copy link
Member

Como pode-se notar pelos PRs #41 e #42 andei fazendo alguns testes para adicionar uma suíte de integração com Behat ao site do PHPSP.

Estou enfrentando dificuldades ao executar os testes contra o deployment no contexto preview:

  1. Existem dois deployments configurados: deploy preview e dev- deploy preview, porém ambos executam o mesmo arquivo netlify.toml

  2. A entrada command do arquivo netlify.toml me permite alterar o comando de build, portanto só é possível executar o teste de integração após o segundo build: Durante o primeiro build não existe nenhuma versão disponível na URL de preview, fazendo com que os testes quebrem.

Por definição a fase de build não é a melhor para executar testes de integração. Penso que uma solução com Webhooks entre Github e Netlify seria uma saída plausível.

Alguma ideia?

@nawarian nawarian added help wanted Extra attention is needed question Further information is requested labels May 21, 2019
@jpjoao
Copy link
Member

jpjoao commented May 22, 2019

Não é lá muito uma ideia, apenas colocando aqui o motivo de ter os dois "ambientes".
Um era para ter previews dos MR que tem como target a master e o outro as que tem cmo target develop.
Mas já que a gente deixou de lado a branch develop seria natural ter apenas um deploy (e decidir se queremos que seja o dev- ou não.

Sobre o teste de integração, o "ideal" seria que ele rodasse depois que o netlify contruiu o site, num estágio seguinte da "pipeline".

@nawarian
Copy link
Member Author

Eu gosto da ideia de remover a build dev- e manter somente o preview normal...

E concordo com a questão de botar o teste pra rodar depois de o netlify buildar o site. A questão é: que meios a gente tem pra desenvolver isso?

@ravanscafi
Copy link
Member

Precisamos usar o netlify pra isso? Por mim configuramos um travis/circleci e rodamos nele mesmo. Habilitamos branch protection no github pra garantir que os testes passaram e pronto. Como nunca vai cair na master sem testes, não vai ser feito o deploy do netlify. Não precisa usar ele pra isso.

O que acham?
Se quiserem posso configurar.

@jpjoao
Copy link
Member

jpjoao commented Jun 14, 2019

O Netlify faz o preview "out of the box", acho que não tem muito pq desabilitar isso. É mais uma forma de validar que as mudanças não quebram nada (e que elas inclusive fazem o que deveriam fazer) antes de dar o merge. Com ou sem testes.

E ter o CircleCI, Travis e tudo mais eu vejo como um "plus a mais" que vem pra somar.

@jpjoao jpjoao added enhancement New feature or request has PR already This issues has already a PR and removed question Further information is requested labels Jun 14, 2019
This was referenced Jun 14, 2019
@ravanscafi ravanscafi removed the has PR already This issues has already a PR label Jul 16, 2019
@ravanscafi
Copy link
Member

@nawarian agora que temos o setup, o que acha de recuperar o PR #42 e colocar no CircleCI?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants