src/main/java/com/stileeducation/markr/controller/TestResultsController.java


1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89

package com.stileeducation.markr.controller;

import com.stileeducation.markr.dto.AggregatedTestResultsDTO;
import com.stileeducation.markr.dto.MCQTestResultDTO;
import com.stileeducation.markr.dto.MCQTestResultsDTO;
import com.stileeducation.markr.entity.Student;
import com.stileeducation.markr.entity.Test;
import com.stileeducation.markr.entity.TestResult;
import com.stileeducation.markr.repository.TestRepository;
import com.stileeducation.markr.repository.TestResultRepository;
import com.stileeducation.markr.service.StudentService;
import com.stileeducation.markr.service.TestResultsService;
import com.stileeducation.markr.service.TestService;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.http.HttpStatus;
import org.springframework.http.ResponseEntity;
import org.springframework.web.bind.annotation.*;

@RestController
@RequestMapping("/")
public class TestResultsController {

  public static final String IMPORT_ENDPOINT = "/import";
  public static final String AGGREGATE_ENDPOINT = "/results/{test-id}/aggregate";

  @Autowired
  private StudentService studentService;

  @Autowired
  private TestService testService;

  @Autowired
  private TestResultsService testResultsService;

  @Autowired
  private TestRepository testRepository;

  @Autowired
  private TestResultRepository testResultRepository;

  public TestResultsController(TestResultsService testResultsService) {
    this.testResultsService = testResultsService;
  }

  // TODO consider return value
  @PostMapping(value = IMPORT_ENDPOINT, consumes = "text/xml+markr", produces = "application/json")
  public ResponseEntity<Void> handleXmlRequest(@RequestBody MCQTestResultsDTO testResults) {

    for (MCQTestResultDTO mcqTestResult : testResults.getMcqTestResults()) {
      Student student = studentService
          .findOrCreateStudent(
              mcqTestResult.getFirstName(),
              mcqTestResult.getLastName(),
              mcqTestResult.getStudentNumber());

      Test test = testService
          .findOrCreateTest(
              mcqTestResult.getTestId(),
              mcqTestResult.getSummaryMarks().getAvailable());

      if (test.getMarksAvailable() < mcqTestResult.getSummaryMarks().getAvailable()) {
        test.setMarksAvailable(mcqTestResult.getSummaryMarks().getAvailable());
        testRepository.save(test);
      }

      // Some edge cases to consider
      // obtained is higher than available (assumption?)

      TestResult testResult = testResultsService
          .findOrCreateTestResult(
              student.getId(),
              test.getId(),
              mcqTestResult.getSummaryMarks().getObtained());

      if (testResult.getMarksAwarded() < mcqTestResult.getSummaryMarks().getObtained()) {
        testResult.setMarksAwarded(mcqTestResult.getSummaryMarks().getObtained());
        testResultRepository.save(testResult);
      }
    }

    return new ResponseEntity<>(HttpStatus.NO_CONTENT);
  }

  @GetMapping(value = AGGREGATE_ENDPOINT, produces = "application/json")
  public AggregatedTestResultsDTO getAggregatedResults(@PathVariable("test-id") String testId) {
    return testResultsService.aggregateTestResults(testId);
  }

}