diff --git a/CHANGELOG.md b/CHANGELOG.md index 71ff135..49f9338 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,6 +5,7 @@ - Upgrade dependencies to last versions - Publish MIT license to NPM with the code (#224) - `universal-cookie`: Add support for generic type with reading cookies (#222) +- `universal-cookie`: Disable accessing browser cookies on JSDOM (#227) - `react-cookie`: Add `WrappedComponent` static property when using `withCookies` (#225) - `react-cookie`: Fix display name to include the original component name diff --git a/packages/universal-cookie/src/Cookies.ts b/packages/universal-cookie/src/Cookies.ts index 8be7b8c..11575ee 100644 --- a/packages/universal-cookie/src/Cookies.ts +++ b/packages/universal-cookie/src/Cookies.ts @@ -17,11 +17,14 @@ export default class Cookies { private cookies: { [name: string]: Cookie }; private changeListeners: CookieChangeListener[] = []; - private HAS_DOCUMENT_COOKIE: boolean; + private HAS_DOCUMENT_COOKIE: boolean = false; constructor(cookies?: string | object | null) { this.cookies = parseCookies(cookies); - this.HAS_DOCUMENT_COOKIE = hasDocumentCookie(); + + new Promise(() => { + this.HAS_DOCUMENT_COOKIE = hasDocumentCookie(); + }).catch(() => {}); } private _updateBrowserValues() { diff --git a/packages/universal-cookie/src/utils.ts b/packages/universal-cookie/src/utils.ts index 93a139c..71b408e 100644 --- a/packages/universal-cookie/src/utils.ts +++ b/packages/universal-cookie/src/utils.ts @@ -2,10 +2,29 @@ import * as cookie from 'cookie'; import { Cookie, CookieGetOptions } from './types'; export function hasDocumentCookie() { + // JSDOM does not support changing cookies, disable it for tests + if (isJsDom()) { + return false; + } + // Can we get/set cookies on document.cookie? return typeof document === 'object' && typeof document.cookie === 'string'; } +function isJsDom(): boolean { + if ( + typeof navigator !== 'object' || + typeof navigator.userAgent !== 'string' + ) { + return false; + } + + return ( + navigator.userAgent.indexOf('Node.js') >= 0 || + navigator.userAgent.indexOf('jsdom') >= 0 + ); +} + export function cleanCookies() { document.cookie.split(';').forEach(function(c) { document.cookie = c