Fix up tests

react/src/context/PostHogProvider.tsx

@@ -18,13 +18,13 @@ export function PostHogProvider({
     const posthog = useMemo(() => {
         if (client && apiKey) {
             console.warn(
-                '[PostHog] You have provided both a client and an apiKey to PostHogProvider. The apiKey will be ignored in favour of the client.'
+                '[PostHog.js] You have provided both a client and an apiKey to PostHogProvider. The apiKey will be ignored in favour of the client.'
             )
         }
 
         if (client && options) {
             console.warn(
-                '[PostHog] You have provided both a client and options to PostHogProvider. The options will be ignored in favour of the client.'
+                '[PostHog.js] You have provided both a client and options to PostHogProvider. The options will be ignored in favour of the client.'
             )
         }
 
@@ -34,7 +34,7 @@ export function PostHogProvider({
 
         if (apiKey) {
             if (posthogJs.__loaded) {
-                console.warn('posthog was already loaded elsewhere. This may cause issues.')
+                console.warn('[PostHog.js] was already loaded elsewhere. This may cause issues.')
             }
             posthogJs.init(apiKey, options)
         }

src/__tests__/decide.js

@@ -25,6 +25,7 @@ describe('Decide', () => {
     given('decide', () => new Decide(given.posthog))
     given('posthog', () => ({
         config: given.config,
+        debug: true,
         persistence: new PostHogPersistence(given.config),
         register: (props) => given.posthog.persistence.register(props),
         unregister: (key) => given.posthog.persistence.unregister(key),
@@ -183,12 +184,13 @@ describe('Decide', () => {
 
         it('Make sure receivedFeatureFlags is not called if the decide response fails', () => {
             given('decideResponse', () => ({ status: 0 }))
+            window.POSTHOG_DEBUG = true
             console.error = jest.fn()
 
             given.subject()
 
             expect(given.posthog.featureFlags.receivedFeatureFlags).not.toHaveBeenCalled()
-            expect(console.error).toHaveBeenCalledWith('Failed to fetch feature flags from PostHog.')
+            expect(console.error).toHaveBeenCalledWith('[PostHog.js]', 'Failed to fetch feature flags from PostHog.')
         })
 
         it('Make sure receivedFeatureFlags is called with empty if advanced_disable_feature_flags_on_first_load is set', () => {
@@ -221,13 +223,14 @@ describe('Decide', () => {
         })
 
         it('does not run site apps code if not opted in', () => {
+            window.POSTHOG_DEBUG = true
             given('config', () => ({ api_host: 'https://test.com', opt_in_site_apps: false, persistence: 'memory' }))
             given('decideResponse', () => ({ siteApps: [{ id: 1, url: '/site_app/1/tokentoken/hash/' }] }))
             expect(() => {
                 given.subject()
             }).toThrow(
                 // throwing only in tests, just an error in production
-                'Unexpected console.error: PostHog site apps are disabled. Enable the "opt_in_site_apps" config to proceed.'
+                'Unexpected console.error: [PostHog.js],PostHog site apps are disabled. Enable the "opt_in_site_apps" config to proceed.'
             )
         })
     })

src/__tests__/featureflags.js

@@ -67,20 +67,23 @@ describe('featureflags', () => {
     })
 
     it('should warn if decide endpoint was not hit and no flags exist', () => {
+        window.POSTHOG_DEBUG = true
         given.featureFlags.instance.decideEndpointWasHit = false
         given.instance.persistence.unregister('$enabled_feature_flags')
         given.instance.persistence.unregister('$active_feature_flags')
 
         expect(given.featureFlags.getFlags()).toEqual([])
         expect(given.featureFlags.isFeatureEnabled('beta-feature')).toEqual(undefined)
         expect(window.console.warn).toHaveBeenCalledWith(
+            '[PostHog.js]',
             'isFeatureEnabled for key "beta-feature" failed. Feature flags didn\'t load in time.'
         )
 
         window.console.warn.mockClear()
 
         expect(given.featureFlags.getFeatureFlag('beta-feature')).toEqual(undefined)
         expect(window.console.warn).toHaveBeenCalledWith(
+            '[PostHog.js]',
             'getFeatureFlag for key "beta-feature" failed. Feature flags didn\'t load in time.'
         )
     })

src/__tests__/gdpr-utils.js

@@ -469,138 +469,4 @@ describe(`GDPR utils`, () => {
             })
         })
     })
-
-    describe(`addOptOutCheckPostHogLib`, () => {
-        const captureEventName = `єνєηт`
-        const captureProperties = { '𝖕𝖗𝖔𝖕𝖊𝖗𝖙𝖞': `𝓿𝓪𝓵𝓾𝓮` }
-        let capture, postHogLib
-
-        function setupMocks(config, silenceErrors = false) {
-            capture = sinon.spy()
-            postHogLib = {
-                config,
-                capture: undefined,
-            }
-            postHogLib.capture = gdpr.addOptOutCheck(postHogLib, capture, silenceErrors)
-        }
-
-        forPersistenceTypes(function (persistenceType) {
-            it(`should call the wrapped method if the user is neither opted in or opted out`, () => {
-                TOKENS.forEach((token) => {
-                    setupMocks({ token, opt_out_capturing_persistence_type: persistenceType })
-
-                    postHogLib.capture(captureEventName, captureProperties)
-
-                    expect(capture.calledOnceWith(captureEventName, captureProperties)).toBe(true)
-                })
-            })
-
-            it(`should call the wrapped method if the user is opted in`, () => {
-                TOKENS.forEach((token) => {
-                    setupMocks({ token, opt_out_capturing_persistence_type: persistenceType })
-
-                    gdpr.optIn(token, { persistenceType })
-                    postHogLib.capture(captureEventName, captureProperties)
-
-                    expect(capture.calledOnceWith(captureEventName, captureProperties)).toBe(true)
-                })
-            })
-
-            it(`should not call the wrapped method if the user is opted out`, () => {
-                TOKENS.forEach((token) => {
-                    setupMocks({ token, opt_out_capturing_persistence_type: persistenceType })
-
-                    gdpr.optOut(token, { persistenceType })
-                    postHogLib.capture(captureEventName, captureProperties)
-
-                    expect(capture.notCalled).toBe(true)
-                })
-            })
-
-            it(`should not invoke the callback directly if the user is neither opted in or opted out`, () => {
-                TOKENS.forEach((token) => {
-                    setupMocks({ token, opt_out_capturing_persistence_type: persistenceType })
-                    const callback = sinon.spy()
-
-                    postHogLib.capture(captureEventName, captureProperties, callback)
-
-                    expect(callback.notCalled).toBe(true)
-                })
-            })
-
-            it(`should not invoke the callback directly if the user is opted in`, () => {
-                TOKENS.forEach((token) => {
-                    setupMocks({ token, opt_out_capturing_persistence_type: persistenceType })
-                    const callback = sinon.spy()
-
-                    gdpr.optIn(token, { persistenceType })
-                    postHogLib.capture(captureEventName, captureProperties, callback)
-
-                    expect(callback.notCalled).toBe(true)
-                })
-            })
-
-            it(`should invoke the callback directly if the user is opted out`, () => {
-                TOKENS.forEach((token) => {
-                    setupMocks({ token, opt_out_capturing_persistence_type: persistenceType })
-                    const callback = sinon.spy()
-
-                    gdpr.optOut(token, { persistenceType })
-                    postHogLib.capture(captureEventName, captureProperties, callback)
-
-                    expect(callback.calledOnceWith(0)).toBe(true)
-                })
-            })
-
-            it(`should call the wrapped method if there is no token available`, () => {
-                TOKENS.forEach((token) => {
-                    setupMocks({ token: null, opt_out_capturing_persistence_type: persistenceType })
-
-                    gdpr.optIn(token, { persistenceType })
-                    postHogLib.capture(captureEventName, captureProperties)
-
-                    expect(capture.calledOnceWith(captureEventName, captureProperties)).toBe(true)
-                })
-            })
-
-            it(`should call the wrapped method if config is undefined`, () => {
-                TOKENS.forEach((token) => {
-                    setupMocks(undefined, false)
-                    console.error = jest.fn()
-
-                    gdpr.optIn(token, { persistenceType })
-                    postHogLib.capture(captureEventName, captureProperties)
-
-                    expect(capture.calledOnceWith(captureEventName, captureProperties)).toBe(true)
-                    // :KLUDGE: Exact error message may vary between runtimes
-                    expect(console.error).toHaveBeenCalled()
-                })
-            })
-
-            it(`should allow use of a custom "persistence prefix" string`, () => {
-                TOKENS.forEach((token) => {
-                    setupMocks({
-                        token,
-                        opt_out_capturing_persistence_type: persistenceType,
-                        opt_out_capturing_cookie_prefix: CUSTOM_PERSISTENCE_PREFIX,
-                    })
-
-                    gdpr.optOut(token, { persistenceType, persistencePrefix: CUSTOM_PERSISTENCE_PREFIX })
-                    postHogLib.capture(captureEventName, captureProperties)
-
-                    expect(capture.notCalled).toBe(true)
-
-                    gdpr.optIn(token, { persistenceType })
-                    postHogLib.capture(captureEventName, captureProperties)
-
-                    expect(capture.notCalled).toBe(true)
-
-                    gdpr.optIn(token, { persistenceType, persistencePrefix: CUSTOM_PERSISTENCE_PREFIX })
-                    postHogLib.capture(captureEventName, captureProperties)
-
-                    expect(capture.calledOnceWith(captureEventName, captureProperties)).toBe(true)
-                })
-            })
-        })
-    })
 })

src/__tests__/posthog-core.identify.js

@@ -225,11 +225,15 @@ describe('identify()', () => {
         it('does not update user', () => {
             console.error = jest.fn()
 
+            given.lib.debug()
             given.subject()
 
             expect(given.overrides.capture).not.toHaveBeenCalled()
             expect(given.overrides.register).not.toHaveBeenCalled()
-            expect(console.error).toHaveBeenCalledWith('Unique user id has not been set in posthog.identify')
+            expect(console.error).toHaveBeenCalledWith(
+                '[PostHog.js]',
+                'Unique user id has not been set in posthog.identify'
+            )
         })
     })
 

src/__tests__/posthog-core.js

@@ -19,6 +19,7 @@ jest.mock('../utils', () => ({
 given('lib', () => {
     const posthog = new PostHog()
     posthog._init('testtoken', given.config, 'testhog')
+    posthog.debug()
     return Object.assign(posthog, given.overrides)
 })
 
@@ -111,7 +112,7 @@ describe('capture()', () => {
 
         expect(() => given.subject()).not.toThrow()
         expect(hook).not.toHaveBeenCalled()
-        expect(console.error).toHaveBeenCalledWith('No event name provided to posthog.capture')
+        expect(console.error).toHaveBeenCalledWith('[PostHog.js]', 'No event name provided to posthog.capture')
     })
 
     it('errors with object event name', () => {
@@ -123,7 +124,7 @@ describe('capture()', () => {
 
         expect(() => given.subject()).not.toThrow()
         expect(hook).not.toHaveBeenCalled()
-        expect(console.error).toHaveBeenCalledWith('No event name provided to posthog.capture')
+        expect(console.error).toHaveBeenCalledWith('[PostHog.js]', 'No event name provided to posthog.capture')
     })
 
     it('truncates long properties', () => {
@@ -509,6 +510,7 @@ describe('bootstrapping feature flags', () => {
         expect(given.lib.get_distinct_id()).not.toEqual(undefined)
         expect(given.lib.getFeatureFlag('multivariant')).toBe(undefined)
         expect(console.warn).toHaveBeenCalledWith(
+            '[PostHog.js]',
             expect.stringContaining('getFeatureFlag for key "multivariant" failed')
         )
         expect(given.lib.getFeatureFlag('disabled')).toBe(undefined)
@@ -861,6 +863,7 @@ describe('group()', () => {
 
         it('handles blank keys being passed', () => {
             window.console.error = jest.fn()
+            window.console.warn = jest.fn()
 
             given.lib.group(null, 'foo')
             given.lib.group('organization', null)
@@ -931,7 +934,7 @@ describe('_loaded()', () => {
 
         given.subject()
 
-        expect(console.error).toHaveBeenCalledWith('`loaded` function failed', expect.anything())
+        expect(console.error).toHaveBeenCalledWith('[PostHog.js]', '`loaded` function failed', expect.anything())
     })
 
     describe('/decide', () => {

src/__tests__/sessionid.js

@@ -272,6 +272,7 @@ describe('Session ID manager', () => {
         })
 
         it('uses the custom session_idle_timeout_seconds if within bounds', () => {
+            window.POSTHOG_DEBUG = true
             expect(mockSessionManager(61)._sessionTimeoutMs).toEqual(61 * 1000)
             expect(console.warn).toBeCalledTimes(0)
             expect(mockSessionManager(59)._sessionTimeoutMs).toEqual(60 * 1000)

src/__tests__/setup.js

@@ -1,8 +1,8 @@
 beforeEach(() => {
-    console.error = (message) => {
-        throw new Error(`Unexpected console.error: ${message}`)
+    console.error = (...args) => {
+        throw new Error(`Unexpected console.error: ${args}`)
     }
-    console.warn = (message) => {
-        throw new Error(`Unexpected console.warn: ${message}`)
+    console.warn = (...args) => {
+        throw new Error(`Unexpected console.warn: ${args}`)
     }
 })

src/utils.ts

@@ -21,6 +21,8 @@ const nativeForEach = ArrayProto.forEach,
     nativeIsArray = Array.isArray,
     breaker: Breaker = {}
 
+const LOGGER_PREFIX = '[PostHog.js]'
+
 export const logger = {
     _log: (level: 'log' | 'warn' | 'error', ...args: any[]) => {
         if ((Config.DEBUG || (window as any).POSTHOG_DEBUG) && !_isUndefined(window.console) && window.console) {
@@ -30,7 +32,7 @@ export const logger = {
                     : window.console[level]
 
             // eslint-disable-next-line no-console
-            consoleLog('[PostHog.js]', ...args)
+            consoleLog(LOGGER_PREFIX, ...args)
         }
     },
 
@@ -43,13 +45,13 @@ export const logger = {
     },
 
     error: (...args: any[]) => {
-        logger._log('warn', ...args)
+        logger._log('error', ...args)
     },
 
     critical: (...args: any[]) => {
         // Critical errors are always logged to the console
         // eslint-disable-next-line no-console
-        console.error('[PostHog]', ...args)
+        console.error(LOGGER_PREFIX, ...args)
     },
 
     unintializedWarning: (methodName: string) => {