@newskit-render/core 0.0.0-d55b7d88 → 0.0.0-d75fd5849

package/Dockerfile.withNewRelic

@@ -4,7 +4,6 @@ WORKDIR /app
 
 ENV NEXT_TELEMETRY_DISABLED 1
 
-RUN npm install @newrelic/next
 COPY .next/standalone ./standalone
 COPY public ./standalone/public
 COPY .next/static ./standalone/.next/static
@@ -12,4 +11,4 @@ COPY .next/static ./standalone/.next/static
 EXPOSE 3000
 ENV PORT 3000
 
-CMD ["node", "-r", "@newrelic/next", "./standalone/server.js"]
+CMD ["node", "./standalone/server.js"]

package/__tests__/pages/__snapshots__/home.test.tsx.snap

@@ -6,7 +6,7 @@ exports[`getServerSideProps Homepage 1`] = `
     class="css-1d2q07x"
   >
     <div
-      class="css-hpal4s"
+      class="css-w8ogm"
       data-testid="NavigationPrimary"
     >
       <div
@@ -102,7 +102,6 @@ exports[`getServerSideProps Homepage 1`] = `
                 aria-label="Section One Link"
                 class="css-1ulbpfm"
                 href="/section-one"
-                type="standalone"
               >
                 <span
                   class="css-1xnclyu"
@@ -126,7 +125,6 @@ exports[`getServerSideProps Homepage 1`] = `
                 aria-label="Section Two Link"
                 class="css-1ulbpfm"
                 href="/section-two"
-                type="standalone"
               >
                 <span
                   class="css-1xnclyu"
@@ -150,7 +148,6 @@ exports[`getServerSideProps Homepage 1`] = `
                 aria-label="Section Three Link"
                 class="css-1ulbpfm"
                 href="/section-three"
-                type="standalone"
               >
                 <span
                   class="css-1xnclyu"

package/components/article/__tests__/__snapshots__/index.test.tsx.snap

@@ -6,7 +6,7 @@ exports[`Article should render tile and sub title 1`] = `
     class="css-1d2q07x"
   >
     <div
-      class="css-hpal4s"
+      class="css-w8ogm"
       data-testid="NavigationPrimary"
     >
       <div
@@ -102,7 +102,6 @@ exports[`Article should render tile and sub title 1`] = `
                 aria-label="Section One Link"
                 class="css-1ulbpfm"
                 href="/section-one"
-                type="standalone"
               >
                 <span
                   class="css-1xnclyu"
@@ -126,7 +125,6 @@ exports[`Article should render tile and sub title 1`] = `
                 aria-label="Section Two Link"
                 class="css-1ulbpfm"
                 href="/section-two"
-                type="standalone"
               >
                 <span
                   class="css-1xnclyu"
@@ -150,7 +148,6 @@ exports[`Article should render tile and sub title 1`] = `
                 aria-label="Section Three Link"
                 class="css-1ulbpfm"
                 href="/section-three"
-                type="standalone"
               >
                 <span
                   class="css-1xnclyu"

package/components/header/banner-messages.ts

@@ -32,6 +32,10 @@ export const pastDueBanner = {
     text: 'We have successfully cancelled your subscription and will be processing your refund shortly. If you have any question please call ##PHONE_NUMBER##.',
     dismissDays: 7,
   },
+  paymentFailure: {
+    title: 'We haven’t been able to take payment',
+    text: 'Your latest payment has failed. Please update your payment details as soon as possible to keep your membership.',
+  },
   treshold: {
     firstNotice: 26,
     secondNotice: 30,

package/cypress/e2e/account/holiday-stops.cy.js

@@ -1,6 +1,6 @@
 import { getTestDates } from '../../fixtures/testDates'
 
-describe('Book a Holiday Stop', () => {
+describe.skip('Book a Holiday Stop', () => {
   beforeEach(() => {
     cy.GetTnlAcsSession('tnl')
     cy.mockConsentAndVisit('/account/holiday-stop')
@@ -13,7 +13,7 @@ describe('Book a Holiday Stop', () => {
   })
 })
 
-describe('Add a Holiday Stop', () => {
+describe.skip('Add a Holiday Stop', () => {
   beforeEach(() => {
     cy.GetTnlAcsSession('tnl')
     cy.mockConsentAndVisit('/account/add/holiday-stop')
@@ -52,7 +52,7 @@ describe('Add a Holiday Stop', () => {
   })
 })
 
-describe('Upcoming Holiday Stops - edit button', () => {
+describe.skip('Upcoming Holiday Stops - edit button', () => {
   beforeEach(() => {
     cy.GetTnlAcsSession('tnl')
     cy.mockConsentAndVisit('/account/holiday-stop')
@@ -86,7 +86,7 @@ describe('Upcoming Holiday Stops - edit button', () => {
   })
 })
 
-describe('Cancel Holiday Stop', () => {
+describe.skip('Cancel Holiday Stop', () => {
   beforeEach(() => {
     cy.GetTnlAcsSession('tnl')
     cy.mockConsentAndVisit('/account/holiday-stop')

package/cypress/e2e/account/newsletters-page.cy.js

@@ -1,33 +1,52 @@
 describe('Newsletters page', () => {
   beforeEach(() => {
     cy.GetTnlAcsSession()
+    cy.mockConsent()
+  })
+
+  it('Should display all the newsletters on the newsletters page when user has a subscription', () => {
     cy.mockConsentAndVisit('/account/newsletters')
+    cy.contains('Newsletters')
+    cy.contains(
+      'Exclusive content and a curated selection of our top stories straight to your inbox.'
+    )
+    cy.get('[data-testid="newsletters-grid"] > div > div').should(
+      'have.length.greaterThan',
+      1
+    )
   })
 
-  it('Should display the newsletters page', () => {
+  it('Should only display selected newsletters on the newsletters page when user is registered', () => {
+    cy.GetTnlAcsSession('tnlNoSubscription')
+    cy.mockConsentAndVisit('/account/newsletters')
     cy.contains('Newsletters')
     cy.contains(
       'Exclusive content and a curated selection of our top stories straight to your inbox.'
     )
-    cy.get('[data-testid="newsletters-grid"] > div').should('have.length', 20)
+    cy.get('[data-testid="newsletters-grid"] > div > div').should(
+      'have.length.greaterThan',
+      1
+    )
   })
 
   it('Should display all required newsletter text in the first grid card', () => {
-    cy.get('[data-testid="newsletters-grid"]').then(($parentDiv) => {
+    cy.mockConsentAndVisit('/account/newsletters')
+    cy.get('[data-testid="newsletters-grid-featured"]').then(($parentDiv) => {
       const firstNewsletter = $parentDiv.find('div:first-child')
 
-      cy.wrap(firstNewsletter).contains('Business').should('be.visible')
+      cy.wrap(firstNewsletter).contains('Daily Briefing').should('be.visible')
       cy.wrap(firstNewsletter).contains('Daily').should('be.visible')
       cy.wrap(firstNewsletter)
         .contains(
-          "The business editor's exclusive analysis of all the latest financial and economic news."
+          'Our flagship newsletter with our top stories and exclusive analysis every morning.'
         )
         .should('be.visible')
     })
   })
 
   it('Should successfully subscribe to a newsletter', () => {
-    cy.get('[data-testid="newsletters-grid"]').then(($parentDiv) => {
+    cy.mockConsentAndVisit('/account/newsletters')
+    cy.get('[data-testid="newsletters-grid-featured"]').then(($parentDiv) => {
       cy.intercept('POST', '/api/account/mutate', { statusCode: 200, ok: true })
 
       const firstNewsletter = $parentDiv.find('div:first-child')
@@ -40,13 +59,14 @@ describe('Newsletters page', () => {
 
       cy.contains(
         '[data-testid="toast-container"]',
-        'Thank you, you are now signed up to the Business newsletter'
+        'Thank you, you are now signed up to the Daily Briefing newsletter'
       )
     })
   })
 
   it('Should show the error toast on server error', () => {
-    cy.get('[data-testid="newsletters-grid"]').then(($parentDiv) => {
+    cy.mockConsentAndVisit('/account/newsletters')
+    cy.get('[data-testid="newsletters-grid-featured"]').then(($parentDiv) => {
       cy.intercept('POST', '/api/account/mutate', {
         statusCode: 500,
         ok: false,
@@ -62,7 +82,7 @@ describe('Newsletters page', () => {
 
       cy.contains(
         '[data-testid="toast-container"]',
-        'Sorry we were unable to sign you up to the Business newsletter, please try again'
+        'Sorry we were unable to sign you up to the Daily Briefing newsletter, please try again'
       )
     })
   })

package/cypress/e2e/account/personal-details.cy.js

@@ -317,44 +317,7 @@ describe('Personal detail edit pages', () => {
         expect(loc.pathname).to.eq('/account')
       })
     })
-    it('Should change user email', () => {
-      const currentEmail = 'cmp-digi-card-20-09-22@yopmail.com'
-      const notRegistredEmail = 'not-registered-email@yopmail.com'
-
-      cy.get('a[href="/account/edit/email"]').should('be.visible').click()
-
-      cy.get('[name="email"]').clear()
-      cy.get('[name="email"]').type(notRegistredEmail)
-      cy.get('[data-testid="primary-button"]').click()
-
-      cy.contains('[data-testid="toast-container"]', 'Updating your email...')
-
-      // Redirects to Personal details page
-      cy.location().should((loc) => {
-        expect(loc.pathname).to.eq('/account')
-      })
-      cy.contains(notRegistredEmail).should('have.length', 1)
-      cy.contains(
-        '[data-testid="toast-container"]',
-        'Your email address was successfully updated'
-      )
-
-      // Back to edit email page
 
-      cy.get('a[href="/account/edit/email"]')
-        .should('be.visible')
-        .click({ force: true })
-
-      // Check the message for verification link
-      cy.contains(
-        `Email sent to ${notRegistredEmail} with a verification link. If you haven't received it yet, check again later or contact customer services on xxxx-xxx-xxxx`
-      )
-
-      // Change email back to the previous email
-      cy.get('[name="email"]').clear()
-      cy.get('[name="email"]').type(currentEmail)
-      cy.get('[data-testid="primary-button"]').click()
-    })
     it('Should NOT change user email if input value is already registered mail', () => {
       const alreadyRegistredEmail = 'cmp-digi-direct-20-09-22@yopmail.com'
 
@@ -370,6 +333,7 @@ describe('Personal detail edit pages', () => {
         'This email address is already in use. Try a different one or contact customer support at 1234-1234'
       )
     })
+
     it('Should show input validation error messages when user tries to add invalid email', () => {
       cy.get('a[href="/account/edit/email"]').should('be.visible').click()
 
@@ -391,6 +355,7 @@ describe('Personal detail edit pages', () => {
         'Updating your email...'
       ).should('not.exist')
     })
+
     it('Should show error toast when updating email if internal server error occures', () => {
       cy.intercept('POST', '/api/account/mutate', { statusCode: 500 })
 
@@ -406,6 +371,45 @@ describe('Personal detail edit pages', () => {
         "Sorry, we're unable to save your email right now. Please try again or come back later."
       )
     })
+
+    it('Should change user email', () => {
+      const currentEmail = 'cmp-digi-card-20-09-22@yopmail.com'
+      const notRegistredEmail = 'not-registered-email@yopmail.com'
+
+      cy.get('a[href="/account/edit/email"]').should('be.visible').click()
+
+      cy.get('[name="email"]').clear()
+      cy.get('[name="email"]').type(notRegistredEmail)
+      cy.get('[data-testid="primary-button"]').click()
+
+      cy.contains('[data-testid="toast-container"]', 'Updating your email...')
+
+      // Redirects to Personal details page
+      cy.location().should((loc) => {
+        expect(loc.pathname).to.eq('/account')
+      })
+      cy.contains(notRegistredEmail).should('have.length', 1)
+      cy.contains(
+        '[data-testid="toast-container"]',
+        'Your email address was successfully updated'
+      )
+
+      // Back to edit email page
+
+      cy.get('a[href="/account/edit/email"]')
+        .should('be.visible')
+        .click({ force: true })
+
+      // Check the message for verification link
+      cy.contains(
+        `Email sent to ${notRegistredEmail} with a verification link. If you haven't received it yet, check again later or contact customer services on xxxx-xxx-xxxx`
+      )
+
+      // Change email back to the previous email
+      cy.get('[name="email"]').clear()
+      cy.get('[name="email"]').type(currentEmail)
+      cy.get('[data-testid="primary-button"]').click()
+    })
   })
 
   describe('Edit password', () => {
@@ -864,7 +868,9 @@ describe('Personal detail edit pages', () => {
       cy.get('[data-testid="select-country"]').click()
       cy.get('[data-testid="United Kingdom"]').should('be.visible')
 
-      cy.get('[data-testid="United States of America"]').click()
+      cy.get('[data-testid="select-panel"]').within(() => {
+        cy.get('[data-testid="United States of America"]').click()
+      })
 
       cy.contains('Save').click()
 
@@ -977,7 +983,9 @@ describe('Personal detail edit pages', () => {
       cy.get('[data-testid="select-country"]').click()
       cy.get('[data-testid="United Kingdom"]').should('be.visible')
 
-      cy.get('[data-testid="United States of America"]').click()
+      cy.get('[data-testid="select-panel"]').within(() => {
+        cy.get('[data-testid="United States of America"]').click()
+      })
 
       cy.contains('Save').click()
 
@@ -1018,6 +1026,12 @@ describe('Personal detail edit pages', () => {
       cy.get('input[name="county"]').type('North Yorkshire')
       cy.get('input[name="postcode"]').clear()
       cy.get('input[name="postcode"]').type('YO1 6DJ')
+      cy.get('[data-testid="select-country"]').click()
+      cy.get('[data-testid="United Kingdom"]').should('be.visible')
+
+      cy.get('[data-testid="select-panel"]').within(() => {
+        cy.get('[data-testid="United Kingdom"]').click()
+      })
 
       cy.contains('Save').click()
 
@@ -1055,6 +1069,10 @@ describe('Personal detail edit pages', () => {
       cy.get('input[name="city"]').type('London')
       cy.get('input[name="postcode"]').clear()
       cy.get('input[name="postcode"]').type('SE1')
+      cy.get('[data-testid="select-country"]').click()
+      cy.get('[data-testid="select-panel"]').within(() => {
+        cy.get('[data-testid="United Kingdom"]').click()
+      })
 
       cy.contains('Save').click()
 

package/cypress/e2e/account/subscription-and-billing.cy.js

@@ -134,18 +134,18 @@ describe('Subscription and billing details for Digital user', () => {
     cy.contains('[data-testid="ContentListIntroduction"]', 'Billing details')
     cy.contains('Payment method')
     cy.get('[data-testid="visa-icon"]').should('exist')
-    cy.contains('************1111')
-    cy.contains('expiry date 03/30')
+    cy.contains('************4242')
+    cy.contains('expiry date 02/27')
   })
 
-  it('Should display Invoice history section', () => {
-    cy.contains('[data-testid="ContentListIntroduction"]', 'Invoice history')
-    cy.contains('Digital pack')
-    cy.contains('20 Sept 2022')
-    cy.contains('Paid')
-    cy.get('a[data-testid="download-button"]')
-      .should('have.attr', 'href', '#')
-      .contains('View')
+  // Section no longer displayed when Invoice history is empty due to graphql change
+  // Invoices are no longer fetched if older than 1 year
+  // This test would periodically fail as it depends on changing user data
+  it('Should not display empty Invoice history section', () => {
+    cy.get('[data-testid="ContentListIntroduction"]').should(
+      'not.contain',
+      'Invoice history'
+    )
   })
 
   it('Should NOT display Your deliveries section', () => {
@@ -213,7 +213,7 @@ describe('Subscription and billing details for Print user', () => {
   it('Should display Your deliveries section', () => {
     cy.contains('[data-testid="ContentListIntroduction"]', 'Your deliveries')
     cy.contains('Address')
-    cy.contains('Buckingham Palace, London, SW1A 1AA')
+    cy.contains('Buckingham Palace, London, SW1A')
     cy.contains('Delivery instructions')
     cy.contains('On my doorstep')
     cy.contains('Holiday Stops')
@@ -226,16 +226,6 @@ describe('Subscription and billing details for Print user', () => {
     )
   })
 
-  it('Should display Invoice history section', () => {
-    cy.contains('[data-testid="ContentListIntroduction"]', 'Invoice history')
-    cy.contains('Print pack')
-    cy.contains('24 Apr 2023')
-    cy.contains('Paid')
-    cy.get('a[data-testid="download-button"]')
-      .should('have.attr', 'href', '#')
-      .contains('View')
-  })
-
   it('Should display Your deliveries section for users with vouchers', () => {
     cy.GetTnlAcsSession('printVouchers')
     cy.mockConsentAndVisit('/account/subscription-and-billing')
@@ -243,8 +233,6 @@ describe('Subscription and billing details for Print user', () => {
     cy.contains('[data-testid="ContentListIntroduction"]', 'Your deliveries')
     cy.contains('Address')
     cy.contains('2 Tuke House, Cromwell Road, York, YO1 6DJ, North Yorkshire')
-    cy.contains('Delivery instructions')
-    cy.contains('On my doorstep')
     cy.contains('Holiday Stops')
     cy.contains(
       'You can pause your print delivery at any time by adding a Holiday Stop to your account. Holiday Stops need to be booked before 16:00 GMT the day before you want to pause delivery.'
@@ -254,7 +242,7 @@ describe('Subscription and billing details for Print user', () => {
       'If your voucher booklet hasn’t arrived within three days of your booklet start date, or you have any issues regarding your vouchers, please contact us.'
     )
     cy.contains('Voucher booklet start date')
-    cy.contains('16/06/24')
+    cy.contains('20/09/2022')
     cy.contains(
       'Your contact address will be used for any of your deliveries. This can be managed via Personal details'
     )
@@ -277,16 +265,6 @@ describe('Subscription and billing details for Digi-Print user', () => {
     cy.contains('Digi-Print')
     //...rest is same as Print user
   })
-
-  it('Should display Invoice history section', () => {
-    cy.contains('[data-testid="ContentListIntroduction"]', 'Invoice history')
-    cy.contains('Digi-Print pack')
-    cy.contains('12 Nov 2023')
-    cy.contains('Paid')
-    cy.get('a[data-testid="download-button"]')
-      .should('have.attr', 'href', '#')
-      .contains('View')
-  })
 })
 
 describe('Subscription and billing details for Paypal user', () => {
@@ -302,14 +280,6 @@ describe('Subscription and billing details for Paypal user', () => {
     cy.get('[data-testid="paypal-icon"]').should('exist')
     cy.contains('cmp.paypal.user.verified.test@news.co.uk')
   })
-
-  it('Should show inline banner for Paypal subscription', () => {
-    cy.get('a[href="/account/payment"]').should('be.visible').click()
-    cy.contains(
-      '[data-testid="Zuora-inline"]',
-      'To update your Paypal credentials, please speak to one of our advisors on xxxx-xxx-xxxx or add a new payment method via credit/debit card using the form below.'
-    )
-  })
 })
 
 describe('Subscription and billing edit pages', () => {

package/cypress/support/commands.js

@@ -1,4 +1,4 @@
-//Note: Check periodically if these consent cookie values exprire
+//Note: Check periodically if these consent cookie values expire
 import _ from 'lodash'
 import {
   consentUUIDValue,
@@ -60,13 +60,13 @@ Cypress.Commands.add('GetTnlAcsSession', (user = 'tnl') => {
 
   cy.request({
     method: 'POST',
-    url: 'https://login.staging-thetimes.co.uk/services/session',
+    url: 'https://login.staging-thetimes.com/services/session',
     headers: {
       'Content-Type': 'application/vnd.newsuk.acs.createsession-v1+json',
     },
     body: {
       rememberMe: false,
-      gotoUrl: 'https://login.staging-thetimes.co.uk/',
+      gotoUrl: 'https://login.staging-thetimes.com/',
       sso: false,
       authCredentials: {
         username,

package/cypress/support/users.js

@@ -22,37 +22,41 @@ export const userMap = {
   },
   tnl: {
     username: 'cmp1aug2022printstag@yopmail.com', //Print user
-    password: 'Password123',
+    password: 'Password123Testing123$',
   },
   digitalCard: {
     //dtc=false
     username: 'cmp-digi-card-20-09-22@yopmail.com',
-    password: 'Password123',
+    password: 'Password123Testing123$',
   },
   digitalDirectDebit: {
     //dtc=false
     username: 'cmp-digi-direct-20-09-22@yopmail.com',
-    password: 'Password123',
+    password: 'Password123Testing123$',
   },
   print: {
     //dtc=true
     username: 'cmp-dtc-card-20-09-22@yopmail.com',
-    password: 'Password123',
+    password: 'Password123Testing123$',
   },
   printVouchers: {
     //dtc=false
     username: 'cmp-test-direct-20-09-22@yopmail.com',
-    password: 'Password123',
+    password: 'Password123Testing123$',
   },
   digiprint: {
     //dtc=true
     username: 'digi-print.dtc.test@yopmail.com',
-    password: 'Password123',
+    password: 'Password123Testing123$',
   },
   digiprintVouchers: {
     //dtc=false
     username: 'digi-print-card-15-11-22@yopmail.com',
-    password: 'Password123',
+    password: 'Password123Testing123$',
   },
-  }
-  
+  tnlNoSubscription: {
+    //registered user with no subscription, tests the number of newsletters displayed
+    username: 'tm-test+staging-registered-user@news.co.uk',
+    password: 'Growth123',
+  },
+}

package/helpers/getProviderPropsUtil.ts

@@ -0,0 +1,20 @@
+import {
+  getProviderProps,
+  getProviderPropsTimes,
+} from '@newskit-render/my-account'
+import { publisher } from '../config'
+
+export const getProviderPropsUtil = (context, options?) => {
+  if (process.env.NODE_ENV === 'development' && publisher === 'TIMES') {
+    return getProviderPropsTimes({
+      context,
+      options: {
+        ...options,
+      },
+    })
+  }
+
+  return getProviderProps(context, {
+    ...options,
+  })
+}