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

[TEST] Realização de testes (shared) (fga-eps-mds/2024.2-ARANDU-DOC#59) #47

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

Joao151104
Copy link
Contributor

@Joao151104 Joao151104 commented Dec 18, 2024

Descrição

Este pull request adiciona/modifica testes do componentes shared , conforme previsto na issue fga-eps-mds/2024.2-ARANDU-DOC#59

Revisão

  • test TextEmail;
  • test TextPassword;
  • test TextName;
  • test ErrorPopUp;

Checklist

  • O conteúdo adicionado nesse PR segue o código de conduta do projeto;
  • Foram seguidos os padrões estabelecidos no guia de contribuição;
  • O conteúdo adicionado foi devidamente testado;
  • O conteúdo do PR foi revisado por membro alheio ao mesmo;
  • O conteúdo deste PR foi devidamente identado, comentado e documentado;
  • Para conteúdos de documentação, foi adicionada tabela de versionamento aos artefatos.

Copy link
Contributor

@GabrielCostaDeOliveira GabrielCostaDeOliveira left a comment

Choose a reason for hiding this comment

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

faça teste de software somente em Error PopUp, Text Email, Text Name, Text Password, não faz sentido fazer teste de software nas outras classes pois ela não tem estado

@@ -0,0 +1,21 @@
import 'package:flutter/material.dart';
Copy link
Contributor

Choose a reason for hiding this comment

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

não faz sentido testar essa classe, porque ela não tem nunhum comportamento

@@ -0,0 +1,46 @@
import 'package:flutter/material.dart';
Copy link
Contributor

Choose a reason for hiding this comment

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

não faz sentido testar essa classe, porque ela não tem nunhum comportamento

@@ -0,0 +1,39 @@
import 'package:flutter/material.dart';
Copy link
Contributor

Choose a reason for hiding this comment

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

não faz sentido testar essa classe, porque ela não tem nunhum comportamento

@dartmol203
Copy link
Contributor

@Joao151104 por favor, remova o teste que não é mais coerente (widget_test.dart) ele está falhando

@dartmol203 dartmol203 merged commit 0dab13d into dev Dec 20, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants