search

MehrCurry / blacklist-jee

0 stars 0 forks source link

PermissionResource findet den BlacklistEntry nicht #6

Closed MehrCurry closed 10 years ago

MehrCurry commented 10 years ago

Hmm, irgendwas ist komisch. Auch wenn die Tests mit dem EM Mock laufen, ein manueller Itegrationstest ist fehlgeschlagen. Obwohl ein entsprechender Eintrag angelegt worden ist, liefert die Resource trotzfem 204 zurück.

curl -X PUT -d '{"name":"ARGH","listedElements":["ARG","nooo","345"]}' --header 'Content-Type: application/json' --header 'Accept: application/json' "http://localhost:8080/blacklist-jee7/resources/blacklist"
$ curl -i http://localhost:8080/blacklist-jee7/resources/permission/ARGH/345
HTTP/1.1 204 No Content
Server: GlassFish Server Open Source Edition  4.1
X-Powered-By: Servlet/3.1 JSP/2.3 (GlassFish Server Open Source Edition  4.1  Java/Oracle Corporation/1.8)
Date: Mon, 20 Oct 2014 12:16:06 GMT
asiepert commented 10 years ago

Liegts vielleicht am scope vom service? Am 20.10.2014 14:17 schrieb "MehrCurry" notifications@github.com:

Hmm, irgendwas ist komisch. Auch wenn die Tests mit dem EM Mock laufen, ein manueller Itegrationstest ist fehlgeschlagen. Obwohl ein entsprechender Eintrag angelegt worden ist, liefert die Resource trotzfem 204 zurück.

curl -X PUT -d '{"name":"ARGH","listedElements":["ARG","nooo","345"]}' --header 'Content-Type: application/json' --header 'Accept: application/json' "http://localhost:8080/blacklist-jee7/resources/blacklist"

$ curl -i http://localhost:8080/blacklist-jee7/resources/permission/ARGH/345 HTTP/1.1 http://localhost:8080/blacklist-jee7/resources/permission/ARGH/345HTTP/1.1 204 No Content Server: GlassFish Server Open Source Edition 4.1 X-Powered-By: Servlet/3.1 JSP/2.3 (GlassFish Server Open Source Edition 4.1 Java/Oracle Corporation/1.8) Date: Mon, 20 Oct 2014 12:16:06 GMT

— Reply to this email directly or view it on GitHub https://github.com/MehrCurry/blacklist-jee/issues/6.

MehrCurry commented 10 years ago

Der Test und Methode gefixt. Die ResultList ist aber immer noch leer. Evtl. irgendwas mit den Transaktionen ...

asiepert commented 10 years ago

Der Code ist ok. Martins manueller Test befüllt die HashMap und will aus der DB etwas laden - nur da ist natürlich nichts drin Hinzukommt, dass er den type einmal mit und einmal ohne 'H' schreibt. Zur Zeit haben wir zwei Datenspeicher: die Hashmap und die DB.

Gute Nacht Andi

2014-10-20 22:07 GMT+02:00 MehrCurry notifications@github.com:

Der Test und Methode gefixt. Die ResultList ist aber immer noch leer. Evtl. irgendwas mit den Transaktionen ...

— Reply to this email directly or view it on GitHub https://github.com/MehrCurry/blacklist-jee/issues/6#issuecomment-59830940 .

AllYouCanEat commented 10 years ago

Ja, das PUT aus dem obersten Kommentar bewirkt eigentlich keine Persistierung - die Dual-Strategy ist da auch wirklich ein wenig verwirrend. Aber von welchem Test redet ihr? Ich habe jetzt 10 Minuten gesucht, aber auch meine ResultList ist immer noch leer ...

MehrCurry commented 10 years ago

Ich habe die HashMap Persistenz entfernt.