Adicionando Validação de CEP #2
Conversation
tests/Flunt.Br.Tests/FluntBrTest.cs
Outdated
| wrong.IsCep("99999", "Cep", "Invalid Cep"); | ||
| Assert.IsFalse(wrong.Valid); | ||
|
|
||
| wrong = new Contract(); |
There was a problem hiding this comment.
@marcoslcosta eu sugiro utilizar a annotation [DataRow("99999")] para cada cep inválido, a lógica é a mesma, o que muda é a entrada, quando executar os testes, ele irá executar um teste para cada DataRow, eu utilizo aqui https://github.com/lira92/flunt.br/compare/1-suportar-mais-telefones?expand=1#diff-74ad8fda737716f9e8c093a1b3a571f6.
There was a problem hiding this comment.
Opa, estou mandando um outro pullrequest com os DataRow.
tests/Flunt.Br.Tests/FluntBrTest.cs
Outdated
| [TestMethod] | ||
| public void IsCep_Valid() | ||
| { | ||
| var right = new Contract(); |
There was a problem hiding this comment.
@marcoslcosta aqui também acho que seria interessante o uso do DataRow.
There was a problem hiding this comment.
perfeitamente.
There was a problem hiding this comment.
acho que para eu mandar outro pull resquest,você deve rejeitar ou aceitar esse.
|
@marcoslcosta perfeito. Consegue fazer squash dos commits? |
| @@ -70,5 +70,28 @@ public void IsCellPhone_Valid() | |||
| .IsCellPhone("(45)99999-9999", "cellphone", "Invalid cellphone"); | |||
| Assert.AreEqual(true, right.Valid); | |||
| } | |||
|
|
|||
| [TestMethod] | |||
| [DataRow("99999")] | |||
There was a problem hiding this comment.
adicionei os DataRow aqui
| } | ||
|
|
||
| [TestMethod] | ||
| [DataRow("99999-999")] |
There was a problem hiding this comment.
Adicionei os DataRow Aqui também
|
@lira92 Estou vendo como fazer |
MarcosCostaDev
force-pushed
the
master
branch
2 times, most recently
from
64bb2bb
to
b243f58
Compare
# This is the 1st commit message: melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow # This is the commit message lira92#2: adicionando validação de CEP
|
@marcoslcosta faz assim, adiciona o remote do projeto: |
|
no caso faltou so o |
|
@marcoslcosta está aparecendo 7 commits, o esperado é que tivesse somente 1 após o squash. você marcou todos os commits exceto o primeito com s ? |
adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP adicionando validação de cep adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP adicionando validação de cep adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP adicionando validação de cep adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP adicionando validação de cep adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP adicionando validação de cep adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP adicionando validação de cep adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP adicionando validação de cep adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP melhoria nos testes, usando DataRow adicionando validação de CEP
MarcosCostaDev
force-pushed
the
master
branch
from
9e55146
to
c726baa
Compare
|
Agora foi. |
|
@marcoslcosta perfeito, obrigado pela contribuição, vou aceitar e assim que possível gero o release. |
# This is the 1st commit message: atualizando readm # This is the commit message lira92#2: Colocando validação de telefone
Adicionei uma validação de CEP ao Flunt.Br, apenas por comodidade.