Open
Conversation
… dados atualizados ao acessar a pagina
Contributor
|
Hello @DevDouglasFigueiredo! Thanks for your submission. Just letting you know that we got your pull request and are in the process of reviewing it. We will soon provide you with our considerations. |
Author
|
Hello @campofernando , thx for the position. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Os testes E2E foram desenvolvidos utilizando Playwright com TypeScript, seguindo o padrão Page Object Model (POM), com o objetivo de deixar os cenários mais organizados, reutilizáveis e fáceis de entender.
Ao longo do projeto, é possível perceber alguns trechos com repetição de código.
Tenho plena consciência da importância do princípio DRY (Don’t Repeat Yourself) em projetos de software. No entanto, em testes automatizados, optei por priorizar a legibilidade dos cenários e a clareza do que está sendo validado em cada teste.
A ideia foi deixar explícito o comportamento esperado em cada fluxo, mesmo que isso gere pequenas repetições.
Ou seja, a principal escolha aqui foi favorecer a compreensão do teste e do comportamento sendo validado, em vez de buscar a máxima reutilização de código.
Além dos testes E2E implementados, consideraria de suma importância a inclusão de testes de API responsáveis pelo envio dos dados para os gráficos no front-end.
A validação da API é importante para garantir que as resposta (estrutura, campos, tipos e valores esperados) permaneçam consistente, reduzindo o risco de falhas na interface causadas por alterações no backend.
Dessa forma, os testes de API complementam os testes E2E, permitindo identificar problemas de integração de forma mais rápida e confiável, antes que impactem diretamente no front-end.
Por fim, o projeto também conta com um pipeline no GitHub Actions e uma imagem Docker, com o objetivo de facilitar a execução e a validação do projeto.
O pipeline automatiza a execução dos testes, garantindo uma verificação rápida e padronizada a cada alteração no código.
Já a imagem Docker permite que qualquer pessoa execute o projeto em um ambiente controlado e padronizado, sem a necessidade de configurar manualmente todas as dependências.
Conforme solicitado nos requisitos ("Espera-se que os testes passem onde os critérios são atendidos e falham onde não estão"), este PR apresenta falhas reais evidenciados pela automação:
deve exibir um cabeçalho com informações das maquinas: Falha pois o campo de tempo está sendo exibido comonull min.deve exibir o tooltip com os valores ao repousar o mouse sobre o gráfico de Temperatura: Falha pois o componente de tooltip não é disparado neste gráfico específico.Análise de Produto e Implementação
Respondendo aos critérios de avaliação sobre detalhes de implementação:
null) e a interatividade prometida nos requisitos (tooltips para todos os gráficos) não foi cumprida no grafico de Temperatura.Relatorios
Para uma revisão detalhada, incluí os seguintes documentos no repositório:
Relatorio da Execução dos Cenarios de Teste