From f153e31857ff5b89887964d6819a4a11c533cc30 Mon Sep 17 00:00:00 2001 From: miferreiro Date: Thu, 21 Feb 2019 17:51:50 +0100 Subject: [PATCH] Fixed bugs - The authentication header has been added to the test of modifying a person's name. - Updated the name of the PetDAO class to PetsDAO in the Pets resource. --- .../java/es/uvigo/esei/daa/rest/PetsResource.java | 13 +++++-------- .../es/uvigo/esei/daa/rest/PeopleResourceTest.java | 4 ++-- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/src/main/java/es/uvigo/esei/daa/rest/PetsResource.java b/src/main/java/es/uvigo/esei/daa/rest/PetsResource.java index 0edc9bf..a7209fe 100644 --- a/src/main/java/es/uvigo/esei/daa/rest/PetsResource.java +++ b/src/main/java/es/uvigo/esei/daa/rest/PetsResource.java @@ -16,7 +16,7 @@ import javax.ws.rs.core.MediaType; import javax.ws.rs.core.Response; import es.uvigo.esei.daa.dao.DAOException; -import es.uvigo.esei.daa.dao.PetDAO; +import es.uvigo.esei.daa.dao.PetsDAO; import es.uvigo.esei.daa.entities.Person; import es.uvigo.esei.daa.entities.Pet; @@ -30,17 +30,17 @@ import es.uvigo.esei.daa.entities.Pet; public class PetsResource { private final static Logger LOG = Logger.getLogger(PetsResource.class.getName()); - private final PetDAO dao; + private final PetsDAO dao; /** * Constructs a new instance of {@link PetResource}. */ public PetsResource() { - this(new PetDAO()); + this(new PetsDAO()); } // Needed for testing purposes - PetsResource(PetDAO dao) { + PetsResource(PetsDAO dao) { this.dao = dao; } @@ -188,10 +188,7 @@ public class PetsResource { .build(); } } - - - - + /** * Deletes a pet from the system. * diff --git a/src/test/java/es/uvigo/esei/daa/rest/PeopleResourceTest.java b/src/test/java/es/uvigo/esei/daa/rest/PeopleResourceTest.java index 1341be8..3c0c3f1 100644 --- a/src/test/java/es/uvigo/esei/daa/rest/PeopleResourceTest.java +++ b/src/test/java/es/uvigo/esei/daa/rest/PeopleResourceTest.java @@ -212,8 +212,8 @@ public class PeopleResourceTest extends JerseyTest { final Form form = new Form(); form.param("name", newName()); - final Response response = target("people/" + existentId()) - .request(MediaType.APPLICATION_JSON_TYPE) + final Response response = target("people/" + existentId()).request(MediaType.APPLICATION_JSON_TYPE) + .header("Authorization", "Basic " + userToken(adminLogin())) .put(entity(form, MediaType.APPLICATION_FORM_URLENCODED_TYPE)); assertThat(response, hasBadRequestStatus()); -- 2.18.1