Dev/#13

[Jankkol]

Usunąłem stary test, dodałem nowy który sprawdza czy jest połączenie. Insert, delete, find, to myślę, że zrobimy test jak już będziemy mieć jakąś encje np user. Nie chcę robić w serwerze sztucznych schematów/tabel i metod tylko do testowania połączenia.

[Jankkol]

Kurde, mam jakiś problem z tym testem. Pisał ktoś kiedyś test js gdzie oczekuje wyjątku ? Wyskrobałem coś takiego i modyfikacje tego:

describe("database mongodb connection - check result if error", function () { it("connected to db failure", function (done) { var err = new Error("Database connection error"); expect(serwer.checkDbConnection(err)).to.throw(err); done(); }); });

ale dalej nie przechodzi.

[Maciekek]

A zobacz tak: ....toThrow(new Error(err));

[Jankkol]

1) database mongodb connection - check result if error connected to db failure Message: Error: Database connection error Stacktrace: Error: Database connection error at null. (C:\workspace\projZesp\ProjektZespolowy2015\WebApp\test\serwer_unit\dbConnectionTest.spec.js:14:19) at null. (C:\workspace\projZesp\ProjektZespolowy2015\WebApp\node_modules\grunt-jasmine-node\node_modules\jasmine-node\lib\jasmine-node\async-callback.js:45:37)

2014-10-28 21:24 GMT+01:00 Maciej Kucharski notifications@github.com:

A zobacz tak: ....toThrow(new Error(err));

— Reply to this email directly or view it on GitHub https://github.com/Maciekek/ProjektZespolowy2015/pull/29#issuecomment-60825094 .

[Maciekek]

commitnij to co masz tam, zobaczę w wolnej chwili

2014-10-28 21:28 GMT+01:00 Jan Kąkol notifications@github.com:

1) database mongodb connection - check result if error connected to db failure Message: Error: Database connection error Stacktrace: Error: Database connection error at null. (C:\workspace\projZesp\ProjektZespolowy2015\WebApp\test\serwer_unit\dbConnectionTest.spec.js:14:19)

at null. (C:\workspace\projZesp\ProjektZespolowy2015\WebApp\node_modules\grunt-jasmine-node\node_modules\jasmine-node\lib\jasmine-node\async-callback.js:45:37)

2014-10-28 21:24 GMT+01:00 Maciej Kucharski notifications@github.com:

A zobacz tak: ....toThrow(new Error(err));

— Reply to this email directly or view it on GitHub < https://github.com/Maciekek/ProjektZespolowy2015/pull/29#issuecomment-60825094>

.

— Reply to this email directly or view it on GitHub https://github.com/Maciekek/ProjektZespolowy2015/pull/29#issuecomment-60825641 .

Pozdrawiam, Maciej Kucharski

[Jankkol]

Ok wrzuciłem to na gita, branch dev/#13

2014-10-28 21:31 GMT+01:00 Maciej Kucharski notifications@github.com:

commitnij to co masz tam, zobaczę w wolnej chwili

2014-10-28 21:28 GMT+01:00 Jan Kąkol notifications@github.com:

1) database mongodb connection - check result if error connected to db failure Message: Error: Database connection error Stacktrace: Error: Database connection error at null.

(C:\workspace\projZesp\ProjektZespolowy2015\WebApp\test\serwer_unit\dbConnectionTest.spec.js:14:19)

at null.

(C:\workspace\projZesp\ProjektZespolowy2015\WebApp\node_modules\grunt-jasmine-node\node_modules\jasmine-node\lib\jasmine-node\async-callback.js:45:37)

2014-10-28 21:24 GMT+01:00 Maciej Kucharski notifications@github.com:

A zobacz tak: ....toThrow(new Error(err));

— Reply to this email directly or view it on GitHub <

https://github.com/Maciekek/ProjektZespolowy2015/pull/29#issuecomment-60825094>

.

— Reply to this email directly or view it on GitHub < https://github.com/Maciekek/ProjektZespolowy2015/pull/29#issuecomment-60825641>

.

Pozdrawiam, Maciej Kucharski

— Reply to this email directly or view it on GitHub https://github.com/Maciekek/ProjektZespolowy2015/pull/29#issuecomment-60826074 .

[Maciekek]

ciekawe jest to, że to nie działa. NIe mam uruchomionego mongo a aplikacja i tak i tak nie rzuca błędem

[Maciekek]

To w ten sposób nie zadziała. Ta funkcja łącząca z bazą nie zwraca tego co byśmy chcieli, a właściwie nie zwraca nic, czyli undefined. Dlatego test na łączenie się z bazą przechodził bo tak była sformułowana logika if`a.

[Maciekek]

Zmieniłem troche sposób działania tej funkcji. Wyrzuciłem rzucanie błędu, bo nie jestem w stanie tego przetestować. NIe działa to tak jakbyśmy chcieli. Przejrzyjcie to i zobaczycie czy akceptujecie, a no i testy

[Jankkol]

Dla mnie spoko