@amplytools/react-native-amply-sdk 0.15.0 → 0.15.1

package/src/__tests__/handleUrl.test.ts

@@ -1,155 +0,0 @@
-/**
- * `handleUrl` hands an incoming link to the native SDK and reports whether the SDK recognised it.
- * It is synchronous, and it never throws: an older native build without the method answers false.
- */
-type SdkModule = typeof import('../index');
-
-function loadWith(native: Record<string, unknown>): SdkModule {
-  // Absent isInitialized means "SDK up" for the plain pass-through tests below.
-  const withState = {isInitialized: () => true, ...native};
-  let Amply!: SdkModule;
-  jest.isolateModules(() => {
-    // eslint-disable-next-line @typescript-eslint/no-var-requires
-    require('../nativeModule').__setNativeModule(withState as never);
-    // eslint-disable-next-line @typescript-eslint/no-var-requires
-    Amply = require('../index');
-  });
-  return Amply;
-}
-
-const DEBUG_URL = 'myapp://amply-debug/campaign?t=abc';
-
-describe('handleUrl', () => {
-  it('passes the DEBUG_URL to the native module and returns its answer', () => {
-    const handleUrl = jest.fn(() => true);
-    const Amply = loadWith({handleUrl});
-
-    expect(Amply.handleUrl(DEBUG_URL)).toBe(true);
-    expect(handleUrl).toHaveBeenCalledWith(DEBUG_URL);
-  });
-
-  it('returns false when the native module does not recognise the DEBUG_URL', () => {
-    const Amply = loadWith({handleUrl: jest.fn(() => false)});
-
-    expect(Amply.handleUrl('myapp://settings')).toBe(false);
-  });
-
-  it('returns false without throwing when the native module has no handleUrl', () => {
-    const Amply = loadWith({});
-
-    expect(Amply.handleUrl(DEBUG_URL)).toBe(false);
-  });
-
-  it('returns false without throwing when the native call throws', () => {
-    const Amply = loadWith({
-      handleUrl: jest.fn(() => {
-        throw new Error('boom');
-      }),
-    });
-
-    expect(Amply.handleUrl(DEBUG_URL)).toBe(false);
-  });
-
-  it('returns false for a non-string DEBUG_URL', () => {
-    const handleUrl = jest.fn(() => true);
-    const Amply = loadWith({handleUrl});
-
-    expect(Amply.handleUrl(null as never)).toBe(false);
-    expect(handleUrl).not.toHaveBeenCalled();
-  });
-
-  it('is on the default export', () => {
-    const Amply = loadWith({handleUrl: jest.fn(() => true)});
-
-    expect(Amply.default.handleUrl(DEBUG_URL)).toBe(true);
-  });
-});
-
-describe('handleUrl before initialize', () => {
-  function loadUninitialised(initialize?: () => Promise<void>) {
-    let up = false;
-    const handleUrl = jest.fn(() => true);
-    const native = {
-      handleUrl,
-      isInitialized: jest.fn(() => up),
-      initialize: jest.fn(
-        initialize ??
-          (async () => {
-            up = true;
-          }),
-      ),
-      addListener: jest.fn(),
-      removeListeners: jest.fn(),
-    };
-    return {Amply: loadWith(native), handleUrl};
-  }
-  const KEYS = {appId: 'id', apiKeyPublic: 'public', apiKeySecret: 'secret'};
-
-  it('parks a debug link, returns true and does not call native', () => {
-    const {Amply, handleUrl} = loadUninitialised();
-
-    expect(Amply.handleUrl(DEBUG_URL)).toBe(true);
-    expect(handleUrl).not.toHaveBeenCalled();
-  });
-
-  it('replays the parked link to native once after initialize', async () => {
-    const {Amply, handleUrl} = loadUninitialised();
-    Amply.handleUrl(DEBUG_URL);
-
-    await Amply.initialize(KEYS);
-
-    expect(handleUrl).toHaveBeenCalledTimes(1);
-    expect(handleUrl).toHaveBeenCalledWith(DEBUG_URL);
-  });
-
-  it('replays only the newer of two parked links', async () => {
-    const {Amply, handleUrl} = loadUninitialised();
-    Amply.handleUrl('myapp://amply-debug/campaign?t=old');
-    Amply.handleUrl('myapp://amply-debug/campaign?t=new');
-
-    await Amply.initialize(KEYS);
-
-    expect(handleUrl).toHaveBeenCalledTimes(1);
-    expect(handleUrl).toHaveBeenCalledWith('myapp://amply-debug/campaign?t=new');
-  });
-
-  it('does not park a non-debug link', async () => {
-    const {Amply, handleUrl} = loadUninitialised();
-
-    expect(Amply.handleUrl('myapp://settings')).toBe(false);
-    await Amply.initialize(KEYS);
-
-    expect(handleUrl).not.toHaveBeenCalled();
-  });
-
-  it('drops the parked link when initialize fails', async () => {
-    const {Amply, handleUrl} = loadUninitialised(async () => {
-      throw new Error('init failed');
-    });
-    Amply.handleUrl(DEBUG_URL);
-
-    await expect(Amply.initialize(KEYS)).rejects.toThrow('init failed');
-
-    expect(handleUrl).not.toHaveBeenCalled();
-  });
-
-  it.each([
-    '',
-    'not a url',
-    'myapp://amply-debug/campaign',
-    'myapp://amply-debug/campaign?t=',
-    'myapp://amply-debug/other?t=abc',
-    'myapp://other/campaign?t=abc',
-    'amply-debug/campaign?t=abc',
-    '://amply-debug/campaign?t=abc',
-  ])('returns false without throwing for %p', async url => {
-    const {Amply, handleUrl} = loadUninitialised();
-
-    expect(Amply.handleUrl(url)).toBe(false);
-    await Amply.initialize(KEYS);
-
-    expect(handleUrl).not.toHaveBeenCalled();
-  });
-});
-
-export {};

package/src/__tests__/index.test.ts

@@ -1,258 +0,0 @@
-import * as Amply from '../index';
-import {__setNativeModule} from '../nativeModule';
-
-const mockNativeModule = {
-  initialize: jest.fn(),
-  isInitialized: jest.fn(() => true),
-  track: jest.fn(),
-  getRecentEvents: jest.fn(),
-  getDataSetSnapshot: jest.fn(),
-  registerDeepLinkListener: jest.fn(),
-  addListener: jest.fn(),
-  removeListeners: jest.fn(),
-  onSystemEvent: jest.fn(),
-  onDeepLink: jest.fn(),
-  setCustomProperties: jest.fn(),
-  getCustomProperty: jest.fn(),
-  removeCustomProperty: jest.fn(),
-  clearCustomProperties: jest.fn(),
-};
-
-describe('Amply JS API', () => {
-  afterEach(() => {
-    jest.clearAllMocks();
-    __setNativeModule(null);
-  });
-
-  beforeEach(() => {
-    __setNativeModule(mockNativeModule as never);
-  });
-
-  it('initializes the native module', async () => {
-    await Amply.initialize({ appId: 'id', apiKeyPublic: 'public', apiKeySecret: 'secret' });
-
-    expect(mockNativeModule.initialize).toHaveBeenCalledWith(
-      expect.objectContaining({ appId: 'id', apiKeyPublic: 'public' })
-    );
-  });
-
-  // A missing key used to reach the native SDK, whose config builder threw an exception that
-  // crossed into Objective-C uncaught and aborted the host app — no JS try/catch could stop it.
-  // Bad keys must cost the app its Amply, never its process.
-  describe.each([
-    ['apiKeySecret', { appId: 'id', apiKeyPublic: 'public' }],
-    ['apiKeySecret', { appId: 'id', apiKeyPublic: 'public', apiKeySecret: '' }],
-    ['apiKeySecret', { appId: 'id', apiKeyPublic: 'public', apiKeySecret: '   ' }],
-    ['apiKeySecret', { appId: 'id', apiKeyPublic: 'public', apiKeySecret: null }],
-    ['apiKeyPublic', { appId: 'id', apiKeyPublic: '', apiKeySecret: 'secret' }],
-    ['appId', { appId: '', apiKeyPublic: 'public', apiKeySecret: 'secret' }],
-  ])('when %s is missing (%j)', (field, config) => {
-    it('rejects with AMP_INVALID_CONFIG, names the field and never calls the native module', async () => {
-      const error = jest.spyOn(console, 'error').mockImplementation(() => {});
-
-      await expect(Amply.initialize(config as never)).rejects.toMatchObject({
-        code: 'AMP_INVALID_CONFIG',
-        message: expect.stringContaining(field),
-      });
-      expect(mockNativeModule.initialize).not.toHaveBeenCalled();
-      expect(error).toHaveBeenCalledWith(expect.stringContaining(field));
-
-      error.mockRestore();
-    });
-  });
-
-  it('passes the base-URL overrides through to the native layer', async () => {
-    await Amply.initialize({
-      appId: 'id',
-      apiKeyPublic: 'public',
-      apiKeySecret: 'secret',
-      configBaseUrl: 'https://config-staging.example.com',
-      backendBaseUrl: 'https://api-staging.example.com',
-    });
-
-    expect(mockNativeModule.initialize).toHaveBeenCalledWith(
-      expect.objectContaining({
-        configBaseUrl: 'https://config-staging.example.com',
-        backendBaseUrl: 'https://api-staging.example.com',
-      })
-    );
-  });
-
-  // `endpoint` is the older single-URL spelling. It stays accepted so existing
-  // callers keep working, but it only ever meant the backend host — the campaign
-  // manifest is served from a different one.
-  it('still accepts the legacy endpoint spelling', async () => {
-    await Amply.initialize({
-      appId: 'id',
-      apiKeyPublic: 'public',
-      apiKeySecret: 'secret',
-      endpoint: 'https://api-staging.example.com',
-    });
-
-    expect(mockNativeModule.initialize).toHaveBeenCalledWith(
-      expect.objectContaining({ endpoint: 'https://api-staging.example.com' })
-    );
-  });
-
-  it('tracks events through the native layer', async () => {
-    await Amply.track({ name: 'Test Event', properties: { foo: 'bar' } });
-
-    expect(mockNativeModule.track).toHaveBeenCalledWith({
-      name: 'Test Event',
-      properties: { foo: 'bar' },
-    });
-  });
-
-  it('reads recent events', async () => {
-    const events = [{ name: 'e1' }];
-    mockNativeModule.getRecentEvents.mockResolvedValueOnce(events);
-
-    const result = await Amply.getRecentEvents(3);
-
-    expect(mockNativeModule.getRecentEvents).toHaveBeenCalledWith(3);
-    expect(result).toBe(events);
-  });
-
-  it('requests dataset snapshots', async () => {
-    const snapshot = { foo: 'bar' };
-    mockNativeModule.getDataSetSnapshot.mockResolvedValueOnce(snapshot);
-
-    const type = { kind: '@device' as const };
-    const result = await Amply.getDataSetSnapshot(type);
-
-    expect(mockNativeModule.getDataSetSnapshot).toHaveBeenCalledWith(type);
-    expect(result).toBe(snapshot);
-  });
-
-  it.each([
-    ['singleton @device', { kind: '@device' as const }],
-    ['singleton @user', { kind: '@user' as const }],
-    ['singleton @custom', { kind: '@custom' as const }],
-    ['singleton @session', { kind: '@session' as const }],
-    [
-      '@triggeredEvent with global strategy',
-      {
-        kind: '@triggeredEvent' as const,
-        data: {
-          countStrategy: 'global' as const,
-          params: [{ name: 'level', value: 5 }],
-          eventName: 'LevelComplete',
-        },
-      },
-    ],
-    [
-      '@triggeredEvent with session strategy and no eventName',
-      {
-        kind: '@triggeredEvent' as const,
-        data: {
-          countStrategy: 'session' as const,
-          params: [],
-        },
-      },
-    ],
-    [
-      '@events with mixed event types',
-      {
-        kind: '@events' as const,
-        data: [
-          { name: 'Purchase', type: 'custom' as const, params: [{ name: 'sku', value: 'ABC' }] },
-          { name: 'SessionStarted', type: 'system' as const, params: [] },
-        ],
-      },
-    ],
-  ])('passes %s to the native bridge unchanged', async (_label, type) => {
-    mockNativeModule.getDataSetSnapshot.mockResolvedValueOnce({});
-
-    await Amply.getDataSetSnapshot(type);
-
-    expect(mockNativeModule.getDataSetSnapshot).toHaveBeenCalledWith(type);
-  });
-
-  it('subscribes to system events through the TurboModule emitter', async () => {
-    const remove = jest.fn();
-    mockNativeModule.onSystemEvent.mockReturnValueOnce({ remove });
-
-    const listener = jest.fn();
-    const unsubscribe = await Amply.addSystemEventListener(listener);
-
-    expect(mockNativeModule.onSystemEvent).toHaveBeenCalledTimes(1);
-
-    // The native listener is a wrapper that filters DebugLog events
-    const nativeListener = mockNativeModule.onSystemEvent.mock.calls[0][0];
-
-    // DebugLog events should be filtered out
-    nativeListener({ name: 'DebugLog', type: 'log', timestamp: 1, properties: {} });
-    expect(listener).not.toHaveBeenCalled();
-
-    // Other events should pass through
-    const systemEvent = { name: 'SessionStart', type: 'system', timestamp: 2, properties: {} };
-    nativeListener(systemEvent);
-    expect(listener).toHaveBeenCalledWith(systemEvent);
-
-    unsubscribe();
-    expect(remove).toHaveBeenCalledTimes(1);
-  });
-
-  it('sets a single custom property via setCustomProperties', () => {
-    Amply.setCustomProperty('plan', 'premium');
-
-    expect(mockNativeModule.setCustomProperties).toHaveBeenCalledWith({
-      plan: 'premium',
-    });
-  });
-
-  it('sets multiple custom properties at once', () => {
-    Amply.setCustomProperties({ plan: 'premium', age: 25, active: true });
-
-    expect(mockNativeModule.setCustomProperties).toHaveBeenCalledWith({
-      plan: 'premium',
-      age: 25,
-      active: true,
-    });
-  });
-
-  it('gets a custom property value', async () => {
-    mockNativeModule.getCustomProperty.mockResolvedValueOnce({ value: 'premium' });
-
-    const result = await Amply.getCustomProperty('plan');
-
-    expect(mockNativeModule.getCustomProperty).toHaveBeenCalledWith('plan');
-    expect(result).toBe('premium');
-  });
-
-  it('returns null for a missing custom property', async () => {
-    mockNativeModule.getCustomProperty.mockResolvedValueOnce({});
-
-    const result = await Amply.getCustomProperty('nonexistent');
-
-    expect(result).toBeNull();
-  });
-
-  it('removes a custom property', () => {
-    Amply.removeCustomProperty('plan');
-
-    expect(mockNativeModule.removeCustomProperty).toHaveBeenCalledWith('plan');
-  });
-
-  it('clears all custom properties', () => {
-    Amply.clearCustomProperties();
-
-    expect(mockNativeModule.clearCustomProperties).toHaveBeenCalled();
-  });
-
-  it('registers deep links once and subscribes via the TurboModule emitter', async () => {
-    const remove = jest.fn();
-    mockNativeModule.onDeepLink.mockReturnValue({ remove });
-
-    const listener = jest.fn();
-    const unsubscribe = await Amply.addDeepLinkListener(listener);
-    expect(mockNativeModule.registerDeepLinkListener).toHaveBeenCalledTimes(1);
-    expect(mockNativeModule.onDeepLink).toHaveBeenCalledWith(listener);
-
-    await Amply.addDeepLinkListener(() => {});
-    expect(mockNativeModule.registerDeepLinkListener).toHaveBeenCalledTimes(1);
-
-    unsubscribe();
-    expect(remove).toHaveBeenCalledTimes(1);
-  });
-});

package/src/__tests__/lastCampaignTouch.test.ts

@@ -1,151 +0,0 @@
-import * as Amply from '../index';
-import {__setNativeModule} from '../nativeModule';
-
-const mockNativeModule = {
-  isInitialized: jest.fn(() => true),
-  getLastCampaignTouch: jest.fn(),
-};
-
-const ONE_HOUR_MS = 60 * 60 * 1000;
-
-describe('getLastCampaignTouch', () => {
-  let warn: jest.SpyInstance;
-
-  beforeEach(() => {
-    __setNativeModule(mockNativeModule as never);
-    warn = jest.spyOn(console, 'warn').mockImplementation(() => {});
-  });
-
-  afterEach(() => {
-    warn.mockRestore();
-    jest.clearAllMocks();
-    __setNativeModule(null);
-  });
-
-  it('passes withinMs through to the native module and returns the touch', async () => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce({
-      campaignId: 'cmp_1',
-      campaignName: 'Spring offer',
-      kind: 'shown',
-      atMs: 1_760_000_000_000,
-    });
-
-    const touch = await Amply.getLastCampaignTouch(ONE_HOUR_MS);
-
-    expect(mockNativeModule.getLastCampaignTouch).toHaveBeenCalledWith(ONE_HOUR_MS);
-    expect(touch).toEqual({
-      campaignId: 'cmp_1',
-      campaignName: 'Spring offer',
-      kind: 'shown',
-      atMs: 1_760_000_000_000,
-    });
-  });
-
-  it('is exposed on the default export too', async () => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce(null);
-
-    await expect(Amply.default.getLastCampaignTouch(0)).resolves.toBeNull();
-  });
-
-  it('resolves null when the native side has no touch', async () => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce(null);
-
-    await expect(Amply.getLastCampaignTouch(ONE_HOUR_MS)).resolves.toBeNull();
-  });
-
-  it('resolves null when the native side resolves undefined', async () => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce(undefined);
-
-    await expect(Amply.getLastCampaignTouch(ONE_HOUR_MS)).resolves.toBeNull();
-  });
-
-  it.each([
-    ['shown', 'shown'],
-    ['engaged', 'engaged'],
-    ['SHOWN', 'shown'],
-    ['ENGAGED', 'engaged'],
-  ])('maps native kind %s to %s', async (nativeKind, expected) => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce({
-      campaignId: 'cmp_1',
-      campaignName: 'Spring offer',
-      kind: nativeKind,
-      atMs: 1,
-    });
-
-    const touch = await Amply.getLastCampaignTouch(ONE_HOUR_MS);
-
-    expect(touch?.kind).toBe(expected);
-  });
-
-  it('resolves null and warns for a kind this bridge does not know', async () => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce({
-      campaignId: 'cmp_1',
-      campaignName: 'Spring offer',
-      kind: 'clicked',
-      atMs: 1,
-    });
-
-    await expect(Amply.getLastCampaignTouch(ONE_HOUR_MS)).resolves.toBeNull();
-    expect(warn).toHaveBeenCalledWith(expect.stringContaining('getLastCampaignTouch'), expect.anything());
-  });
-
-  it.each([
-    ['a missing campaignId', {campaignName: 'x', kind: 'shown', atMs: 1}],
-    ['a non-numeric atMs', {campaignId: 'c', campaignName: 'x', kind: 'shown', atMs: '1'}],
-  ])('resolves null for a payload with %s', async (_label, payload) => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce(payload);
-
-    await expect(Amply.getLastCampaignTouch(ONE_HOUR_MS)).resolves.toBeNull();
-  });
-
-  it('keeps an empty campaign name rather than dropping the touch', async () => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce({
-      campaignId: 'cmp_1',
-      campaignName: '',
-      kind: 'engaged',
-      atMs: 5,
-    });
-
-    await expect(Amply.getLastCampaignTouch(ONE_HOUR_MS)).resolves.toEqual({
-      campaignId: 'cmp_1',
-      campaignName: '',
-      kind: 'engaged',
-      atMs: 5,
-    });
-  });
-
-  it('resolves null instead of rejecting when the native call fails', async () => {
-    mockNativeModule.getLastCampaignTouch.mockRejectedValueOnce(new Error('bridge gone'));
-
-    await expect(Amply.getLastCampaignTouch(ONE_HOUR_MS)).resolves.toBeNull();
-    expect(warn).toHaveBeenCalled();
-  });
-
-  it('accepts zero', async () => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce(null);
-
-    await Amply.getLastCampaignTouch(0);
-
-    expect(mockNativeModule.getLastCampaignTouch).toHaveBeenCalledWith(0);
-  });
-
-  it('leaves a large window to the native clamp', async () => {
-    mockNativeModule.getLastCampaignTouch.mockResolvedValueOnce(null);
-    const ninetyDaysMs = 90 * 24 * ONE_HOUR_MS;
-
-    await Amply.getLastCampaignTouch(ninetyDaysMs);
-
-    expect(mockNativeModule.getLastCampaignTouch).toHaveBeenCalledWith(ninetyDaysMs);
-  });
-
-  it.each([
-    ['negative', -1],
-    ['NaN', Number.NaN],
-    ['Infinity', Number.POSITIVE_INFINITY],
-    ['a string', '3600000'],
-    ['undefined', undefined],
-  ])('rejects with a RangeError for %s and never calls native', async (_label, value) => {
-    await expect(Amply.getLastCampaignTouch(value as never)).rejects.toBeInstanceOf(RangeError);
-    expect(mockNativeModule.getLastCampaignTouch).not.toHaveBeenCalled();
-  });
-});

package/src/__tests__/systemEventUtils.test.ts

@@ -1,97 +0,0 @@
-import {formatSystemEventLabel} from '../systemEventUtils';
-import type {EventRecord} from '../nativeSpecs/NativeAmplyModule';
-
-function makeEvent(
-  name: string,
-  properties: Record<string, unknown> = {},
-): EventRecord {
-  return {name, type: 'system', timestamp: Date.now(), properties};
-}
-
-describe('formatSystemEventLabel', () => {
-  describe('SessionStarted', () => {
-    it('returns "Session started (cold)" for cold start', () => {
-      const event = makeEvent('SessionStarted', {type: 'cold'});
-      expect(formatSystemEventLabel(event)).toBe('Session started (cold)');
-    });
-
-    it('returns "Session started (warm)" for warm start', () => {
-      const event = makeEvent('SessionStarted', {type: 'warm'});
-      expect(formatSystemEventLabel(event)).toBe('Session started (warm)');
-    });
-
-    it('returns "Session started" when type is absent', () => {
-      const event = makeEvent('SessionStarted');
-      expect(formatSystemEventLabel(event)).toBe('Session started');
-    });
-
-    it('handles unexpected type values gracefully', () => {
-      const event = makeEvent('SessionStarted', {type: 'unknown'});
-      expect(formatSystemEventLabel(event)).toBe('Session started (unknown)');
-    });
-  });
-
-  describe('other events', () => {
-    it('formats CampaignShown with campaignId and source', () => {
-      const event = makeEvent('CampaignShown', {
-        campaignId: 'abc',
-        source: 'click',
-      });
-      expect(formatSystemEventLabel(event)).toBe(
-        'Campaign shown (abc) via click',
-      );
-    });
-
-    it('names the campaign in CampaignShown when the SDK sends campaignName', () => {
-      const event = makeEvent('CampaignShown', {
-        campaignId: 'abc',
-        campaignName: 'Spring offer',
-        source: 'click',
-      });
-      expect(formatSystemEventLabel(event)).toBe(
-        'Campaign shown "Spring offer" (abc) via click',
-      );
-    });
-
-    it('ignores an empty campaignName in CampaignShown', () => {
-      const event = makeEvent('CampaignShown', {
-        campaignId: 'abc',
-        campaignName: '',
-        source: 'click',
-      });
-      expect(formatSystemEventLabel(event)).toBe(
-        'Campaign shown (abc) via click',
-      );
-    });
-
-    it('formats EventTriggered with source event', () => {
-      const event = makeEvent('EventTriggered', {sourceEvent: 'purchase'});
-      expect(formatSystemEventLabel(event)).toBe('Event triggered (purchase)');
-    });
-
-    it('formats ConfigFetchFinished with campaign count', () => {
-      const event = makeEvent('ConfigFetchFinished', {
-        success: true,
-        campaignCount: 3,
-      });
-      expect(formatSystemEventLabel(event)).toBe(
-        'Config fetch finished (3 campaigns)',
-      );
-    });
-
-    it('formats ConfigFetchFinished failure', () => {
-      const event = makeEvent('ConfigFetchFinished', {success: false});
-      expect(formatSystemEventLabel(event)).toBe('Config fetch failed');
-    });
-
-    it('returns default label for SessionFinished', () => {
-      const event = makeEvent('SessionFinished');
-      expect(formatSystemEventLabel(event)).toBe('Session finished');
-    });
-
-    it('returns fallback for unknown events', () => {
-      const event = makeEvent('UnknownEvent');
-      expect(formatSystemEventLabel(event)).toBe('System event UnknownEvent');
-    });
-  });
-});