search

ese-unibe-ch / ese2012-team4

5 stars 1 forks source link

Fix tests #61

Closed gianlucamateo closed 11 years ago

gianlucamateo commented 11 years ago

Some of the tests do not work anymore, and I can't figure out why. I suppose some of them fail because Organisations do not have email addresses. I would be grateful if someone could have a look at the tests.

misch commented 11 years ago

Meinst du die auction tests? Alles andere von production läuft bei mir fehlerfrei durch

gianlucamateo commented 11 years ago

test_validation in item_test funktioniert bei mir auch nicht, falsche assertion in Zeile 188

misch commented 11 years ago

test_validation läuft bei mir. Was ist denn da los...? :-(

gianlucamateo commented 11 years ago

Huh... Ich schaus noch mal an, vielleicht ist was mit dem Bild nicht in Ordnung

LukasSkywalker commented 11 years ago

Bei mir laufen alle ausser die Auktionstests. Dort failen 2, weil sie nicht mailen können, und 2 andere wegen einer assertion.

r-burn commented 11 years ago

Hab das mit den E-Mails gefixt. Eine weitere assertion (also noch insgesamt 3) failt neuerdings auch, wahrscheinlich weil ich ein Bisschen am Code rumgespielt habe (hab es dort in den comments notiert). Ich glaube wir müssen noch einen Blick auf die Logik werfen... Vor Allem, was passiert, wenn der Höchstbietende unterboten wird, das Gebot aber über dem aktuellen Preis liegt. Ich wollte das eigentlich lösen, indem der Preis auf die Höhe des neuen Gebots + increment gesetzt wird, das scheint aber noch nicht zu klappen.