@financial-times/dotcom-ui-data-embed 7.3.0 → 7.3.2

package/package.json

@@ -1,19 +1,18 @@
 {
   "name": "@financial-times/dotcom-ui-data-embed",
-  "version": "7.3.0",
+  "version": "7.3.2",
   "description": "",
   "main": "component.js",
   "browser": "browser.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",
     "clean:install": "npm run clean && npm i",
-    "build:node": "npm run tsc -- --module commonjs --outDir ./dist/node",
-    "build:browser": "npm run tsc -- --module es2015 --outDir ./dist/browser",
+    "build:browser": "tsc --module es2015 --outDir ./dist/browser",
+    "build:node": "tsc",
     "build": "npm run build:node && npm run build:browser",
     "dev": "echo -n node browser | parallel -u -d ' ' npm run build:{} -- --watch",
     "preinstall": "[ \"$INIT_CWD\" != \"$PWD\" ] || npm_config_yes=true npx check-engine"
@@ -28,6 +27,13 @@
   "peerDependencies": {
     "react": "16.x || 17.x"
   },
+  "files": [
+    "dist/",
+    "src/",
+    "browser.js",
+    "component.js",
+    "server.js"
+  ],
   "repository": {
     "type": "git",
     "repository": "https://github.com/Financial-Times/dotcom-page-kit.git",
@@ -41,4 +47,4 @@
     "check-engine": "^1.10.1",
     "react": "^16.8.6"
   }
-}
+}

package/src/__test__/client/DataEmbedStore.spec.ts

@@ -0,0 +1,37 @@
+import subject from '../../client/DataEmbedStore'
+
+const fakeData = {
+  foo: 1,
+  bar: true,
+  baz: 'qux'
+}
+
+describe('dotcom-ui-data-embed/src/client/DataEmbedStore', () => {
+  let instance
+
+  beforeEach(() => {
+    instance = new subject(fakeData)
+  })
+
+  describe('.get()', () => {
+    it('returns the value of an existing data', () => {
+      expect(instance.get('foo')).toBe(1)
+      expect(instance.get('bar')).toBe(true)
+      expect(instance.get('baz')).toBe('qux')
+    })
+
+    it('returns undefined for properties that do not exist', () => {
+      expect(instance.get('buzz')).toBeUndefined()
+    })
+  })
+
+  describe('.getAll()', () => {
+    it('returns all data', () => {
+      expect(instance.getAll()).toEqual(fakeData)
+    })
+
+    it('freezes the data', () => {
+      expect(Object.isFrozen(instance.getAll())).toBe(true)
+    })
+  })
+})

package/src/__test__/client/index.spec.ts

@@ -0,0 +1,16 @@
+import { init as subject } from '../../client'
+import loadDataEmbed from '../../client/loadDataEmbed'
+import DataEmbedStore from '../../client/DataEmbedStore'
+jest.mock('../../client/loadDataEmbed.ts')
+jest.mock('../../client/DataEmbedStore.ts')
+
+describe('dotcom-ui-data-embed/src/client/index', () => {
+  describe('.init()', () => {
+    it('returns the value of an existing data embed store', () => {
+      const result = subject({ id: 'data-embed' })
+      expect(result).toBeInstanceOf(DataEmbedStore)
+      expect(loadDataEmbed).toBeCalledWith('data-embed')
+      expect(DataEmbedStore).toHaveBeenCalledTimes(1)
+    })
+  })
+})

package/src/__test__/client/loadDataEmbed.spec.ts

@@ -0,0 +1,36 @@
+/**
+ * @jest-environment jsdom
+ */
+
+import subject from '../../client/loadDataEmbed'
+const SCRIPT_ELEMENT_ID = 'data-embed'
+
+describe('dotcom-ui-data-embed/src/client/loadDataEmbed', () => {
+  describe('when there is a configuration object', () => {
+    beforeEach(() => {
+      document.body.innerHTML = `
+        <script id="${SCRIPT_ELEMENT_ID}">{"foo":1,"bar":true,"baz":"qux"}</script>
+      `
+    })
+
+    it('returns a frozen object', () => {
+      const result = subject(SCRIPT_ELEMENT_ID)
+
+      expect(result).toEqual({ foo: 1, bar: true, baz: 'qux' })
+      expect(Object.isFrozen(result)).toBe(true)
+    })
+  })
+
+  describe('when there is no a configuration object', () => {
+    beforeEach(() => {
+      document.body.innerHTML = ''
+    })
+
+    it('returns a frozen empty object', () => {
+      const result = subject(SCRIPT_ELEMENT_ID)
+
+      expect(result).toEqual({})
+      expect(Object.isFrozen(result)).toBe(true)
+    })
+  })
+})

package/src/__test__/components/DataEmbed.spec.ts

@@ -0,0 +1,18 @@
+import renderer from 'react-test-renderer'
+import { DataEmbed as subject } from '../../components/DataEmbed'
+
+const data = {
+  id: 'data-embed',
+  data: {
+    foo: 1,
+    bar: true,
+    baz: 'qux'
+  }
+}
+
+describe('dotcom-ui-data-embed/src/components/dataEmbed', () => {
+  it('renders a script element containing data', () => {
+    const tree = renderer.create(subject(data))
+    expect(tree).toMatchSnapshot()
+  })
+})

package/src/__test__/components/__snapshots__/DataEmbed.spec.ts.snap

@@ -0,0 +1,13 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`dotcom-ui-data-embed/src/components/dataEmbed renders a script element containing data 1`] = `
+<script
+  dangerouslySetInnerHTML={
+    Object {
+      "__html": "{\\"foo\\":1,\\"bar\\":true,\\"baz\\":\\"qux\\"}",
+    }
+  }
+  id="data-embed"
+  type="application/json"
+/>
+`;