#13 Tests de controlador de preguntas, bug arreglado

Arquisoft · Apr 26, 2024 · 29141c5 · 29141c5
1 parent 571eca2
commit 29141c5
Show file tree

Hide file tree

Showing 94 changed files with 5 additions and 3,145 deletions.
diff --git a/syg-backend/SYG-rest-controller/src/main/java/syg/controller/QuestionsController.java b/syg-backend/SYG-rest-controller/src/main/java/syg/controller/QuestionsController.java
@@ -23,13 +23,13 @@ public class QuestionsController {
 	private QuestionService questionService;
 
     @GetMapping
-    public ResponseEntity<Object> findAll() {
+    public ResponseEntity<Object> findAll(@RequestParam(name = "maxNumberOfQuestions", defaultValue = "15", required = false) int maxNumberOfQuestions) {
     	List<Question> questions = questionService.findAll();
     	if(questions.size() <= 0) {
     		return ResponseEntity.status(HttpStatus.OK).body(questions);
     	}
     	Collections.shuffle(questions);
-    	return ResponseEntity.status(HttpStatus.OK).body(questions.subList(0, Math.min(questions.size(), 15)));
+    	return ResponseEntity.status(HttpStatus.OK).body(questions.subList(0, Math.min(questions.size(), maxNumberOfQuestions)));
     }
 
     @GetMapping("/id")
@@ -44,12 +44,13 @@ public ResponseEntity<Object> findById(@RequestParam(name = "id") Long id) {
 		}
     }
     @GetMapping("/category")
-    public ResponseEntity<Object> findByCategory(@RequestParam(name = "categoryId") Long categoryId) {
+    public ResponseEntity<Object> findByCategory(@RequestParam(name = "categoryId") Long categoryId,
+    		@RequestParam(name = "maxNumberOfQuestions", defaultValue = "15", required = false) int maxNumberOfQuestions) {
     	List<Question> questions = questionService.findByCategory(categoryId);
     	if(questions.size() <= 0) {
     		return ResponseEntity.status(HttpStatus.OK).body(questions);
     	}
     	Collections.shuffle(questions);
-    	return ResponseEntity.status(HttpStatus.OK).body(questions.subList(0, Math.min(questions.size(), 15)));    		
+    	return ResponseEntity.status(HttpStatus.OK).body(questions.subList(0, Math.min(questions.size(), maxNumberOfQuestions)));    		
     }
 }
diff --git a/syg-backend/coverage/SYG-domain/index.html b/syg-backend/coverage/SYG-domain/index.html
diff --git a/syg-backend/coverage/SYG-domain/syg.domain.exception/ConflictException.html b/syg-backend/coverage/SYG-domain/syg.domain.exception/ConflictException.html
diff --git a/syg-backend/coverage/SYG-domain/syg.domain.exception/ConflictException.java.html b/syg-backend/coverage/SYG-domain/syg.domain.exception/ConflictException.java.html
diff --git a/syg-backend/coverage/SYG-domain/syg.domain.exception/NotFoundException.html b/syg-backend/coverage/SYG-domain/syg.domain.exception/NotFoundException.html
diff --git a/syg-backend/coverage/SYG-domain/syg.domain.exception/NotFoundException.java.html b/syg-backend/coverage/SYG-domain/syg.domain.exception/NotFoundException.java.html
diff --git a/syg-backend/coverage/SYG-domain/syg.domain.exception/index.html b/syg-backend/coverage/SYG-domain/syg.domain.exception/index.html