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] Remove web modules tests from travis #1954

Merged
merged 1 commit into from
Jun 2, 2022

Conversation

kmmelcher
Copy link
Contributor

#1950 resolvendo o problema discutido nessa PR.

Remove os seguintes módulos dos testes do travis:

  • payment_pagseguro
  • l10n_br_website_sale_delivery
  • l10n_br_website_sale

Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kmmelcher valeu!
Ai:

  • se vocês pensam ter uma solução que aumenta a estabilidade desses testes depois a gente tanta habilitar eles de novo.
  • de qualquer forma, o foco do projeto esta mudando para a 14.0. Vamos ver como se comportam esses testes na 14.0, bem capaz que não tenha mais esse problema.
  • quando houver um PR que mexe com esses módulos, vamos ficar nos maintainers do projeto e vcs autores desses módulos, responsáveis verificar que os testes realmente passam usando um teste local.

@rvalyi rvalyi merged commit b6c1eb4 into OCA:12.0 Jun 2, 2022
@kmmelcher
Copy link
Contributor Author

@kmmelcher valeu! Ai:

  • se vocês pensam ter uma solução que aumenta a estabilidade desses testes depois a gente tanta habilitar eles de novo.
  • de qualquer forma, o foco do projeto esta mudando para a 14.0. Vamos ver como se comportam esses testes na 14.0, bem capaz que não tenha mais esse problema.
  • quando houver um PR que mexe com esses módulos, vamos ficar nos maintainers do projeto e vcs autores desses módulos, responsáveis verificar que os testes realmente passam usando um teste local.

Pessoal, eu ainda não tinha testado se o código funcionava (não sei como rodar o travis localmente).
Abri a PR para que os testes podessem ser rodados, mas deveria ter aberto no status Rascunho, perdão, não me atentei e não sabia que seria feito merge tão rápido. A mudança que eu fiz não alterou os testes do Travis, tenho que tentar um pouco mais para descobrir como faz:

image

O travis ainda está testando a pagseguro.

@kmmelcher
Copy link
Contributor Author

A parte dos testes tem 4 ambientes:

image

Olhando outras PRs, eu vi que antes só tinham 3:

image

No travis yaml só tem 2:

image

No último ambiente de teste a variável de ambiente realmente funcionou:

image

Porém nos outros continuam sendo rodados os testes da pagseguro.
Vou tentar adicionar a variável de excludo no repo "odoo/odoo" também

@kmmelcher
Copy link
Contributor Author

Vou abrir outra PR da mesma branch, porém no estado Draft agora, para evitar esse problema novamente, desculpe pelo deslize

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants