search

webtech-uos / nodejs-lrs

node.js implementation of a learning record store (LRS) as described in the XAPI specifications
MIT License
9 stars 7 forks source link

DB-Modellierung: Views erstellen #11

Closed tthelen closed 10 years ago

tthelen commented 10 years ago
brandt-codes commented 10 years ago

Verständnis-Frage: "Standarddaten für Tests bereitlegen" und "Tests erweitern" verstehe ich noch nicht so ganz. Was genau soll da passieren? Blicke noch nicht so richtig durch, sorry.

tthelen commented 10 years ago
TiloW commented 10 years ago

@brajos Möglicherweise reicht es auch aus, Unit-Tests für die Datenbank-Mapper (wie z.B. StatementMapper) zu implementieren. Das sollte meiner Meinung nach sowieso gemacht werden.

Weil die Modellierung der Datenbank nur so schleppend vorankam, Sebastian und ich aber auf die Datenbank angewiesen sind, haben wir jetzt erstmal eine Modellierung erstellt. Als Ausgangspunkt dafür wurde das Skript zur DB-Initialisierung von Artem verwendet.

spuetz commented 10 years ago

Ich denke das ist natürlich à la Test-Driven Development eine gute Sache, doch denke ich, dass das nicht wirklich nötig ist und vielleicht sogar überflüssig ist. Laufen allen anderen Tests für REST und co durch ist damit auch sicher gestellt, dass der DB-Mapper (StatementMapper) funktioniert. Vielleicht kann man da die Zeit besser nutzen um das Projekt und die anderen Tests für z.B. voidedStatements, Authentifizierung und und und zu schreiben. Was denkt ihr?

artpetro commented 10 years ago

Mir erscheinen die Tests zu DB-Initialisirung auch als überflussig.

TiloW commented 10 years ago

Aufgabe des StatementMapper ist nicht Initialisierung sondern Modellierung der Statements.

tthelen commented 10 years ago

Noch offen:

TiloW commented 10 years ago

Einfügen von mehreren statements mittels bulk-insertion.