Skip to content

Commit 251f893

Browse files
Thor-tech-of-metalOllie-Smithshravanmechinenidavejones74
authored
Pay it feature (#926)
* Revert "PAY-8067 removed PayIt journey" * Revert "PAY-7934 remove RefundWhenContacted (#859)" This reverts commit 674de01. * Revert "Revert "PAY-7934 remove RefundWhenContacted (#859)"" This reverts commit 325f40f. * Revert "PAY-7934 remove RefundWhenContacted (#859)" This reverts commit 674de01. * PAY-8182 added Expired refund status to the liquibase scripts. * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * PAY-8185 added backend for PayIt * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * PAY-8185 added backend for PayIt * PAY-8185 added backend for PayIt * PAY-8185 added backend for PayIt - checkstyle correction * PAY-8185 added backend for PayIt - checkstyle correction * PAY-8185 added backend for PayIt - checkstyle correction * PAY-8185 added backend for PayIt - checkstyle correction * PAY-8185 added backend for PayIt - checkstyle correction * PAY-8185 added backend for PayIt - checkstyle correction * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * WIP https://tools.hmcts.net/jira/browse/PAY-8191 * PAY-8185 made changes to status histories. * WIP https://tools.hmcts.net/jira/browse/PAY-8219 * WIP https://tools.hmcts.net/jira/browse/PAY-8219 * WIP https://tools.hmcts.net/jira/browse/PAY-8219 * WIP https://tools.hmcts.net/jira/browse/PAY-8219 * PAY-8185 further changes for the expired refund from Liberata * PAY-8185 further changes checkstyle fixes * PAY-8185 further changes checkstyle fixes * PAY-8185 further changes checkstyle fixes * PAY-8185 further changes removal of liquibase = false * WIP https://tools.hmcts.net/jira/browse/PAY-8219 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * PAY-8201 * PAY-8201 checkstyles * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * PAY-8201 checkstyles * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * PAY-8201 checkstyles * PAY-8201 changed notification to be fired when Accepted rather than approved * PAY-8201 changed system user created by and updated by * PAY-8201 changed refund Status logic * PAY-8201 changed refund Status logic * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * add PayIt refunds tests * fix checkstyle issues * fix checkstyle issues * PAY-8201 changed createdby and updatedby * PAY-8201 changed status update for refund status * Add bulkscan payments refunds tests with the PayIt journey * fix checkstyle issues * update FTs * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * update test * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * update assertion messages * fix checkstyle issues * fix typo * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * WIP https://tools.hmcts.net/jira/browse/PAY-8237 * PayIt-feature: Improved Coverage. * PayIt-feature: Checkstyle fix. * PAY-8201: Minor update initially rejected refunds (#947) * PAY-8102: Improve code coverage. * PAY-8102: Update to use correct template when resending a notification (#949) * PAY-8102: Update to correct notification template when sending new refund notifications. * PAY-8102: Checkstyle corrections. * PAY-8102: Checkstyle corrections. * PAY-8102: Removing changes to see if FT pass. * PAY-8102: Updated Unit tests, new test coverage for StatusHistoryRespository. * PAY-8102: Minor change to pickup original refund for notification. * PAY-8102: Moved helper functions to StatusHistoryUtil class. * PAY-8102: Missed annotation from helper class. * PAY-8102: Checkstyle fix. * PAY-8201: Update for initial main code * PAY-8201: Update Unit tests. * PAY-8201: Add refund notification doc preview proxy to add templateId (#950) * PAY-8201: Update for initial main code * PAY-8201: Update Unit tests. * PAY-8201:Minor update to logic to pickup correct refund reference. * PAY-8201:Corrections made after merge of notification fix. * PAY-8201:Add missing contact details from request. * PAY-8201: Minor update to notifications for a new refund. * PAY-8201: Minor update to notifications for a new refund. * PAY-8201: Fix reissue version in label (#951) * PAY-8201: Updated getReissueLabel to identify correct version * PAY-8201: Minor update to remove redundant test. * add FT * remove TODO comment --------- Co-authored-by: shravanmechineni <shravanmechineni5@gmail.com> --------- Co-authored-by: Ollie-Smith <120727833+Ollie-Smith@users.noreply.github.com> Co-authored-by: Ollie-Smith <oliver.smith@hmcts.net> Co-authored-by: shravanmechineni <shravanmechineni5@gmail.com> Co-authored-by: Dave Jones <dave.jones@hmcts.net> Co-authored-by: Dave Jones <38425925+davejones74@users.noreply.github.com>
1 parent ba8145a commit 251f893

File tree

65 files changed

+6475
-59
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

65 files changed

+6475
-59
lines changed

Jenkinsfile_CNP

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,10 @@ def secrets = [
1717
secret('paybubble-idam-client-secret', 'OIDC_CLIENT_SECRET'),
1818
secret('refunds-api-client-secret', 'REFUND_SERVICE_ACCOUNT_CLIENT_SECRET'),
1919
secret('refunds-api-user-password', 'REFUND_SERVICE_ACCOUNT_PASSWORD'),
20-
secret('paybubble-idam-client-secret','IDAM_PAYBUBBLE_CLIENT_SECRET')
20+
secret('paybubble-idam-client-secret','IDAM_PAYBUBBLE_CLIENT_SECRET'),
21+
secret('notifications-email-apikey', 'NOTIFY_API_KEY'),
22+
secret('probate-caseworker-username', 'PROBATE_CASE_WORKER_USER_NAME'),
23+
secret('probate-caseworker-password', 'PROBATE_CASE_WORKER_PASSWORD')
2124
]
2225
]
2326

@@ -124,12 +127,14 @@ withPipeline(type, product, app) {
124127

125128
before('functionalTest:aat') {
126129
env.TEST_URL_PAYMENT = 'http://payment-api-aat.service.core-compute-aat.internal'
130+
env.BULKSCAN_API_URL = 'http://ccpay-bulkscanning-api-aat.service.core-compute-aat.internal'
127131
println """\
128132
Using TEST_URL_PAYMENT: ${env.TEST_URL_PAYMENT}""".stripIndent()
129133
}
130134

131135
before('functionalTest:preview') {
132136
env.TEST_URL_PAYMENT = "https://payment-api-ccpay-refunds-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
137+
env.BULKSCAN_API_URL = "https://ccpay-bulkscanning-api-ccpay-refunds-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
133138
println """\
134139
Using TEST_URL_PAYMENT: ${env.TEST_URL_PAYMENT}""".stripIndent()
135140
}
@@ -152,6 +157,7 @@ withPipeline(type, product, app) {
152157
env.PAYMENT_API_URL = "https://payment-api-ccpay-refunds-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
153158
env.NOTIFICATION_API_URL = "https://ccpay-notifications-service-ccpay-refunds-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
154159
env.REFUND_API_URL = "https://ccpay-refunds-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
160+
env.BULKSCAN_API_URL = "https://ccpay-bulkscanning-api-ccpay-refunds-api-pr-${CHANGE_ID}.preview.platform.hmcts.net"
155161
}
156162
}
157163

Jenkinsfile_nightly

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,15 @@ properties([
55
pipelineTriggers([cron('H 07 * * 1-5')]),
66

77
parameters([
8+
string(name: 'ENVIRONMENT', defaultValue: 'aat', description: 'Environment to fetch the keyvault secrets from'),
89
string(name: 'URL_TO_TEST', defaultValue: 'http://ccpay-refunds-api-aat.service.core-compute-aat.internal', description: 'The URL you want to run these tests against'),
910
string(name: 'TEST_URL_PAYMENT', defaultValue: 'http://payment-api-aat.service.core-compute-aat.internal', description: 'The payment app url you want to run these tests against'),
11+
string(name: 'BULKSCAN_API_URL', defaultValue: 'http://ccpay-bulkscanning-api-aat.service.core-compute-aat.internal', description: 'The bulkscan app url you want to run these tests against'),
1012
string(name: 'IDAM_API_URL', defaultValue: 'https://idam-api.aat.platform.hmcts.net', description: 'The Idam url you want to run these tests against'),
1113
string(name: 'S2S_URL', defaultValue: 'http://rpe-service-auth-provider-aat.service.core-compute-aat.internal', description: 'The s2s url you want to run these tests against'),
12-
string(name: 'OAUTH2_REDIRECT_URI', defaultValue: 'https://moneyclaims.aat.platform.hmcts.net', description: 'The redirect url you want to authenticate users and get idam token'),
14+
string(name: 'OAUTH2_REDIRECT_URI', defaultValue: 'https://moneyclaims.aat.platform.hmcts.net', description: 'The cmc redirect url you want to authenticate users and get idam token'),
15+
string(name: 'IDAM_PAYBUBBLE_REDIRECT_URI', defaultValue: 'https://paybubble.aat.platform.hmcts.net/oauth2/callback', description: 'The idam redirect url you want to authenticate users and get idam token'),
16+
string(name: 'CCD_API_URL', defaultValue: 'http://ccd-data-store-api-aat.service.core-compute-aat.internal', description: 'The Idam url you want to run these tests against'),
1317
string(name: 'SecurityRules',
1418
defaultValue: 'http://raw.githubusercontent.com/hmcts/security-test-rules/master/conf/security-rules.conf',
1519
description: 'SecurityUrl against which rules will be executed'),
@@ -23,7 +27,7 @@ def product = "payment"
2327
def component = "api"
2428

2529
def secrets = [
26-
'ccpay-${env}': [
30+
"ccpay-${params.ENVIRONMENT}": [
2731
secret('payment-s2s-secret', 'S2S_SERVICE_SECRET_PAYMENT_APP'),
2832
secret('cmc-service-secret', 'S2S_SERVICE_SECRET_CMC'),
2933
secret('paybubble-s2s-secret', 'S2S_SERVICE_SECRET_PAYBUBBLE'),
@@ -33,7 +37,10 @@ def secrets = [
3337
secret('paybubble-idam-client-secret', 'OIDC_CLIENT_SECRET'),
3438
secret('refunds-api-client-secret', 'REFUND_SERVICE_ACCOUNT_CLIENT_SECRET'),
3539
secret('refunds-api-user-password', 'REFUND_SERVICE_ACCOUNT_PASSWORD'),
36-
secret('paybubble-idam-client-secret','IDAM_PAYBUBBLE_CLIENT_SECRET')
40+
secret('paybubble-idam-client-secret','IDAM_PAYBUBBLE_CLIENT_SECRET'),
41+
secret('notifications-email-apikey', 'NOTIFY_API_KEY'),
42+
secret('probate-caseworker-username', 'PROBATE_CASE_WORKER_USER_NAME'),
43+
secret('probate-caseworker-password', 'PROBATE_CASE_WORKER_PASSWORD')
3744
]
3845
]
3946

@@ -53,6 +60,9 @@ withNightlyPipeline(type, product, component) {
5360
env.IDAM_API_URL = params.IDAM_API_URL
5461
env.S2S_URL = params.S2S_URL
5562
env.OAUTH2_REDIRECT_URI = params.OAUTH2_REDIRECT_URI
63+
env.IDAM_PAYBUBBLE_REDIRECT_URI = params.IDAM_PAYBUBBLE_REDIRECT_URI
64+
env.CCD_API_URL = params.CCD_API_URL
65+
env.BULKSCAN_API_URL = params.BULKSCAN_API_URL
5666
loadVaultSecrets(secrets)
5767
enableSecurityScan()
5868
enableMutationTest()

build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -285,6 +285,7 @@ dependencies {
285285
implementation group: 'org.springframework.security', name: 'spring-security-oauth2-client'
286286
implementation group: 'org.testng', name: 'testng', version: '7.11.0'
287287
implementation group: "io.hypersistence", name: "hypersistence-utils-hibernate-63", version: "3.11.0"
288+
implementation group: 'uk.gov.service.notify', name: 'notifications-java-client', version: '5.2.1-RELEASE'
288289

289290
compileOnly group: 'net.sourceforge.findbugs', name: 'annotations', version: '1.3.2'
290291
compileOnly group: 'org.projectlombok', name: 'lombok', version: versions.lombok

charts/ccpay-refunds-api/values.preview.template.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ java:
2727
PAYMENT_API_URL: "http://${SERVICE_NAME}-payment-api"
2828
TEST_URL_PAYMENT: "http://${SERVICE_NAME}-payment-api"
2929
NOTIFICATION_API_URL: "http://${SERVICE_NAME}-ccpay-notifications-service"
30+
BULKSCAN_API_URL: "http://${SERVICE_NAME}-ccpay-bulkscanning-api"
3031

3132
# PR Pointing - these should no longer be required unless testing simultanous changes between apps.
3233
# PAYMENT_API_URL: "http://payment-api-pr-1376-java"

0 commit comments

Comments
 (0)