Skip to content

Commit

Permalink
RESTWS-716: Orders resource returns no results when only status is sp…
Browse files Browse the repository at this point in the history
…ecified
  • Loading branch information
IamMujuziMoses committed Sep 17, 2023
1 parent 9a62219 commit 0f4083f
Showing 1 changed file with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -592,7 +592,7 @@ public void invalidCareCenterShouldThrowException() throws Exception {
}

@Test(expected = InvalidSearchException.class)
public void doSearch_shouldThrowExceptionIfNoPatientUuidIsSpecified() throws Exception {
public void doSearch_shouldReturnExceptionNotNullSetIfNoPatientUuidIsSpecified() throws Exception {
MockHttpServletRequest req = newGetRequest(getURI(),
new Parameter("status", "active")
);
Expand All @@ -607,12 +607,10 @@ public void shouldGetOrdersByPatientUuid() throws Exception {
// get all drug orders
List<Order> activeDrugOrders = orderService.getActiveOrders(patient, orderService.getOrderTypeByName("Drug order"),
null, null);
int activeDrugOrderCount = activeDrugOrders.size();

// get all test orders
List<Order> activeTestOrders = orderService.getActiveOrders(patient, orderService.getOrderTypeByName("Test order"),
null, null);
int activeTestOrderCount = activeTestOrders.size();

// order service should return all active orders with patientUuid specified
MockHttpServletRequest req = newGetRequest(getURI(),
Expand All @@ -621,7 +619,7 @@ public void shouldGetOrdersByPatientUuid() throws Exception {
);
SimpleObject orders = deserialize(handle(req));
ArrayList<Object> resp = (ArrayList<Object>) PropertyUtils.getProperty(orders, "results");
assertEquals(activeTestOrderCount + activeDrugOrderCount, resp.size());
assertEquals(activeTestOrders.size() + activeDrugOrders.size(), resp.size());

}
}

0 comments on commit 0f4083f

Please sign in to comment.