diff --git a/src/main/java/org/wise/vle/web/CRaterController.java b/src/main/java/org/wise/vle/web/CRaterController.java index 13d965fb5..9e2e85b38 100644 --- a/src/main/java/org/wise/vle/web/CRaterController.java +++ b/src/main/java/org/wise/vle/web/CRaterController.java @@ -54,8 +54,8 @@ HashMap scoreItem(@RequestBody CRaterScoringRequest request) { response.put("score", cRaterResponse.getScore()); } else { response.put("scores", cRaterResponse.getScores()); - response.put("ideas", cRaterResponse.getIdeas()); } + response.put("ideas", cRaterResponse.getIdeas()); response.put("cRaterResponse", cRaterResponse.getResponse()); return response; } diff --git a/src/test/java/org/wise/vle/web/CRaterControllerTest.java b/src/test/java/org/wise/vle/web/CRaterControllerTest.java index 025d282ad..5f7877be2 100644 --- a/src/test/java/org/wise/vle/web/CRaterControllerTest.java +++ b/src/test/java/org/wise/vle/web/CRaterControllerTest.java @@ -68,12 +68,15 @@ public void verifyItemId_InvalidId_ReturnFalse() { } @Test - public void scoreItem_SingleScoreItem_ReturnScore() { + @SuppressWarnings("unchecked") + public void scoreItem_SingleScoreItem_ReturnScoreAndIdeas() { CRaterScoringRequest request = new CRaterScoringRequest(); String cRaterXmlResponse = "" + "" + "" + - "0" + + "0" + + "" + + "" + ""; CRaterScoringResponse cRaterResponse = new CRaterScoringResponse(cRaterXmlResponse); expect(CRaterHttpClient.getScoringResponse(request)).andReturn(cRaterResponse); @@ -83,6 +86,10 @@ public void scoreItem_SingleScoreItem_ReturnScore() { assertTrue(scoreItemResponse.containsKey("score")); assertEquals(1, scoreItemResponse.get("score")); assertEquals(cRaterXmlResponse, scoreItemResponse.get("cRaterResponse")); + assertTrue(scoreItemResponse.containsKey("ideas")); + List ideas = (List) scoreItemResponse.get("ideas"); + assertEquals(1, ideas.size()); + assertTrue(ideas.get(0).isDetected()); } @Test