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

TestGit #28

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

JoSchmidtIBW
Copy link

No description provided.

class UserValidatorTest {

@Nested
class isValidUsername{

@Test
void returnsTrueIfOnlyLetters(){
throw new IllegalArgumentException("you should implement code here");
//kein mokito
Copy link
Contributor

Choose a reason for hiding this comment

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

Die Idee all dieser isValidUsername Testfälle ist es, sicherzustellen, dass alle Szenarien (z.B. "OnlyLetters") von der Methode isValidUsername korrekt erkannt werden.
z.B. für den returnsTrueIfOnlyLetters Testfall könnten die Assertion sein: Assertions.assertFalse(new User(",,,,").isValidUsername()); Assertions.assertTrue(new User("abcdefg").isValidUsername());

Im Testfall selbst befindet sich keine if oder for Schlaufe. Dieser Validierungsalgorithmus muss in die User Klasse.

}

@Test
void returnsTrueIfUsernameInDB(){
throw new IllegalArgumentException("you should implement code here");
void returnsTrueIfUsernameInDB(){ //Test ist immer grün, auch wenn irgend welcher Code gelöscht wird...??????????????????????????
Copy link
Contributor

Choose a reason for hiding this comment

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

Klar ist der immer grün ;) Schau mal wo überall du auf deine mockDatabase zugreifst, und wo überall du darauf zugreifen solltest ;)

Mok/ Mokito noch offen
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.

2 participants