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

Supertest-Tests vervollständigen #3

Open tthelen opened 10 years ago

tthelen commented 10 years ago

TODOs:

Ziel:

grunt test -> Alle vorhandenen Tests werden abgearbeitet und reportet

Verschiedene Targets:

tthelen commented 10 years ago
TiloW commented 10 years ago

Schnittstelle zum Einbinden eines isolierten Servers existiert jetzt. siehe test/functional/node_modules/server_setup.coffee

nmeyering commented 10 years ago

Cool, danke.

TiloW commented 10 years ago

Ich hab noch ne kleine Änderungen an den POST/PUT Tests vorgenommen, du hast den Puffer anstelle des JSON-Strings übermittelt. Außerdem muss auch im Request der Content-Type deklariert werden (sonst 400: Bad Request), bin mir allerdings nicht sicher ob das in der xAPI auch so spezifiziert ist? Wir könnten gegebenenfalls im Request den Content-Type ignorieren.. hässlich aber vll etwas robuster.

TiloW commented 10 years ago

@nmeyering Die Namen von Header-Feldern sind case-insensitive, bitte beachten!

TiloW commented 10 years ago

Weshalb verwenden wir eigentlich diese Verzeichnisstruktur? Weshalb heißt jede Datei innerhalb des functional-Verzeichnis test.coffee?

nmeyering commented 10 years ago

Die Verzeichnisstruktur bildet die Struktur der Spec wieder, und so wenige Dateien hauptsaechlich aus Faulheit. Sonst muss man ueberall die ganzen requires einzeln hinschreiben, aber die relativen Pfade aendern sich ja...

TiloW commented 10 years ago

Das mit dem require könnte man ja gerade noch verkraften... Aber wieso jedesmal ein Ordner für eine einzelne Datei namens test.coffee?

Ist auch nicht so wichtig, habe nur gerade überlegt wenn wir die Integration-Tests (z.B. für OAuth) anlegen und ob das nach dem gleichen Schema gemacht werden sollte.