fix: hard code returns url in the njk

DEFRA · May 3, 2024 · b895c74 · b895c74
1 parent aa2092c
commit b895c74
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 6 deletions.
diff --git a/app/presenters/licences/view-licence-returns.presenter.js b/app/presenters/licences/view-licence-returns.presenter.js
@@ -17,14 +17,13 @@ function go (returnsData) {
 
   return {
     activeTab: 'returns',
-    returnsUrl: 'return/internal',
     returns
   }
 }
 
 function _formatPurpose (purpose) {
   const [firstPurpose] = purpose
-  
+
   return firstPurpose.alias ? firstPurpose.alias : firstPurpose.tertiary.description
 }
 

diff --git a/app/views/licences/tabs/returns.njk b/app/views/licences/tabs/returns.njk
@@ -34,7 +34,7 @@
     {% for return in returns %}
       <tr class="govuk-table__row govuk-body-s">
         <th class="govuk-table__cell">
-          <a href="/{{ returnsUrl }}?returnId={{ return.id }}" class="govuk-link">{{ return.reference }} </a>
+          <a href="/return/internal?returnId={{ return.id }}" class="govuk-link">{{ return.reference }} </a>
           <p class="govuk-body-s"> {{ return.dates }}</p>
         </th>
         <td class="govuk-table__cell">{{ return.purpose }} <p class="govuk-body-s"> {{ return.description }}</p></td>

diff --git a/test/presenters/licences/view-licence-returns-presenter.test.js b/test/presenters/licences/view-licence-returns-presenter.test.js
@@ -22,7 +22,6 @@ describe('View Licence returns presenter', () => {
 
       expect(result).to.equal({
         activeTab: 'returns',
-        returnsUrl: 'return/internal',
         returns: [
           {
             id: 'mock-id-1',

diff --git a/test/services/licences/view-licence-returns.service.test.js b/test/services/licences/view-licence-returns.service.test.js
@@ -41,7 +41,6 @@ describe('View Licence service returns', () => {
 
         expect(result).to.equal({
           licenceName: 'fake licence',
-          returnsUrl: 'return/internal',
           returns: [],
           activeTab: 'returns',
           pagination: { page: 1 }
@@ -53,7 +52,6 @@ describe('View Licence service returns', () => {
 
 function _returnsPresenter () {
   return {
-    returnsUrl: 'return/internal',
     returns: [],
     activeTab: 'returns'
   }