@financial-times/dotcom-middleware-navigation 7.3.0 → 7.3.2

package/package.json

@@ -1,17 +1,16 @@
 {
   "name": "@financial-times/dotcom-middleware-navigation",
-  "version": "7.3.0",
+  "version": "7.3.2",
   "description": "",
   "main": "dist/node/index.js",
   "types": "src/index.ts",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1",
-    "tsc": "../../node_modules/.bin/tsc --incremental",
     "clean": "npm run clean:dist && npm run clean:node_modules",
     "clean:dist": "rm -rf dist",
     "clean:node_modules": "rm -rf node_modules",
+    "build:node": "tsc",
     "build": "npm run build:node",
-    "build:node": "npm run tsc -- --module commonjs --outDir ./dist/node",
     "dev": "npm run build:node -- --watch",
     "preinstall": "[ \"$INIT_CWD\" != \"$PWD\" ] || npm_config_yes=true npx check-engine"
   },
@@ -19,8 +18,8 @@
   "author": "",
   "license": "MIT",
   "dependencies": {
-    "@financial-times/dotcom-server-navigation": "^7.3.0",
-    "@financial-times/dotcom-types-navigation": "^7.3.0"
+    "@financial-times/dotcom-server-navigation": "file:../../packages/dotcom-server-navigation",
+    "@financial-times/dotcom-types-navigation": "file:../dotcom-types-navigation"
   },
   "devDependencies": {
     "check-engine": "^1.10.1",
@@ -30,6 +29,10 @@
     "node": ">= 14.0.0",
     "npm": "7.x || 8.x"
   },
+  "files": [
+    "dist/",
+    "src/"
+  ],
   "repository": {
     "type": "git",
     "repository": "https://github.com/Financial-Times/dotcom-page-kit.git",
@@ -39,4 +42,4 @@
   "volta": {
     "extends": "../../package.json"
   }
-}
+}

package/src/__test__/handleEdition.spec.ts

@@ -0,0 +1,64 @@
+import subject from '../handleEdition'
+import httpMocks from 'node-mocks-http'
+
+describe('dotcom-middleware-navigation/src/handleEdition', () => {
+  describe('with no query string parameter or cookie set', () => {
+    let request
+    let response
+    let result
+
+    beforeAll(() => {
+      request = httpMocks.createRequest()
+      response = httpMocks.createResponse()
+      result = subject(request, response)
+    })
+
+    it('returns the default edition', () => {
+      expect(result).toEqual('uk')
+    })
+
+    it('sets an edition vary header', () => {
+      expect(response.getHeader('vary')).toBe('FT-Edition')
+    })
+  })
+
+  describe('with an edition header set', () => {
+    let request
+    let response
+    let result
+
+    beforeAll(() => {
+      request = httpMocks.createRequest({ headers: { 'ft-edition': 'international' } })
+      response = httpMocks.createResponse()
+      result = subject(request, response)
+    })
+
+    it('returns the set edition', () => {
+      expect(result).toEqual('international')
+    })
+
+    it('sets an edition vary header', () => {
+      expect(response.getHeader('vary')).toBe('FT-Edition')
+    })
+  })
+
+  describe('with a query string parameter set', () => {
+    let request
+    let response
+    let result
+
+    beforeAll(() => {
+      request = httpMocks.createRequest({ query: { edition: 'international' } })
+      response = httpMocks.createResponse()
+      result = subject(request, response)
+    })
+
+    it('returns the selected edition', () => {
+      expect(result).toEqual('international')
+    })
+
+    it('sets a next-edition cookie with the selected edition', () => {
+      expect(response.cookies['next-edition'].value).toBe('international')
+    })
+  })
+})

package/src/__test__/navigation.spec.ts

@@ -0,0 +1,184 @@
+import * as subject from '../navigation'
+import httpMocks from 'node-mocks-http'
+
+const fakeMenusData = {
+  navbar: {
+    label: 'Navbar UK',
+    items: [{ label: 'Foo', url: '#' }]
+  },
+  drawer: {
+    label: 'Drawer UK',
+    items: [{ label: 'Foo', url: '#' }]
+  }
+}
+
+const fakeEditionsData = {
+  current: { id: 'uk', name: 'UK', url: '/' },
+  others: [{ id: 'international', name: 'International', url: '/' }]
+}
+
+const fakeActionsData = { id: 'subscribe', name: 'Subscribe for full access', url: '/products?' }
+
+const fakeSubNavigationData = {
+  breadcrumb: 'some-breadcrumb',
+  subsections: 'some-subsections'
+}
+
+const fakeNavigationData = { ...fakeMenusData, editions: fakeEditionsData, currentPath: '' }
+
+const FakeNavigation = {
+  getSubscribeAction: jest.fn().mockResolvedValue(fakeActionsData),
+  getMenusFor: jest.fn().mockResolvedValue(fakeMenusData),
+  getSubNavigationFor: jest.fn().mockResolvedValue(fakeSubNavigationData),
+  getEditionsFor: jest.fn().mockReturnValue(fakeEditionsData)
+}
+
+jest.mock(
+  '@financial-times/dotcom-server-navigation',
+  () => {
+    return {
+      Navigation: jest.fn().mockImplementation(() => FakeNavigation)
+    }
+  },
+  { virtual: true }
+)
+
+describe('dotcom-middleware-navigation', () => {
+  let request
+  let response
+  let next
+  let instance
+
+  beforeEach(() => {
+    request = httpMocks.createRequest()
+    response = httpMocks.createResponse({ locals: {} })
+    next = jest.fn()
+    instance = subject.init()
+  })
+
+  afterEach(() => {
+    jest.clearAllMocks()
+  })
+
+  it('returns a middleware function', () => {
+    expect(instance).toBeInstanceOf(Function)
+    expect(instance.length).toBe(3)
+  })
+
+  describe('when handling a request', () => {
+    it('appends the navigation properties to response.locals', async () => {
+      await instance(request, response, next)
+      expect(response.locals.navigation).toEqual(expect.objectContaining(fakeNavigationData))
+    })
+
+    it('prefers the vanity URL if set', async () => {
+      const request = httpMocks.createRequest({
+        path: '/path/to/app/1234',
+        headers: {
+          'ft-vanity-url': '/vanity-url'
+        }
+      })
+
+      await instance(request, response, next)
+      expect(FakeNavigation.getMenusFor).toHaveBeenCalledWith('/vanity-url', 'uk')
+    })
+
+    it('calls the fallthrough function', async () => {
+      await instance(request, response, next)
+      expect(next).toHaveBeenCalled()
+    })
+  })
+
+  describe('when handling a request with sub-navigation enabled', () => {
+    let instance
+
+    beforeEach(() => {
+      instance = subject.init({ enableSubNavigation: true })
+    })
+
+    it('prefers the path provided by the vanity URL header when available', async () => {
+      request = httpMocks.createRequest({
+        path: '/path/to/page/123',
+        headers: {
+          'ft-vanity-url': '/vanity-url'
+        }
+      })
+
+      await instance(request, response, next)
+
+      expect(FakeNavigation.getSubNavigationFor).toHaveBeenCalledWith('/vanity-url')
+    })
+
+    it('normalizes the current path', async () => {
+      request = httpMocks.createRequest({
+        path: '/path/to/page/123',
+        query: {
+          page: 2
+        },
+        headers: {
+          'ft-vanity-url': '/vanity-url?page=2'
+        }
+      })
+
+      await instance(request, response, next)
+
+      expect(FakeNavigation.getSubNavigationFor).toHaveBeenCalledWith('/vanity-url')
+    })
+
+    it('appends the sub-navigation properties on response.locals for the current path', async () => {
+      await instance(request, response, next)
+      expect(response.locals.navigation).toEqual(expect.objectContaining(fakeSubNavigationData))
+    })
+
+    it('fails silently if the the sub-navigation request fails', async () => {
+      FakeNavigation.getSubNavigationFor.mockRejectedValueOnce(new Error('Fail'))
+
+      await instance(request, response, next)
+
+      expect(next).not.toHaveBeenCalledWith(expect.any(Error))
+    })
+  })
+
+  describe('when handling a request with a custom getCurrentPath', () => {
+    it('executes the provided getCurrentPath function', async () => {
+      request = httpMocks.createRequest({
+        path: '/path/to/page/123',
+        headers: {
+          'ft-vanity-url': '/vanity-url?page=2'
+        }
+      })
+      const dummyPath = '/foo'
+      const instance = subject.init({ getCurrentPath: () => dummyPath })
+      await instance(request, response, next)
+
+      expect(FakeNavigation.getMenusFor).toHaveBeenCalledWith(dummyPath, 'uk')
+    })
+
+    it('allows overriding of how to calculate current path logic', async () => {
+      request = httpMocks.createRequest({
+        path: '/path/to/page/123',
+        headers: {
+          'ft-blocked-url': '/ig-content-test'
+        }
+      })
+
+      const instance = subject.init({ getCurrentPath: (request) => request.get('ft-blocked-url') })
+      await instance(request, response, next)
+
+      expect(FakeNavigation.getMenusFor).toHaveBeenCalledWith('/ig-content-test', 'uk')
+    })
+  })
+
+  describe('when something goes wrong', () => {
+    beforeEach(() => {
+      FakeNavigation.getMenusFor = jest.fn(() => {
+        throw Error('Whoops')
+      })
+    })
+
+    it('catches the error safely', async () => {
+      await instance(request, response, next)
+      expect(next).toHaveBeenCalledWith(expect.any(Error))
+    })
+  })
+})