Skip to content

2. Hausaufgabe, Philip Jach, Matr.-Nr. 574366#58

Open
realphilip wants to merge 5 commits intoProfWider:masterfrom
realphilip:master
Open

2. Hausaufgabe, Philip Jach, Matr.-Nr. 574366#58
realphilip wants to merge 5 commits intoProfWider:masterfrom
realphilip:master

Conversation

@realphilip
Copy link

Fertig

@DanielWagner87 DanielWagner87 self-requested a review October 29, 2020 08:36
@DanielWagner87 DanielWagner87 self-assigned this Oct 29, 2020
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 erste rote Test (Percentage Operator) ist eigentlich kein Bug, der aufgedeckt wurde, sondern eine neue Funktion.

Ich würde mich freuen, wenn Sie noch einen roten Test hinsichtlich einer fehlerhaften Implementierung nachliefern könnten, z. B. die Addition von Fließkommazahlen.

@DanielWagner87
Copy link
Collaborator

An anderer Stelle wurde die Aufgabe bereits mit %-Funtion bestanden, daher auch hier bestanden.

@realphilip
Copy link
Author

Ich dachte es wär ein akzeptables Fix da der Percentage Operator in dem Online-Calculator.com enthalten aber nicht in dem Code enthalten ist. Ich reiche aber gerne noch einen Test nach.

@DanielWagner87
Copy link
Collaborator

Ich dachte es wär ein akzeptables Fix da der Percentage Operator in dem Online-Calculator.com enthalten aber nicht in dem Code enthalten ist. Ich reiche aber gerne noch einen Test nach.

Ganz wie Sie mögen.

@DanielWagner87
Copy link
Collaborator

@realphilip
Hier ist jetzt Travis leider noch rot geworden:

Retro calculator > should move decimal point two spaces forward FAILED
    org.opentest4j.AssertionFailedError: expected: <8.98> but was: <0898>
        at org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:55)
        at org.junit.jupiter.api.AssertionUtils.failNotEqual(AssertionUtils.java:62)
        at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
        at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:177)
        at org.junit.jupiter.api.Assertions.assertEquals(Assertions.java:1124)
        at tdd.setup.CalculatorTest.calculatorCanDecimalizePercentages(CalculatorTest.java:39)

Ein Fix scheint hier noch nicht zu funktionieren.

@DanielWagner87
Copy link
Collaborator

@realphilip
Noch einmal der Hinweis, dass die Hausaufgabe hier aktuell noch nicht bestanden ist. Ein Test schlägt derzeit noch fehl.

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