Skip to content

Commit 1f27f41

Browse files
tomxelliottTom Elliott
andauthored
DTSSTCI-864 (#1802)
* DTSSTCI-864: Add date uploaded for any docs added through update case * DTSSTCI-864: Remove sout --------- Co-authored-by: Tom Elliott <tomelliott@Toms-MacBook-Pro.local>
1 parent 0f15b8f commit 1f27f41

File tree

3 files changed

+20
-1
lines changed

3 files changed

+20
-1
lines changed

src/functionalTest/resources/responses/response-dss-update-case-submission-about-to-submit.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,8 @@
3838
"document_filename": "mockFileUpdated.pdf",
3939
"document_binary_url": "http://dm-store-aat.service.core-compute-aat.internal/documents/328c674f-59c3-418a-b76b-78f2cc52d21a/binary",
4040
"category_id": "DSS"
41-
}
41+
},
42+
"date": "${json-unit.any-string}"
4243
}
4344
}
4445
],

src/main/java/uk/gov/hmcts/sptribs/citizen/event/CicDssUpdateCaseEvent.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,7 @@ private CaseData addDocumentsToCaseData(final CaseData caseData, final DssCaseDa
114114
.documentLink(document)
115115
.documentEmailContent(documentComment)
116116
.documentCategory(DocumentType.DSS_OTHER)
117+
.date(LocalDate.now())
117118
.build();
118119
if (!documentList.contains(caseworkerCICDocument)) {
119120
documentList.add(caseworkerCICDocument);

src/test/java/uk/gov/hmcts/sptribs/citizen/event/CicDssUpdateCaseEventTest.java

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import uk.gov.hmcts.sptribs.notification.exception.NotificationException;
2727
import uk.gov.hmcts.sptribs.testutil.TestDataHelper;
2828

29+
import java.time.LocalDate;
2930
import java.util.ArrayList;
3031
import java.util.List;
3132

@@ -103,6 +104,10 @@ void shouldAddDocumentsAndMessagesToCaseDataInAboutToSubmitCallback() {
103104

104105
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded()).isNotEmpty();
105106
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded()).hasSize(3);
107+
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded().get(1).getValue().getDate())
108+
.isEqualTo(LocalDate.now());
109+
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded().get(2).getValue().getDate())
110+
.isEqualTo(LocalDate.now());
106111
assertThat(response.getData().getMessages()).isNotEmpty();
107112
assertThat(response.getData().getMessages()).hasSize(2);
108113
assertThat(response.getData().getDssCaseData().getOtherInfoDocuments()).isEmpty();
@@ -143,6 +148,10 @@ void shouldAddDocumentsOnlyToCaseDataInAboutToSubmitCallbackIfAdditionalInfoFiel
143148

144149
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded()).isNotEmpty();
145150
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded()).hasSize(3);
151+
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded().get(1).getValue().getDate())
152+
.isEqualTo(LocalDate.now());
153+
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded().get(2).getValue().getDate())
154+
.isEqualTo(LocalDate.now());
146155
assertThat(response.getData().getMessages()).isNotEmpty();
147156
assertThat(response.getData().getMessages()).hasSize(1);
148157
assertThat(response.getData().getDssCaseData().getOtherInfoDocuments()).isEmpty();
@@ -183,6 +192,10 @@ void shouldAddDocumentsOnlyToCaseDataInAboutToSubmitCallbackIfAdditionalInfoFiel
183192

184193
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded()).isNotEmpty();
185194
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded()).hasSize(3);
195+
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded().get(1).getValue().getDate())
196+
.isEqualTo(LocalDate.now());
197+
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded().get(2).getValue().getDate())
198+
.isEqualTo(LocalDate.now());
186199
assertThat(response.getData().getMessages()).isNotEmpty();
187200
assertThat(response.getData().getMessages()).hasSize(1);
188201
assertThat(response.getData().getDssCaseData().getOtherInfoDocuments()).isEmpty();
@@ -212,6 +225,10 @@ void shouldCreateNewApplicantDocumentsUploadedAndMessagesListIfEmptyInAboutToSub
212225

213226
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded()).isNotEmpty();
214227
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded()).hasSize(2);
228+
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded().get(0).getValue().getDate())
229+
.isEqualTo(LocalDate.now());
230+
assertThat(response.getData().getCicCase().getApplicantDocumentsUploaded().get(1).getValue().getDate())
231+
.isEqualTo(LocalDate.now());
215232
assertThat(response.getData().getMessages()).isNotEmpty();
216233
assertThat(response.getData().getMessages()).hasSize(1);
217234
assertThat(response.getData().getDssCaseData().getOtherInfoDocuments()).isEmpty();

0 commit comments

Comments
 (0)