Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SSCSCI-1179 #4075

Open
wants to merge 42 commits into
base: SSCSCI-1128
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
fd03bf0
defs update
nilay913 Oct 14, 2024
cf43f91
Merge branch 'master' into SSCSCI-1123-defs
nilay913 Oct 14, 2024
396af8a
defs update
nilay913 Oct 14, 2024
301d8f6
defs update
nilay913 Oct 14, 2024
75db3c1
Merge branch 'master' into SSCSCI-1123-defs
nilay913 Oct 14, 2024
9c9aa19
getting a basic in uk iba appeal submitted successfully to exui previ…
nilay913 Oct 14, 2024
c60a949
updating commons
nilay913 Oct 14, 2024
9bfba5e
fixing sonar issues
nilay913 Oct 15, 2024
61e86e2
pointing preview values to relevant sya pr
nilay913 Oct 15, 2024
3378d56
TEMP reverting port of entry back to fixed list
nilay913 Oct 15, 2024
31b4eee
Merge branch 'master' into SSCSCI-1179
nilay913 Oct 15, 2024
ee1e48b
Revert "TEMP reverting port of entry back to fixed list"
nilay913 Oct 15, 2024
1982521
Merge branch 'master' into SSCSCI-1179
nilay913 Oct 17, 2024
1390ab7
TEMP pointing to commons
nilay913 Oct 17, 2024
567c755
getting SYA to submit successfully with in uk or out of uk IBA appeals
nilay913 Oct 17, 2024
cc7bcfe
updating TODO info and test for coverage
nilay913 Oct 17, 2024
e1daecf
updating coverage
nilay913 Oct 17, 2024
3db2d1a
sonar fixes
nilay913 Oct 18, 2024
b9946dd
coverage upping
nilay913 Oct 18, 2024
11c54b9
unit test coverage and fix
nilay913 Oct 18, 2024
3d39cc5
checkstyle fix
nilay913 Oct 18, 2024
a1e5518
resolving master conflicts
nilay913 Oct 18, 2024
164b6c7
Merge branch 'master' into SSCSCI-1179
nilay913 Oct 18, 2024
b1ea0ee
helping flakey functional tests
nilay913 Oct 18, 2024
5a14a34
helping flakey functional tests
nilay913 Oct 18, 2024
e9c88d1
helping flakey functional tests
nilay913 Oct 18, 2024
3940afb
Merge branch 'master' into SSCSCI-1179
nilay913 Oct 18, 2024
06d54b4
potential functional fix
nilay913 Oct 18, 2024
8bb4cc9
Merge branch 'master' into SSCSCI-1179
nilay913 Oct 21, 2024
2ce8edf
allowing representative address in/out of Uk handling
nilay913 Oct 21, 2024
2f9625f
replacing some instances of Infected Blood Appeal with Infected Blood…
nilay913 Oct 21, 2024
cedbc92
showing nino / ibca ref depending on case type and validating in upda…
nilay913 Oct 21, 2024
f8e4a12
checkstyle fix
nilay913 Oct 21, 2024
936efc1
Merge branch 'master' into SSCSCI-1179
nilay913 Oct 22, 2024
48a1b0c
pulling in 1128 changes
nilay913 Oct 22, 2024
6b43a70
Merge branch 'SSCSCI-1128' into SSCSCI-1179
nilay913 Oct 22, 2024
40642db
test fixes
nilay913 Oct 22, 2024
f1f5644
replacing SC case number with IBCA reference number for iba in summar…
nilay913 Oct 22, 2024
b6cc474
checkstyle fix
nilay913 Oct 22, 2024
f949ddb
disabling hearings-api for build
nilay913 Oct 22, 2024
1d9ed68
upgrading commons
nilay913 Oct 22, 2024
00abd4f
temp for build
nilay913 Oct 22, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 8 additions & 6 deletions Jenkinsfile_CNP
Original file line number Diff line number Diff line change
Expand Up @@ -39,12 +39,12 @@ def secrets = [
secret('evidence-share-topic-shared-access-key', 'AMQP_PASSWORD'),

// hearing-api
secret('idam-s2s-api', 'IDAM_S2S_URL'),
secret('caseworker-ref-api', 'CASEWORKER_REF_API_URL'),
secret('ccd-api', 'CORE_CASE_DATA_API_URL'),
secret('judicial-ref-api', 'JUDICIAL_REF_API_URL'),
secret('s2s-micro', 'IDAM_S2S_AUTH_MICROSERVICES'),
secret('common-ref-data-api', 'COMMON_REF_API_URL'),
// secret('idam-s2s-api', 'IDAM_S2S_URL'),
// secret('caseworker-ref-api', 'CASEWORKER_REF_API_URL'),
// secret('ccd-api', 'CORE_CASE_DATA_API_URL'),
// secret('judicial-ref-api', 'JUDICIAL_REF_API_URL'),
// secret('s2s-micro', 'IDAM_S2S_AUTH_MICROSERVICES'),
// secret('common-ref-data-api', 'COMMON_REF_API_URL'),

// e2es
secret('test-e2e-caseofficer-username', 'TEST_CASEOFFICER_USERNAME'),
Expand Down Expand Up @@ -207,6 +207,7 @@ def setAatEnvVars() {
env.PREVIEW_BRANCH_WITH_LOCAL_CCD = false;
env.TEST_E2E_URL_WEB = "https://manage-case.aat.platform.hmcts.net/"
env.TEST_E2E_API_URI = "http://sscs-tribunals-api-aat.service.core-compute-aat.internal"
env.TEST_URL = "http://sscs-tribunals-api-aat.service.core-compute-aat.internal"
env.CCD_API_URL = "http://ccd-data-store-api-aat.service.core-compute-aat.internal"
env.CORE_CASE_DATA_API_URL = "http://ccd-data-store-api-aat.service.core-compute-aat.internal"
env.CCD_GW_API = "https://gateway-ccd.aat.platform.hmcts.net"
Expand All @@ -218,6 +219,7 @@ def setPreviewEnvVars() {
env.PREVIEW_BRANCH_WITH_LOCAL_CCD = true;
env.TEST_E2E_URL_WEB = "https://xui-sscs-tribunals-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
env.TEST_E2E_API_URI = "https://sscs-tribunals-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
env.TEST_URL = "https://sscs-tribunals-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
env.CCD_API_URL = "https://ccd-data-store-api-sscs-tribunals-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
env.CORE_CASE_DATA_API_URL = "https://ccd-data-store-api-sscs-tribunals-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
env.CCD_GW_API = "http://sscs-tribunals-api-pr-${CHANGE_ID}-ccd-api-gw"
Expand Down
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -351,7 +351,7 @@ dependencies {
implementation group: 'org.flywaydb', name: 'flyway-core', version: '9.22.3'
implementation group: 'org.postgresql', name: 'postgresql', version: '42.6.0'

implementation group: 'com.github.hmcts', name: 'sscs-common', version: '5.12.8'
implementation group: 'com.github.hmcts', name: 'sscs-common', version: '5.12.9'

implementation group: 'com.github.hmcts', name: 'sscs-pdf-email-common', version: '5.2.14'

Expand Down
6 changes: 3 additions & 3 deletions charts/sscs-tribunals-api/values.preview.template.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -538,7 +538,7 @@ sscs-tribunals-frontend:
nodejs:
memoryLimits: "2Gi"
cpuLimits: 2000m
image: hmctspublic.azurecr.io/sscs/tribunals-frontend:latest
image: hmctspublic.azurecr.io/sscs/tribunals-frontend:pr-1548
ingressHost: sya-${SERVICE_FQDN}
registerAdditionalDns:
enabled: true
Expand Down Expand Up @@ -583,9 +583,9 @@ redis:
#Turning either hearings api (with either listener on) or cft hearing service on requires
#service bus to be turned on.
sscs-hearings-api:
enabled: true
enabled: false
java:
image: "hmctspublic.azurecr.io/sscs/hearings-api:pr-646" #Change to pr if required
image: "hmctspublic.azurecr.io/sscs/hearings-api:latest" #Change to pr if required
releaseNameOverride: ${SERVICE_NAME}-sscs-hearings-api
ingressHost: hearings-${SERVICE_FQDN}
environment:
Expand Down
5 changes: 3 additions & 2 deletions definitions/benefit/sheets/CaseTypeTab/CaseTypeTab.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "urgentCase", "TabFieldDisplayOrder": 9},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryName", "TabFieldDisplayOrder": 10},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryAddress", "TabFieldDisplayOrder": 11},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryNino", "TabFieldDisplayOrder": 12},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryNino", "FieldShowCondition": "appeal.benefitType.code!=\"infectedBloodAppeal\"", "TabFieldDisplayOrder": 12},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryDOB", "TabFieldDisplayOrder": 13},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryRole", "FieldShowCondition": "appeal.appellant.role.name!=\"\"", "TabFieldDisplayOrder": 14},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryRoleDescription", "FieldShowCondition": "appeal.appellant.role.description!=\"\"", "TabFieldDisplayOrder": 15},
Expand All @@ -33,7 +33,8 @@
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "confidentialityRequestOutcomeJointParty", "TabFieldDisplayOrder": 26},

{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryBenefitType", "TabFieldDisplayOrder": 27},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryCaseReference", "TabFieldDisplayOrder": 28},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryCaseReference", "FieldShowCondition": "appeal.benefitType.code!=\"infectedBloodAppeal\"", "TabFieldDisplayOrder": 28},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryIbcaReference", "FieldShowCondition": "appeal.benefitType.code=\"infectedBloodAppeal\"", "TabFieldDisplayOrder": 28},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryState", "TabFieldDisplayOrder": 29},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryHearingType", "TabFieldDisplayOrder": 30},
{"LiveFrom": "", "CaseTypeID": "Benefit", "TabID": "summary_sscs", "TabLabel": "Summary", "TabDisplayOrder": 1, "CaseFieldID": "summaryConfidentialityRequired", "FieldShowCondition": "appeal.appellant.confidentialityRequired!=\"\"", "TabFieldDisplayOrder": 31},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,18 @@
import static org.hamcrest.Matchers.is;

import java.io.IOException;
import org.junit.Rule;
import org.junit.Test;
import uk.gov.hmcts.reform.sscs.ccd.domain.SscsCaseDetails;
import uk.gov.hmcts.reform.sscs.functional.tyanotifications.Retry;

public class SubscriptionTest extends BaseFunctionTest {
private static final String YES = "yes";
private static final String NO = "no";

@Rule
public Retry retry = new Retry(3);

@Test
public void shouldUpdateSubscription() throws IOException, InterruptedException {
String newUserEmail = createRandomEmail();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,10 @@ public AdminAppealWithdrawnNotificationsTest() {
}

@Rule
public Retry retry = new Retry(0);
public Retry retry = new Retry(3);

//Test method runs three times and in worst case it needs 90 seconds waiting time.
@Rule
public Timeout globalTimeout = Timeout.seconds(100);
public Timeout globalTimeout = Timeout.seconds(90);

@Before
public void setUp() {
Expand Down Expand Up @@ -63,12 +62,15 @@ public void givenCallbackWithSubscriptions_shouldSendEmailSmsAndLetterNotificati
}

private boolean fetchLetters(int expectedNumLetters, String subscription) {
int fetchCount = 0;
do {
if (getNumberOfLetterCorrespondence(subscription) == expectedNumLetters) {
return true;
}
fetchCount++;
delayInSeconds(5);
} while (true);
} while (fetchCount < 5);
return false;
}

private void initialiseCcdCase() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,15 @@
import java.util.stream.Collectors;
import junitparams.Parameters;
import org.junit.Ignore;
import org.junit.Rule;
import org.junit.Test;
import org.springframework.beans.factory.annotation.Value;
import uk.gov.hmcts.reform.sscs.ccd.domain.ReasonableAdjustmentStatus;
import uk.gov.hmcts.reform.sscs.ccd.domain.SscsCaseData;
import uk.gov.hmcts.reform.sscs.ccd.domain.SscsCaseDetails;
import uk.gov.hmcts.reform.sscs.ccd.domain.YesNo;
import uk.gov.hmcts.reform.sscs.functional.tyanotifications.AbstractFunctionalTest;
import uk.gov.hmcts.reform.sscs.functional.tyanotifications.Retry;
import uk.gov.service.notify.Notification;
import uk.gov.service.notify.NotificationClientException;

Expand Down Expand Up @@ -154,6 +156,9 @@ public NotificationsFunctionalTest() {
super(30);
}

@Rule
public Retry retry = new Retry(3);

@Test
public void shouldSendEvidenceReceivedNotification() throws NotificationClientException, IOException {
simulateCcdCallback(EVIDENCE_RECEIVED);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,17 @@ public class SyaAppellant {

private String nino;

private String ibcaReference;

@JsonProperty("contactDetails")
private SyaContactDetails contactDetails;

@JsonProperty("isAddressSameAsAppointee")
private Boolean isAddressSameAsAppointee;

@JsonProperty("ibcRole")
private String ibcRole;

public void setNino(String nino) {
this.nino = normaliseNino(nino);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,7 @@ public class SyaContactDetails {
private String emailAddress;
private String postcodeLookup;
private String postcodeAddress;
private String portOfEntry;
private String country;
private Boolean inMainlandUk;
}
Loading