@financial-times/n-myft-ui 40.0.2 → 40.1.0

package/.circleci/config.yml

@@ -3,10 +3,6 @@ orbs:
   tool-kit: financial-times/dotcom-tool-kit@5
 executors:
   node:
-    docker:
-      - image: cimg/node:18.20-browsers
-    working_directory: /home/circleci/project/node_workspace
-  node22_4-browsers:
     docker:
       - image: cimg/node:22.4-browsers
     working_directory: /home/circleci/project/node22_workspace
@@ -63,7 +59,6 @@ workflows:
             parameters:
               executor:
                 - node
-                - node22_4-browsers
           filters:
             tags:
               only: /^v\d+\.\d+\.\d+(-.+)?/
@@ -75,7 +70,6 @@ workflows:
             parameters:
               executor:
                 - node
-                - node22_4-browsers
           filters:
             tags:
               only: /^v\d+\.\d+\.\d+(-.+)?/
@@ -87,7 +81,6 @@ workflows:
             parameters:
               executor:
                 - node
-                - node22_4-browsers
           filters:
             tags:
               only: /^v\d+\.\d+\.\d+(-.+)?/
@@ -122,7 +115,6 @@ workflows:
             parameters:
               executor:
                 - node
-                - node22_4-browsers
       - tool-kit/build:
           name: tool-kit/build-<< matrix.executor >>
           requires:
@@ -131,7 +123,6 @@ workflows:
             parameters:
               executor:
                 - node
-                - node22_4-browsers
       - tool-kit/test:
           name: tool-kit/test-<< matrix.executor >>
           requires:
@@ -140,4 +131,3 @@ workflows:
             parameters:
               executor:
                 - node
-                - node22_4-browsers

package/.toolkitrc.yml

@@ -23,7 +23,6 @@ hooks:
 options:
   "@dotcom-tool-kit/circleci":
     nodeVersion:
-      - '18.20-browsers'
       - '22.4-browsers'
   "@dotcom-tool-kit/eslint": {
     files: "{,!(public)/**/}*.js"

package/.toolkitstate/ci.json

@@ -1,6 +1,6 @@
 {
   "branch": "",
   "repo": "n-myft-ui",
-  "version": "da6a59658803a3898eb4a2d02f16561dded775aa",
-  "tag": "v40.0.2"
+  "version": "98257025e278efb50a8bb294bbada6961c55696e",
+  "tag": "v40.1.0"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@financial-times/n-myft-ui",
-  "version": "40.0.2",
+  "version": "40.1.0",
   "description": "Client side component for interaction with myft",
   "main": "server.js",
   "scripts": {
@@ -133,7 +133,7 @@
     "node": "22.11.0"
   },
   "engines": {
-    "node": "18.x || 22.x"
+    "node": "22.x"
   },
   "x-dash": {
     "engine": {

package/components/unread-articles-indicator/README.md

@@ -1,5 +0,0 @@
-# n-myft-ui/unread-articles-indicator
-
-Note: this component is decommissioned but a small function (`getNewArticlesSinceTime`) of it is still in use by next-myft-page to display a "New" label under an article in the Timeline view only.
-
-All other previous functionality of the component has been removed.

package/components/unread-articles-indicator/date-fns.js

@@ -1,25 +0,0 @@
-// date-fns from v2 doesn't accept String arguments anymore.
-// the detail => https://github.com/date-fns/date-fns/blob/HEAD/CHANGELOG.md#200---2019-08-20
-// By adding validation for dates before their functions allows us to know it when unexpected value passed.
-
-import isTodayOriginal from 'date-fns/src/isToday';
-import isAfterOriginal from 'date-fns/src/isAfter';
-import addMinutesOriginal from 'date-fns/src/addMinutes';
-import startOfDayOriginal from 'date-fns/src/startOfDay';
-import isValidOriginal from 'date-fns/src/isValid';
-import parseISO from 'date-fns/src/parseISO';
-
-const isValid = (date) => {
-	if (!isValidOriginal(date)) {
-		console.error("Invalid date passed", [date]); //eslint-disable-line
-	}
-	return date;
-};
-
-const isToday = (date) => isTodayOriginal(isValid(date));
-const isAfter = (date, dateToCompare) =>
-	isAfterOriginal(isValid(date), isValid(dateToCompare));
-const addMinutes = (date, amount) => addMinutesOriginal(isValid(date), amount);
-const startOfDay = (date) => startOfDayOriginal(isValid(date));
-
-export { isToday, isAfter, addMinutes, startOfDay, isValid, parseISO };

package/components/unread-articles-indicator/device-session.js

@@ -1,19 +0,0 @@
-import { isAfter, addMinutes } from 'date-fns';
-import * as storage from './storage';
-const SESSION_THRESHOLD_MINUTES = 30;
-
-export default class DeviceSession {
-	constructor () {
-		this.expiry = storage.getDeviceSessionExpiry();
-		const newExpiry = addMinutes(new Date(), SESSION_THRESHOLD_MINUTES);
-		storage.setDeviceSessionExpiry(newExpiry);
-	}
-
-	isNewSession () {
-		if (this.expiry) {
-			return isAfter(new Date(), this.expiry);
-		} else {
-			return true;
-		}
-	}
-}

package/components/unread-articles-indicator/index.js

@@ -1,31 +0,0 @@
-import { startOfDay } from 'date-fns';
-import * as storage from './storage';
-import initialiseFeedStartTime from './initialise-feed-start-time';
-import sessionClient from 'next-session-client';
-
-let initialFeedStartTime;
-let userId;
-
-async function getValidSession () {
-	if (!userId) {
-		const { uuid } = await sessionClient.uuid();
-		if (!uuid) throw new Error('No userId');
-		userId = uuid;
-	}
-	return userId;
-}
-
-// Export used in next-myft -page to determine whether to add "New" label to articles in feed
-//KEEP: This function is in use in next-myft-page do not delete!
-export async function getNewArticlesSinceTime () {
-	const user = await getValidSession();
-	const dayStart = startOfDay(new Date());
-
-	if (!initialFeedStartTime && storage.isAvailable()) {
-		try {
-			initialFeedStartTime = await initialiseFeedStartTime(user, new Date());
-		} catch (e) {}
-	}
-
-	return initialFeedStartTime || dayStart;
-}

package/components/unread-articles-indicator/initialise-feed-start-time.js

@@ -1,54 +0,0 @@
-import { isToday, startOfDay } from 'date-fns';
-import DeviceSession from './device-session';
-import * as storage from './storage';
-import { json as fetchJson } from 'fetchres';
-
-const deviceSession = new DeviceSession();
-
-export const fetchUserLastVisitedAt = (userId) => {
-	const url = '/__myft/users/:userId/last-seen?source=next-myft'.replace(
-		':userId',
-		userId
-	);
-	return fetch(url, { credentials: 'include' })
-		.then(fetchJson)
-		.then(({ lastSeen }) => new Date(lastSeen));
-};
-
-/**
- * @param {Date} now  Date representing the time now
- * @param {Date} previousFeedStartTime  Date representing the time we last used to determine if articles are new for the user
- * @return {Promise<Date>} date when we now determine articles to be 'new' for the user
- */
-const determineFeedStartTime = (userId, now, previousFeedStartTime) => {
-	if (
-		previousFeedStartTime &&
-		isToday(previousFeedStartTime) &&
-		!deviceSession.isNewSession()
-	) {
-		return Promise.resolve(previousFeedStartTime);
-	}
-
-	return fetchUserLastVisitedAt(userId)
-		.then((userLastVisitedAt) =>
-			isToday(userLastVisitedAt) ? userLastVisitedAt : Promise.reject(new Error('No last visited user data today'))
-		)
-		.catch(() => startOfDay(now));
-};
-
-/**
- * Sets/updates the time after which articles must be published to count towards indicator count
- * @param {Date} now  Date representing the time now
- */
-export default (userId, now) => {
-	const previousFeedStartTime =
-		storage.isAvailable() && storage.getFeedStartTime();
-	return determineFeedStartTime(userId, now, previousFeedStartTime).then(
-		(startTime) => {
-			if (storage.isAvailable()) {
-				storage.setFeedStartTime(startTime);
-			}
-			return startTime;
-		}
-	);
-};

package/components/unread-articles-indicator/storage.js

@@ -1,35 +0,0 @@
-import { isValid } from 'date-fns';
-
-const DEVICE_SESSION_EXPIRY = 'deviceSessionExpiry';
-const FEED_START_TIME = 'newArticlesSinceTime';
-
-const isISOString = (str) => typeof str === 'string' && str.charAt(10) === 'T';
-const getStoredDate = (key) => {
-	const value = window.localStorage.getItem(key);
-	const date = new Date(value);
-
-	return isISOString(value) && isValid(date) ? date : null;
-};
-
-export const getDeviceSessionExpiry = () =>
-	getStoredDate(DEVICE_SESSION_EXPIRY);
-
-export const setDeviceSessionExpiry = (date) =>
-	window.localStorage.setItem(DEVICE_SESSION_EXPIRY, date.toISOString());
-
-export const getFeedStartTime = () => getStoredDate(FEED_START_TIME);
-
-export const setFeedStartTime = (date) =>
-	window.localStorage.setItem(FEED_START_TIME, date.toISOString());
-
-export const isAvailable = () => {
-	try {
-		const storage = window.localStorage;
-		const x = '__storage_test__';
-		storage.setItem(x, x);
-		storage.removeItem(x);
-		return true;
-	} catch (e) {
-		return false;
-	}
-};

package/test/unread-articles-indicator/device-session.spec.js

@@ -1,97 +0,0 @@
-/* global expect */
-import sinon from 'sinon';
-import addMinutes from 'date-fns/addMinutes';
-import parseISO from 'date-fns/parseISO';
-
-const expiryTimestamp = parseISO('2018-06-14T12:00:00.000Z');
-const timestampBeforeExpiry = '2018-06-14T11:40:00.000Z';
-const timestampAfterExpiry = '2018-06-14T12:10:00.000Z';
-
-const SESSION_THRESHOLD_MINUTES = 30;
-
-describe('DeviceSession', () => {
-
-	let clock;
-	let deviceSession;
-	let deviceSessionExpiry;
-
-	const mockStorage = {
-		getDeviceSessionExpiry: sinon.stub().callsFake(() => deviceSessionExpiry),
-		setDeviceSessionExpiry: sinon.spy()
-	};
-
-	const subjectInjector = require('inject-loader!../../components/unread-articles-indicator/device-session');
-	const DeviceSession = subjectInjector({
-		'./storage': mockStorage
-	});
-
-	afterEach(() => {
-		deviceSessionExpiry = undefined;
-		clock.restore();
-		sinon.resetHistory();
-	});
-
-	describe('constructor:', () => {
-
-		const timeNow = timestampBeforeExpiry;
-
-		beforeEach(() => {
-			clock = sinon.useFakeTimers(new Date(timeNow));
-			deviceSessionExpiry = expiryTimestamp;
-			deviceSession = new DeviceSession();
-		});
-
-		it('should set this.expiry a timestamp from localStorage', () => {
-			expect(mockStorage.getDeviceSessionExpiry).to.have.been.calledOnce;
-			expect(deviceSession.expiry).to.equal(expiryTimestamp);
-		});
-
-		it('should call storage.setDeviceSessionExpiry with correct timestamp', () => {
-			const correctTimestamp = addMinutes(parseISO(timeNow), SESSION_THRESHOLD_MINUTES);
-
-			expect(mockStorage.setDeviceSessionExpiry).to.have.been.calledOnce;
-			expect(mockStorage.setDeviceSessionExpiry).to.have.been.calledWith(correctTimestamp);
-		});
-	});
-
-	describe('isNewSession:', () => {
-
-		it('should return true if the user visits for the first time (initially no value in localStorage)', () => {
-			const timeNow = '2018-06-14T08:00:00.000Z';
-			const newExpiryTimestamp = addMinutes(parseISO(timeNow), SESSION_THRESHOLD_MINUTES);
-
-			clock = sinon.useFakeTimers(new Date(timeNow));
-			deviceSessionExpiry = undefined;
-			deviceSession = new DeviceSession();
-
-			expect(deviceSession.isNewSession()).to.be.true;
-			expect(mockStorage.setDeviceSessionExpiry).to.have.been.calledWith(newExpiryTimestamp);
-		});
-
-		it('should return true if the user returns after session (initial value in localStorage is in the past)', () => {
-			const timeNow = timestampAfterExpiry;
-			const newExpiryTimestamp = addMinutes(parseISO(timeNow), SESSION_THRESHOLD_MINUTES);
-
-			clock = sinon.useFakeTimers(new Date(timeNow));
-			deviceSessionExpiry = expiryTimestamp;
-			deviceSession = new DeviceSession();
-
-			expect(deviceSession.isNewSession()).to.be.true;
-			expect(mockStorage.setDeviceSessionExpiry).to.have.been.calledWith(newExpiryTimestamp);
-		});
-
-		it('should return false if the user returns within session (initial value in localStorage is in the future)', () => {
-			const timeNow = timestampBeforeExpiry;
-			const newExpiryTimestamp = addMinutes(parseISO(timeNow), SESSION_THRESHOLD_MINUTES);
-
-			clock = sinon.useFakeTimers(new Date(timeNow));
-			deviceSessionExpiry = expiryTimestamp;
-			deviceSession = new DeviceSession();
-
-			expect(deviceSession.isNewSession()).to.be.false;
-			expect(mockStorage.setDeviceSessionExpiry).to.have.been.calledWith(newExpiryTimestamp);
-		});
-
-	});
-
-});

package/test/unread-articles-indicator/index.spec.js

@@ -1,43 +0,0 @@
-/* global expect */
-import sinon from 'sinon';
-
-const FEED_START_TIME = '2018-06-05T10:00:00.000Z';
-
-describe('unread stories indicator', () => {
-	let unreadArticlesComponent;
-	let mockInitialiseFeedStartTime;
-	let mockStorage;
-	let isStorageAvailable;
-
-	beforeEach(() => {
-		mockStorage = {
-			getFeedStartTime: sinon.stub().returns(FEED_START_TIME),
-			setFeedStartTime: sinon.stub(),
-			isAvailable: sinon.stub().callsFake(() => isStorageAvailable)
-		};
-
-		mockInitialiseFeedStartTime = sinon.stub().resolves();
-
-		unreadArticlesComponent = require('inject-loader!../../components/unread-articles-indicator')({
-			'next-session-client': {
-				uuid: sinon.stub().resolves({uuid: '00000000-0000-0000-0000-000000000000'})
-			},
-			'./storage': mockStorage,
-			'./initialise-feed-start-time': mockInitialiseFeedStartTime
-		});
-	});
-
-	describe('getNewsArticlesSinceTime', () => {
-		it('should not do anything if storage is not available', () => {
-			isStorageAvailable = false;
-			unreadArticlesComponent.getNewArticlesSinceTime();
-			expect(mockInitialiseFeedStartTime).to.not.have.been.called;
-		});
-
-		it('should initialise feed start time', () => {
-			isStorageAvailable = true;
-			return unreadArticlesComponent.getNewArticlesSinceTime();
-			expect(mockInitialiseFeedStartTime).to.have.been.calledOnce;
-		});
-	});
-});

package/test/unread-articles-indicator/initialise-feed-start-time.spec.js

@@ -1,178 +0,0 @@
-/* global expect */
-
-import sinon from 'sinon';
-import fetchMock from 'fetch-mock';
-
-const mockDateFns = {
-	isToday (x) {
-		return x && x.toISOString().startsWith('2018-03-05T');
-	},
-	startOfDay (x) {
-		return new Date(x.toISOString().slice(0,11)+'00:00:00Z');
-	}
-};
-
-const TIME_NOW = new Date('2018-03-05T16:00:00Z');
-const START_OF_TODAY = new Date('2018-03-05T00:00:00Z');
-const VISIT_TIME_TODAY = new Date('2018-03-05T14:00:00Z');
-const VISIT_TIME_YESTERDAY = new Date('2018-03-04T14:00:00Z');
-const FEED_TIME_TODAY = new Date('2018-03-05T12:00:00Z');
-const FEED_TIME_YESTERDAY = new Date('2018-03-04T12:00:00Z');
-const USER_ID = '00000000-0000-0000-0000-000000000000';
-
-const mockSetFeedStartTime = sinon.stub();
-
-
-describe('initialiseFeedStartTime', () => {
-	let isNewSession;
-	let feedStartTime;
-	let lastVisitTime;
-	const injector = require('inject-loader!../../components/unread-articles-indicator/initialise-feed-start-time');
-	const initialiseFeedStartTime = injector({
-		'date-fns': {
-			isToday: mockDateFns.isToday,
-			startOfDay: mockDateFns.startOfDay
-		},
-		'./device-session': () => ({ isNewSession: () => isNewSession }),
-		'./storage': {
-			setFeedStartTime: mockSetFeedStartTime,
-			getFeedStartTime: sinon.stub().callsFake( () => feedStartTime ),
-			isAvailable: sinon.stub().returns(true)
-		}
-	}).default;
-
-	const expectStartTime = startTime => expect(mockSetFeedStartTime.firstCall.args[0].toISOString()).equal(startTime.toISOString());
-
-	before(() => {
-		fetchMock.get(`begin:/__myft/users/${USER_ID}/last-seen`, () => ({
-			lastSeen: lastVisitTime && lastVisitTime.toISOString()
-		}));
-	});
-	after(() => {
-		fetchMock.restore();
-	});
-
-	context('with no previous feed start time', () => {
-		before(() => {
-			feedStartTime = undefined;
-		});
-		context('and previous visit today', () => {
-			before(() => {
-				mockSetFeedStartTime.resetHistory();
-				lastVisitTime = VISIT_TIME_TODAY;
-				return initialiseFeedStartTime(USER_ID, TIME_NOW);
-			});
-			it('sets start time to visit time', () => {
-				expectStartTime(VISIT_TIME_TODAY);
-			});
-		});
-		context('and previous visit yesterday', () => {
-			before(() => {
-				mockSetFeedStartTime.resetHistory();
-				lastVisitTime = VISIT_TIME_YESTERDAY;
-				return initialiseFeedStartTime(USER_ID, TIME_NOW);
-			});
-			it('sets start time to start of today', () => {
-				expectStartTime(START_OF_TODAY);
-			});
-		});
-		context('and no previous visit', () => {
-			before(() => {
-				mockSetFeedStartTime.resetHistory();
-				lastVisitTime = undefined;
-				return initialiseFeedStartTime(USER_ID, TIME_NOW);
-			});
-			it('sets start time to start of today', () => {
-				expectStartTime(START_OF_TODAY);
-			});
-		});
-	});
-	context('with previous feed start time today', () => {
-		before(() => {
-			feedStartTime = FEED_TIME_TODAY;
-		});
-		context('and no new session', () => {
-			before(() => {
-				mockSetFeedStartTime.resetHistory();
-				lastVisitTime = VISIT_TIME_TODAY;
-				return initialiseFeedStartTime(USER_ID, );
-			});
-			it('sets start time to previous feed start time', () => {
-				expectStartTime(FEED_TIME_TODAY);
-			});
-		});
-		context('and new session started', () => {
-			before(() => {
-				isNewSession = true;
-			});
-			after(() => {
-				isNewSession = false;
-			});
-			context('and previous visit today', () => {
-				before(() => {
-					mockSetFeedStartTime.resetHistory();
-					lastVisitTime = VISIT_TIME_TODAY;
-					return initialiseFeedStartTime(USER_ID, TIME_NOW);
-				});
-				it('sets start time to visit time', () => {
-					expectStartTime(VISIT_TIME_TODAY);
-				});
-			});
-			context('and previous visit yesterday', () => {
-				before(() => {
-					mockSetFeedStartTime.resetHistory();
-					lastVisitTime = VISIT_TIME_YESTERDAY;
-					return initialiseFeedStartTime(USER_ID, TIME_NOW);
-				});
-				it('sets start time to start of today', () => {
-					expectStartTime(START_OF_TODAY);
-				});
-			});
-			context('and no previous visit', () => {
-				before(() => {
-					mockSetFeedStartTime.resetHistory();
-					lastVisitTime = undefined;
-					return initialiseFeedStartTime(USER_ID, TIME_NOW);
-				});
-				it('sets start time to start of today', () => {
-					expectStartTime(START_OF_TODAY);
-				});
-			});
-		});
-	});
-	context('with previous feed start time yesterday', () => {
-		before(() => {
-			feedStartTime = FEED_TIME_YESTERDAY;
-		});
-		context('and previous visit today', () => {
-			before(() => {
-				mockSetFeedStartTime.resetHistory();
-				lastVisitTime = VISIT_TIME_TODAY;
-				return initialiseFeedStartTime(USER_ID, TIME_NOW);
-			});
-			it('sets start time to visit time', () => {
-				expectStartTime(VISIT_TIME_TODAY);
-			});
-		});
-		context('and previous visit yesterday', () => {
-			before(() => {
-				mockSetFeedStartTime.resetHistory();
-				lastVisitTime = VISIT_TIME_YESTERDAY;
-				return initialiseFeedStartTime(USER_ID, TIME_NOW);
-			});
-			it('sets start time to start of today', () => {
-				expectStartTime(START_OF_TODAY);
-			});
-		});
-		context('and no previous visit', () => {
-			before(() => {
-				mockSetFeedStartTime.resetHistory();
-				lastVisitTime = undefined;
-				return initialiseFeedStartTime(USER_ID, TIME_NOW);
-			});
-			it('sets start time to start of today', () => {
-				expectStartTime(START_OF_TODAY);
-			});
-		});
-	});
-});

package/test/unread-articles-indicator/storage.spec.js

@@ -1,142 +0,0 @@
-/* global expect */
-
-import sinon from 'sinon';
-import * as storage from '../../components/unread-articles-indicator/storage';
-
-describe('storage', () => {
-	let clock;
-	let mockStorage;
-	let now;
-
-	beforeEach(() => {
-		mockStorage = {};
-		now = new Date();
-		sinon.stub(window.Storage.prototype, 'getItem').callsFake(key => mockStorage[key]);
-		sinon.stub(window.Storage.prototype, 'setItem').callsFake((key, value) => mockStorage[key] = value);
-		clock = sinon.useFakeTimers(now);
-	});
-
-	afterEach(() => {
-		window.Storage.prototype.getItem.restore();
-		window.Storage.prototype.setItem.restore();
-		clock.restore();
-	});
-
-	describe('getDeviceSessionExpiry', () => {
-		describe('given a valid iso date is stored', () => {
-			beforeEach(() => {
-				mockStorage.deviceSessionExpiry = now.toISOString();
-			});
-
-			it('should return the correct iso date', () => {
-				expect(storage.getDeviceSessionExpiry().toISOString()).to.equal(now.toISOString());
-			});
-		});
-
-		describe('given no value is stored', () => {
-			beforeEach(() => {
-				mockStorage.deviceSessionExpiry = null;
-			});
-
-			it('should return null', () => {
-				expect(storage.getDeviceSessionExpiry()).to.equal(null);
-			});
-		});
-
-		describe('given an invalid value is stored', () => {
-			beforeEach(() => {
-				mockStorage.deviceSessionExpiry = 'abc';
-			});
-
-			it('should return null', () => {
-				expect(storage.getDeviceSessionExpiry()).to.equal(null);
-			});
-		});
-
-		describe('given a unix timestamp is stored (old format)', () => {
-			beforeEach(() => {
-				mockStorage.deviceSessionExpiry = String(now.getTime());
-			});
-
-			it('should return null', () => {
-				expect(storage.getDeviceSessionExpiry()).to.equal(null);
-			});
-		});
-	});
-
-	describe('setDeviceSessionExpiry', () => {
-		it('should store the date as an iso string', () => {
-			const date = new Date(2018, 6, 14, 11, 0, 0);
-
-			storage.setDeviceSessionExpiry(date);
-
-			expect(mockStorage.deviceSessionExpiry).to.equal(date.toISOString());
-		});
-	});
-
-	describe('getFeedStartTime', () => {
-		describe('given a valid iso date is stored', () => {
-			beforeEach(() => {
-				mockStorage.newArticlesSinceTime = now.toISOString();
-			});
-
-			it('should return the correct iso date', () => {
-				expect(storage.getFeedStartTime().toISOString()).to.equal(now.toISOString());
-			});
-		});
-
-		describe('given no value is stored', () => {
-			beforeEach(() => {
-				mockStorage.newArticlesSinceTime = null;
-			});
-
-			it('should return null', () => {
-				expect(storage.getFeedStartTime()).to.equal(null);
-			});
-		});
-
-		describe('given an invalid value is stored', () => {
-			beforeEach(() => {
-				mockStorage.newArticlesSinceTime = 'abc';
-			});
-
-			it('should return null', () => {
-				expect(storage.getFeedStartTime()).to.equal(null);
-			});
-		});
-
-		describe('given a unix timestamp is stored (old format)', () => {
-			beforeEach(() => {
-				mockStorage.newArticlesSinceTime = String(now.getTime());
-			});
-
-			it('should return the correct iso date', () => {
-				expect(storage.getFeedStartTime()).to.equal(null);
-			});
-		});
-	});
-
-	describe('setFeedStartTime', () => {
-		it('should store the date as an iso string', () => {
-			const date = new Date(2018, 5, 1, 11, 30, 0);
-
-			storage.setFeedStartTime(date);
-
-			expect(mockStorage.newArticlesSinceTime).to.equal(date.toISOString());
-		});
-	});
-
-	describe('isAvailable', () => {
-		it('should return true if it is available', () => {
-			sinon.stub(window.Storage.prototype, 'removeItem').callsFake((key) => delete mockStorage[key]);
-
-			expect(storage.isAvailable()).to.be.true;
-		});
-
-		it('should return false if it is not available', () => {
-			mockStorage = null;
-
-			expect(storage.isAvailable()).to.be.false;
-		});
-	});
-});