Skip to content

Commit 7a4ab0e

Browse files
committed
using a refactored code instead of a stub implementation
1 parent 8a1a8f5 commit 7a4ab0e

File tree

2 files changed

+13
-2
lines changed

2 files changed

+13
-2
lines changed

service/src/intTest/java/uk/nhs/adaptors/gp2gp/ehr/EhrResendControllerTest.java

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
import org.springframework.http.HttpStatus;
1515
import org.springframework.test.annotation.DirtiesContext;
1616
import org.springframework.test.context.junit.jupiter.SpringExtension;
17+
import uk.nhs.adaptors.gp2gp.common.service.RandomIdGeneratorService;
1718
import uk.nhs.adaptors.gp2gp.common.task.TaskDispatcher;
1819
import uk.nhs.adaptors.gp2gp.ehr.model.EhrExtractStatus;
1920
import uk.nhs.adaptors.gp2gp.gpc.GetGpcStructuredTaskDefinition;
@@ -42,13 +43,17 @@ public class EhrResendControllerTest {
4243
private static final Instant FIVE_DAYS_AGO = NOW.minus(Duration.ofDays(5));
4344
private static final String URI_TYPE = "https://fhir.nhs.uk/STU3/StructureDefinition/GPConnect-OperationOutcome-1";
4445
private static final String CONVERSATION_ID = "123-456";
46+
public static final String NHS_NUMBER = "12345";
4547

4648
@Autowired
4749
private EhrExtractStatusRepository ehrExtractStatusRepository;
4850

4951
@Autowired
5052
private EhrResendController ehrResendController;
5153

54+
@MockBean
55+
private RandomIdGeneratorService randomIdGeneratorService;
56+
5257
@MockBean
5358
private TaskDispatcher taskDispatcher;
5459

@@ -70,6 +75,7 @@ public void When_AnEhrExtractHasFailed_Expect_RespondsWith202() {
7075
.received(FIVE_DAYS_AGO)
7176
.rootId(generateRandomUppercaseUUID())
7277
.build());
78+
ehrExtractStatus.setEhrRequest(EhrExtractStatus.EhrRequest.builder().nhsNumber(NHS_NUMBER).build());
7379

7480
ehrExtractStatusRepository.save(ehrExtractStatus);
7581

@@ -97,12 +103,14 @@ public void When_AnEhrExtractHasFailed_Expect_GetGpcStructuredTaskScheduled() {
97103
.received(FIVE_DAYS_AGO)
98104
.rootId(generateRandomUppercaseUUID())
99105
.build());
106+
ehrExtractStatus.setEhrRequest(EhrExtractStatus.EhrRequest.builder().nhsNumber(NHS_NUMBER).build());
100107

101108
ehrExtractStatusRepository.save(ehrExtractStatus);
102109

103110
ehrResendController.scheduleEhrExtractResend(CONVERSATION_ID);
104111

105-
var taskDefinition = GetGpcStructuredTaskDefinition.builder().build();
112+
var taskDefinition = GetGpcStructuredTaskDefinition.getGetGpcStructuredTaskDefinition(randomIdGeneratorService,
113+
ehrExtractStatus);
106114
verify(taskDispatcher, times(1)).createTask(taskDefinition);
107115

108116
}

service/src/main/java/uk/nhs/adaptors/gp2gp/ehr/EhrResendController.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
import org.springframework.web.bind.annotation.PostMapping;
1414
import org.springframework.web.bind.annotation.RequestMapping;
1515
import org.springframework.web.bind.annotation.RestController;
16+
import uk.nhs.adaptors.gp2gp.common.service.RandomIdGeneratorService;
1617
import uk.nhs.adaptors.gp2gp.common.task.TaskDispatcher;
1718
import uk.nhs.adaptors.gp2gp.ehr.model.EhrExtractStatus;
1819
import uk.nhs.adaptors.gp2gp.gpc.GetGpcStructuredTaskDefinition;
@@ -30,6 +31,7 @@ public class EhrResendController {
3031

3132
private EhrExtractStatusRepository ehrExtractStatusRepository;
3233
private TaskDispatcher taskDispatcher;
34+
private RandomIdGeneratorService randomIdGeneratorService;
3335

3436
@PostMapping("/{conversationId}")
3537
public ResponseEntity<OperationOutcome> scheduleEhrExtractResend(@PathVariable String conversationId) {
@@ -51,7 +53,8 @@ public ResponseEntity<OperationOutcome> scheduleEhrExtractResend(@PathVariable S
5153
return new ResponseEntity<>(operationOutcome, HttpStatus.NOT_FOUND);
5254
}
5355

54-
var taskDefinition = GetGpcStructuredTaskDefinition.builder().build();
56+
var taskDefinition = GetGpcStructuredTaskDefinition.getGetGpcStructuredTaskDefinition(randomIdGeneratorService,
57+
ehrExtractStatus.get());
5558
taskDispatcher.createTask(taskDefinition);
5659

5760
return new ResponseEntity<>(HttpStatus.ACCEPTED);

0 commit comments

Comments
 (0)