Skip to content

Commit

Permalink
Fix more missed links
Browse files Browse the repository at this point in the history
  • Loading branch information
Cruikshanks committed Oct 20, 2024
1 parent 2935399 commit e244d61
Show file tree
Hide file tree
Showing 21 changed files with 50 additions and 50 deletions.
2 changes: 1 addition & 1 deletion app/presenters/licences/view-licence-history.presenter.js
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ function _returnVersionEntries (licence) {
dateCreated: formatLongDate(createdAt),
displayNote: notes.length > 0,
notes,
link: `/system/return-requirements/${returnVersion.id}/view`,
link: `/system/return-requirements/${returnVersion.id}`,
reason: mappedReason ?? reason,
type: { index: 2, name: 'Return version' }
}
Expand Down
6 changes: 3 additions & 3 deletions test/controllers/return-requirements-setup.controller.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ describe('Return requirements controller', () => {
Sinon.stub(SubmitAbstractionPeriod, 'go').resolves({})
})

it('redirects to /system/return-requirements/{sessionId}/returns-cycle/{requirementIndex}', async () => {
it('redirects to /system/return-requirements/setup/{sessionId}/returns-cycle/{requirementIndex}', async () => {
const response = await server.inject(_postOptions(path, requirementIndex))

expect(response.statusCode).to.equal(302)
Expand Down Expand Up @@ -349,7 +349,7 @@ describe('Return requirements controller', () => {
Sinon.stub(SubmitExistingService, 'go').resolves({})
})

it('redirects to /system/return-requirements/{sessionId}/check', async () => {
it('redirects to /system/return-requirements/setup/{sessionId}/check', async () => {
const response = await server.inject(_postOptions(path))

expect(response.statusCode).to.equal(302)
Expand Down Expand Up @@ -413,7 +413,7 @@ describe('Return requirements controller', () => {
Sinon.stub(SubmitFrequencyCollectedService, 'go').resolves({ checkPageVisited: true })
})

it('redirects to /system/return-requirements/{sessionId}/check', async () => {
it('redirects to /system/return-requirements/setup/{sessionId}/check', async () => {
const response = await server.inject(_postOptions(path, requirementIndex))

expect(response.statusCode).to.equal(302)
Expand Down
4 changes: 2 additions & 2 deletions test/presenters/licences/set-up.presenter.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -575,7 +575,7 @@ describe('Licences - Set Up presenter', () => {
{
action: [
{
link: '/system/return-requirements/0312e5eb-67ae-44fb-922c-b1a0b81bc08d',
link: '/system/return-requirements/setup/0312e5eb-67ae-44fb-922c-b1a0b81bc08d',
text: 'View'
}
],
Expand All @@ -601,7 +601,7 @@ describe('Licences - Set Up presenter', () => {
{
action: [
{
link: '/system/return-requirements/0312e5eb-67ae-44fb-922c-b1a0b81bc08d',
link: '/system/return-requirements/setup/0312e5eb-67ae-44fb-922c-b1a0b81bc08d',
text: 'View'
}
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ describe('View Licence History presenter', () => {
dateCreated: '5 April 2021',
displayNote: true,
notes: ['Test note'],
link: '/system/return-requirements/3f09ce0b-288c-4c0b-b519-7329fe70a6cc/view',
link: '/system/return-requirements/setup/3f09ce0b-288c-4c0b-b519-7329fe70a6cc/view',
reason: 'New licence',
type: { index: 2, name: 'Return version' }
}
Expand Down Expand Up @@ -131,7 +131,7 @@ describe('View Licence History presenter', () => {
it('returns the return version link', () => {
const result = ViewLicenceHistoryPresenter.go(licence)

expect(result.entries[2].link).to.equal('/system/return-requirements/3f09ce0b-288c-4c0b-b519-7329fe70a6cc/view')
expect(result.entries[2].link).to.equal('/system/return-requirements/setup/3f09ce0b-288c-4c0b-b519-7329fe70a6cc/view')
})
})
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ describe('Return Requirements Setup - Abstraction Period presenter', () => {

expect(result).to.equal({
abstractionPeriod: null,
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/points/0',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/points/0',
licenceId: '8b7f78ba-f3ad-4cb6-a058-78abc4d1383d',
licenceRef: '01/ABC',
sessionId: '61e07498-f309-4829-96a9-72084a54996d'
Expand Down Expand Up @@ -89,15 +89,15 @@ describe('Return Requirements Setup - Abstraction Period presenter', () => {
it('returns a link back to the "check" page', () => {
const result = AbstractionPeriodPresenter.go(session, requirementIndex)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})

describe('when the user has come from somewhere else', () => {
it('returns a link back to the "points" page', () => {
const result = AbstractionPeriodPresenter.go(session, requirementIndex)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/points/0')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/points/0')
})
})
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ describe('Return Requirements Setup - Additional Submission Options presenter',
const result = AdditionalSubmissionOptionsPresenter.go(session)

expect(result).to.be.equal({
backLink: `/system/return-requirements/${session.id}/check`,
backLink: `/system/return-requirements/setup/${session.id}/check`,
licenceId: '8b7f78ba-f3ad-4cb6-a058-78abc4d1383d',
additionalSubmissionOptions: [],
licenceRef: '01/ABC',
Expand All @@ -50,7 +50,7 @@ describe('Return Requirements Setup - Additional Submission Options presenter',
it('returns a link back to the "check" page', () => {
const result = AdditionalSubmissionOptionsPresenter.go(session)

expect(result.backLink).to.equal(`/system/return-requirements/${session.id}/check`)
expect(result.backLink).to.equal(`/system/return-requirements/setup/${session.id}/check`)
})
})

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ describe('Return Requirements Setup - Agreements Exceptions presenter', () => {

expect(result).to.equal({
agreementsExceptions: null,
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/frequency-reported/0',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/frequency-reported/0',
licenceId: '8b7f78ba-f3ad-4cb6-a058-78abc4d1383d',
licenceRef: '01/ABC',
sessionId: '61e07498-f309-4829-96a9-72084a54996d'
Expand Down Expand Up @@ -79,15 +79,15 @@ describe('Return Requirements Setup - Agreements Exceptions presenter', () => {
it('returns a link back to the "check" page', () => {
const result = AgreementsExceptionsPresenter.go(session, requirementIndex)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})

describe('when the user has come from somewhere else', () => {
it('returns a link back to the "frequency-reported" page', () => {
const result = AgreementsExceptionsPresenter.go(session, requirementIndex)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/frequency-reported/0')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/frequency-reported/0')
})
})
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ describe('Return Requirements Setup - Cancel presenter', () => {
const result = CancelPresenter.go(session)

expect(result).to.equal({
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check',
licenceId: '8b7f78ba-f3ad-4cb6-a058-78abc4d1383d',
licenceRef: '01/ABC',
reason: 'Major change',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ describe('Return Requirements Setup - Frequency Collected presenter', () => {
const result = FrequencyCollectedPresenter.go(session, requirementIndex)

expect(result).to.equal({
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/site-description/0',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/site-description/0',
frequencyCollected: null,
licenceId: '8b7f78ba-f3ad-4cb6-a058-78abc4d1383d',
licenceRef: '01/ABC',
Expand All @@ -57,15 +57,15 @@ describe('Return Requirements Setup - Frequency Collected presenter', () => {
it('returns a link back to the "check" page', () => {
const result = FrequencyCollectedPresenter.go(session, requirementIndex)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})

describe('when the user has come from somewhere else', () => {
it('returns a link back to the "site-description" page', () => {
const result = FrequencyCollectedPresenter.go(session, requirementIndex)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/site-description/0')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/site-description/0')
})
})
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ describe('Return Requirements Setup - Frequency Reported presenter', () => {
const result = FrequencyReportedPresenter.go(session, requirementIndex)

expect(result).to.equal({
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/frequency-collected/0',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/frequency-collected/0',
frequencyReported: null,
licenceId: '8b7f78ba-f3ad-4cb6-a058-78abc4d1383d',
licenceRef: '01/ABC',
Expand All @@ -57,15 +57,15 @@ describe('Return Requirements Setup - Frequency Reported presenter', () => {
it('returns a link back to the "check" page', () => {
const result = FrequencyReportedPresenter.go(session, requirementIndex)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})

describe('when the user has come from somewhere else', () => {
it('returns a link back to the "frequency-collected" page', () => {
const result = FrequencyReportedPresenter.go(session, requirementIndex)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/frequency-collected/0')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/frequency-collected/0')
})
})
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ describe('Return Requirements Setup - Method presenter', () => {
const result = MethodPresenter.go(session)

expect(result).to.equal({
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/reason',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/reason',
displayCopyExisting: true,
licenceRef: '01/ABC',
sessionId: '61e07498-f309-4829-96a9-72084a54996d',
Expand All @@ -55,7 +55,7 @@ describe('Return Requirements Setup - Method presenter', () => {
it('returns a link back to the "start-date" page', () => {
const result = MethodPresenter.go(session)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/reason')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/reason')
})
})

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ describe('Return Requirements Setup - No Returns Required presenter', () => {
const result = NoReturnsRequiredPresenter.go(session)

expect(result).to.equal({
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/start-date',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/start-date',
licenceRef: '01/ABC',
reason: null,
sessionId: '61e07498-f309-4829-96a9-72084a54996d'
Expand All @@ -53,15 +53,15 @@ describe('Return Requirements Setup - No Returns Required presenter', () => {
it('returns a link back to the "check" page', () => {
const result = NoReturnsRequiredPresenter.go(session)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})

describe('when the user has come from somewhere else', () => {
it('returns a link back to the "start-date" page', () => {
const result = NoReturnsRequiredPresenter.go(session)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/start-date')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/start-date')
})
})
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ describe('Return Requirements Setup - Note presenter', () => {
const result = NotePresenter.go(session)

expect(result).to.be.equal({
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check',
licenceRef: '01/ABC',
note: null,
sessionId: '61e07498-f309-4829-96a9-72084a54996d'
Expand All @@ -49,7 +49,7 @@ describe('Return Requirements Setup - Note presenter', () => {
it('returns a link back to the "check" page', () => {
const result = NotePresenter.go(session)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ describe('Return Requirements Setup - Points presenter', () => {
const result = PointsPresenter.go(session, requirementIndex, points)

expect(result).to.equal({
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/purpose/0',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/purpose/0',
licenceId: '8b7f78ba-f3ad-4cb6-a058-78abc4d1383d',
licencePoints: [{
id: 'd03d7d7c-4e33-4b4d-ac9b-6ebac9a5e5f6',
Expand Down Expand Up @@ -73,15 +73,15 @@ describe('Return Requirements Setup - Points presenter', () => {
it('returns a link back to the "check" page', () => {
const result = PointsPresenter.go(session, requirementIndex, points)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})

describe('when the user has come from somewhere else', () => {
it('returns a link back to the "purpose" page', () => {
const result = PointsPresenter.go(session, requirementIndex, points)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/purpose/0')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/purpose/0')
})
})
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ describe('Return Requirements Setup - Purpose presenter', () => {
const result = PurposePresenter.go(session, requirementIndex, licencePurposes)

expect(result).to.equal({
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/method',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/method',
licenceId: '8b7f78ba-f3ad-4cb6-a058-78abc4d1383d',
licenceRef: '01/ABC',
purposes: [
Expand All @@ -71,7 +71,7 @@ describe('Return Requirements Setup - Purpose presenter', () => {
it('returns a link back to the "check" page', () => {
const result = PurposePresenter.go(session, requirementIndex, licencePurposes)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})

Expand All @@ -86,7 +86,7 @@ describe('Return Requirements Setup - Purpose presenter', () => {
it('returns a link back to the "check" page', () => {
const result = PurposePresenter.go(session, requirementIndex, licencePurposes)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})
})
Expand All @@ -95,7 +95,7 @@ describe('Return Requirements Setup - Purpose presenter', () => {
it('returns a link back to the "method" page', () => {
const result = PurposePresenter.go(session, requirementIndex, licencePurposes)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/method')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/method')
})
})
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ describe('Return Requirements Setup - Reason presenter', () => {
const result = ReasonPresenter.go(session)

expect(result).to.equal({
backLink: '/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/start-date',
backLink: '/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/start-date',
licenceRef: '01/ABC',
reason: null,
sessionId: '61e07498-f309-4829-96a9-72084a54996d'
Expand All @@ -53,15 +53,15 @@ describe('Return Requirements Setup - Reason presenter', () => {
it('returns a link back to the "check" page', () => {
const result = ReasonPresenter.go(session)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/check')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/check')
})
})

describe('when the user has come from somewhere else', () => {
it('returns a link back to the "start-date" page', () => {
const result = ReasonPresenter.go(session)

expect(result.backLink).to.equal('/system/return-requirements/61e07498-f309-4829-96a9-72084a54996d/start-date')
expect(result.backLink).to.equal('/system/return-requirements/setup/61e07498-f309-4829-96a9-72084a54996d/start-date')
})
})
})
Expand Down
Loading

0 comments on commit e244d61

Please sign in to comment.