Skip to content

2.HA S0575481#66

Open
phamquochuy2905 wants to merge 14 commits intoProfWider:masterfrom
phamquochuy2905:master
Open

2.HA S0575481#66
phamquochuy2905 wants to merge 14 commits intoProfWider:masterfrom
phamquochuy2905:master

Conversation

@phamquochuy2905
Copy link

No description provided.

neuer grüner Test
neuer roter Test
Bugfix zum 1.Test
2.Roter Test
Bugfix zum 2.Test
@DanielWagner87 DanielWagner87 self-assigned this Oct 29, 2020
@DanielWagner87 DanielWagner87 self-requested a review October 29, 2020 14:59
Copy link
Collaborator

@DanielWagner87 DanielWagner87 left a comment

Choose a reason for hiding this comment

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

Folgenden Punkt aus der Aufgabenstellung sehe ich noch nicht erfüllt:

Da die Kombination pressOperationKey, pressDigitKey, pressEquals schon getestet ist, sollte also noch mindestens eine der Methoden pressClear, pressDotKey, oder pressNegative getestet werden.

Streng genommen ist die Prozentfunktion auch nicht die Aufdeckung eines bestehenden Fehlers (wie von der Aufgabenstellung vorgegeben), sondern die Implementierung einer gänzlich neuen Funktionalität.

Copy link
Collaborator

@DanielWagner87 DanielWagner87 left a comment

Choose a reason for hiding this comment

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

Der Fix löst das Problem nicht. Bei mir ist der Test noch rot.

Copy link
Collaborator

@DanielWagner87 DanielWagner87 left a comment

Choose a reason for hiding this comment

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

Der letzte Test ist immer noch rot.
Bitte führen Sie gradle test auf Ihrer lokalen Maschine aus, bevor Sie pushen. Damit können Sie selbst ganz einfach überprüfen, ob es nun passt oder nicht.

@phamquochuy2905
Copy link
Author

Der letzte Test ist immer noch rot.

Bitte führen Sie gradle test auf Ihrer lokalen Maschine aus, bevor Sie pushen. Damit können Sie selbst ganz einfach überprüfen, ob es nun passt oder nicht.

Leider war es rot :( ich weiß nicht woran es liegt aber mein Laptop ist seit gestern Nachmittag defekt.

Immer wenn ich IntelliJ Java starte, kommt es nach ein paar Minuten "Blue screen error" :(

Tests kann ich auch nicht mehr ausführen. Es ist ja ärgerlich :(

Ich versuche immer noch das Problem zu beheben.

@DanielWagner87
Copy link
Collaborator

Der letzte Test ist immer noch rot.
Bitte führen Sie gradle test auf Ihrer lokalen Maschine aus, bevor Sie pushen. Damit können Sie selbst ganz einfach überprüfen, ob es nun passt oder nicht.

Leider war es rot :( ich weiß nicht woran es liegt aber mein Laptop ist seit gestern Nachmittag defekt.

Immer wenn ich IntelliJ Java starte, kommt es nach ein paar Minuten "Blue screen error" :(

Tests kann ich auch nicht mehr ausführen. Es ist ja ärgerlich :(

Ich versuche immer noch das Problem zu beheben.

Versteifen Sie sich bitte nicht zu sehr auf IntelliJ. Sie können die Hausaufgabe auch mit einem ganz normalen Texteditor lösen. gradle test läuft auf Ihrem Terminal.

@phamquochuy2905
Copy link
Author

Hmm. Hier auf Github geht es immer noch nicht, obwohl der Test auf meinem Rechner schon grün ist.

@DanielWagner87
Copy link
Collaborator

Denken Sie bitte daran, dass am Ende alle Tests grün sein müssen. Wie mir scheint machen Sie mit einem Fix gleichzeitig wieder einen anderen Fix kaputt.

Prüfen Sie vor einem Push Ihre Tests immer einmal mit gradle test. Damit sind Sie auf der sicheren Seite. Wenn dann Travis rot ist, schau ich noch mal auf meiner lokalen Maschine.

@DanielWagner87
Copy link
Collaborator

Bestanden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants