Improved error message logging in console

.changeset/spotty-files-wave.md

@@ -0,0 +1,5 @@
+---
+'@segment/analytics-next': minor
+---
+
+Enhances console error logging when requests to settings api fail

packages/browser/package.json

@@ -43,7 +43,7 @@
   "size-limit": [
     {
       "path": "dist/umd/index.js",
-      "limit": "25.9 KB"
+      "limit": "25.95 KB"
     }
   ],
   "dependencies": {

packages/browser/src/browser/__tests__/analytics-pre-init.integration.test.ts

@@ -5,7 +5,6 @@ import { Context } from '../../core/context'
 import * as Factory from '../../test-helpers/factories'
 import { sleep } from '../../test-helpers/sleep'
 import { setGlobalCDNUrl } from '../../lib/parse-cdn'
-import { User } from '../../core/user'
 
 jest.mock('unfetch')
 
@@ -60,38 +59,19 @@ describe('Pre-initialization', () => {
       expect(trackSpy).toBeCalledTimes(1)
     })
 
-    test('"return types should not change over the lifecycle for async methods', async () => {
+    test('"return types should not change over the lifecycle for ordinary methods', async () => {
       const ajsBrowser = AnalyticsBrowser.load({ writeKey })
 
       const trackCtxPromise1 = ajsBrowser.track('foo', { name: 'john' })
       expect(trackCtxPromise1).toBeInstanceOf(Promise)
-      await ajsBrowser
+      const ctx1 = await trackCtxPromise1
+      expect(ctx1).toBeInstanceOf(Context)
 
       // loaded
       const trackCtxPromise2 = ajsBrowser.track('foo', { name: 'john' })
       expect(trackCtxPromise2).toBeInstanceOf(Promise)
-
-      expect(await trackCtxPromise1).toBeInstanceOf(Context)
-      expect(await trackCtxPromise2).toBeInstanceOf(Context)
-    })
-
-    test('return types should not change over lifecycle for sync methods', async () => {
-      const ajsBrowser = AnalyticsBrowser.load({ writeKey })
-      const user = ajsBrowser.user()
-      expect(user).toBeInstanceOf(Promise)
-      await ajsBrowser
-
-      // loaded
-      const user2 = ajsBrowser.user()
-      expect(user2).toBeInstanceOf(Promise)
-
-      expect(await user).toBeInstanceOf(User)
-      expect(await user2).toBeInstanceOf(User)
-    })
-
-    test('version should return version', async () => {
-      const ajsBrowser = AnalyticsBrowser.load({ writeKey })
-      expect(typeof ajsBrowser.VERSION).toBe('string')
+      const ctx2 = await trackCtxPromise2
+      expect(ctx2).toBeInstanceOf(Context)
     })
 
     test('If a user sends multiple events, all of those event gets flushed', async () => {
@@ -123,10 +103,11 @@ describe('Pre-initialization', () => {
         status: 403,
         statusText: 'Forbidden',
         json: undefined,
+        text: () => Promise.resolve('text'),
       }
       mockFetchSettingsErrorResponse(err)
       const consoleSpy = jest
-        .spyOn(console, 'warn')
+        .spyOn(console, 'error')
         .mockImplementationOnce(() => {})
       AnalyticsBrowser.load({ writeKey: 'abc' })
       await sleep(500)

packages/browser/src/browser/__tests__/csp-detection.test.ts

@@ -4,6 +4,7 @@ import { LegacySettings } from '..'
 import { onCSPError } from '../../lib/csp-detection'
 import { pWhile } from '../../lib/p-while'
 import { snippet } from '../../tester/__fixtures__/segment-snippet'
+import * as Factory from '../../test-helpers/factories'
 
 const cdnResponse: LegacySettings = {
   integrations: {
@@ -24,9 +25,7 @@ const cdnResponse: LegacySettings = {
   },
 }
 
-const fetchSettings = Promise.resolve({
-  json: () => Promise.resolve(cdnResponse),
-})
+const fetchSettings = Factory.createSuccess(cdnResponse)
 
 jest.mock('unfetch', () => {
   return jest.fn()

packages/browser/src/browser/__tests__/standalone-analytics.test.ts

@@ -6,6 +6,7 @@ import { install, AnalyticsSnippet } from '../standalone-analytics'
 import unfetch from 'unfetch'
 import { PersistedPriorityQueue } from '../../lib/priority-queue/persisted'
 import { sleep } from '../../test-helpers/sleep'
+import * as Factory from '../../test-helpers/factories'
 
 const track = jest.fn()
 const identify = jest.fn()
@@ -32,12 +33,7 @@ jest.mock('@/core/analytics', () => ({
   }),
 }))
 
-const fetchSettings = Promise.resolve({
-  json: () =>
-    Promise.resolve({
-      integrations: {},
-    }),
-})
+const fetchSettings = Factory.createSuccess({ integrations: {} })
 
 jest.mock('unfetch', () => {
   return jest.fn()
@@ -81,6 +77,7 @@ describe('standalone bundle', () => {
     const documentSpy = jest.spyOn(global, 'document', 'get')
 
     jest.spyOn(console, 'warn').mockImplementationOnce(() => {})
+    jest.spyOn(console, 'error').mockImplementationOnce(() => {})
 
     windowSpy.mockImplementation(() => {
       return jsd.window as unknown as Window & typeof globalThis

packages/browser/src/browser/__tests__/standalone-errors.test.ts

@@ -4,6 +4,7 @@ import { snippet } from '../../tester/__fixtures__/segment-snippet'
 import { pWhile } from '../../lib/p-while'
 import unfetch from 'unfetch'
 import { RemoteMetrics } from '../../core/stats/remote-metrics'
+import * as Factory from '../../test-helpers/factories'
 
 const cdnResponse: LegacySettings = {
   integrations: {
@@ -24,9 +25,7 @@ const cdnResponse: LegacySettings = {
   },
 }
 
-const fetchSettings = Promise.resolve({
-  json: () => Promise.resolve(cdnResponse),
-})
+const fetchSettings = Factory.createSuccess(cdnResponse)
 
 jest.mock('unfetch', () => {
   return jest.fn()

packages/browser/src/browser/__tests__/standalone.test.ts

@@ -3,6 +3,7 @@ import unfetch from 'unfetch'
 import { LegacySettings } from '..'
 import { pWhile } from '../../lib/p-while'
 import { snippet } from '../../tester/__fixtures__/segment-snippet'
+import * as Factory from '../../test-helpers/factories'
 
 const cdnResponse: LegacySettings = {
   integrations: {
@@ -23,9 +24,7 @@ const cdnResponse: LegacySettings = {
   },
 }
 
-const fetchSettings = Promise.resolve({
-  json: () => Promise.resolve(cdnResponse),
-})
+const fetchSettings = Factory.createSuccess(cdnResponse)
 
 jest.mock('unfetch', () => {
   return jest.fn()

packages/browser/src/browser/index.ts

@@ -83,9 +83,16 @@ export function loadLegacySettings(
   const baseUrl = cdnURL ?? getCDN()
 
   return fetch(`${baseUrl}/v1/projects/${writeKey}/settings`)
-    .then((res) => res.json())
+    .then((res) => {
+      if (!res.ok) {
+        return res.text().then((errorResponseMessage) => {
+          throw { ...res, errorResponseMessage }
+        })
+      }
+      return res.json()
+    })
     .catch((err) => {
-      console.warn('Failed to load settings', err)
+      console.error(err.errorResponseMessage)
       throw err
     })
 }

packages/browser/src/plugins/ajs-destination/__tests__/index.test.ts

@@ -9,6 +9,7 @@ import { Plan } from '../../../core/events'
 import { tsubMiddleware } from '../../routing-middleware'
 import { AMPLITUDE_WRITEKEY } from '../../../test-helpers/test-writekeys'
 import { PersistedPriorityQueue } from '../../../lib/priority-queue/persisted'
+import * as Factory from '../../../test-helpers/factories'
 
 const cdnResponse: LegacySettings = {
   integrations: {
@@ -66,9 +67,7 @@ const cdnResponse: LegacySettings = {
   },
 }
 
-const fetchSettings = Promise.resolve({
-  json: () => Promise.resolve(cdnResponse),
-})
+const fetchSettings = Factory.createSuccess(cdnResponse)
 
 jest.mock('unfetch', () => {
   return jest.fn()