@object-ui/data-objectstack 17.5.0 → 17.7.0

package/src/integration.ts

@@ -1,200 +0,0 @@
-/**
- * ObjectUI
- * Copyright (c) 2024-present ObjectStack Inc.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-/**
- * Integration module for @objectstack/spec v3.0.0
- * Provides third-party service connectors for Slack, email, webhooks.
- */
-
-export type IntegrationProvider = 'slack' | 'email' | 'webhook' | 'teams' | 'discord';
-
-export interface IntegrationConfig {
-  /** Integration provider type */
-  provider: IntegrationProvider;
-  /** Whether this integration is enabled */
-  enabled: boolean;
-  /** Provider-specific configuration */
-  config: Record<string, unknown>;
-  /** Event triggers */
-  triggers?: IntegrationTrigger[];
-}
-
-export interface IntegrationTrigger {
-  /** Event name (e.g. 'record.created', 'record.updated') */
-  event: string;
-  /** Filter condition */
-  filter?: string;
-  /** Template for the message/payload */
-  template?: string;
-}
-
-export interface SlackIntegrationConfig extends IntegrationConfig {
-  provider: 'slack';
-  config: {
-    webhookUrl: string;
-    channel?: string;
-    username?: string;
-    iconEmoji?: string;
-  };
-}
-
-export interface EmailIntegrationConfig extends IntegrationConfig {
-  provider: 'email';
-  config: {
-    smtpHost: string;
-    smtpPort: number;
-    secure: boolean;
-    from: string;
-    to: string[];
-    subject?: string;
-  };
-}
-
-export interface WebhookIntegrationConfig extends IntegrationConfig {
-  provider: 'webhook';
-  config: {
-    url: string;
-    method: 'GET' | 'POST' | 'PUT' | 'PATCH';
-    headers?: Record<string, string>;
-    retryCount?: number;
-    retryDelay?: number;
-  };
-}
-
-/**
- * Integration service manager.
- * Manages third-party service connections and event dispatching.
- */
-export class IntegrationManager {
-  private integrations: Map<string, IntegrationConfig> = new Map();
-
-  /**
-   * Register a new integration.
-   */
-  register(id: string, config: IntegrationConfig): void {
-    this.integrations.set(id, config);
-  }
-
-  /**
-   * Remove an integration.
-   */
-  unregister(id: string): void {
-    this.integrations.delete(id);
-  }
-
-  /**
-   * Get all registered integrations.
-   */
-  getAll(): Map<string, IntegrationConfig> {
-    return new Map(this.integrations);
-  }
-
-  /**
-   * Get integrations that match a specific event.
-   */
-  getForEvent(event: string): IntegrationConfig[] {
-    return Array.from(this.integrations.values()).filter(
-      (integration) =>
-        integration.enabled &&
-        integration.triggers?.some((t) => t.event === event)
-    );
-  }
-
-  /**
-   * Dispatch an event to all matching integrations.
-   * Returns results for each integration.
-   */
-  async dispatch(event: string, payload: Record<string, unknown>): Promise<Array<{ id: string; success: boolean; error?: string }>> {
-    const matching = this.getForEvent(event);
-    const results: Array<{ id: string; success: boolean; error?: string }> = [];
-
-    for (const [id, integration] of this.integrations) {
-      if (!matching.includes(integration)) continue;
-      
-      try {
-        await this.send(integration, payload);
-        results.push({ id, success: true });
-      } catch (err) {
-        results.push({ id, success: false, error: (err as Error).message });
-      }
-    }
-
-    return results;
-  }
-
-  /**
-   * Send payload to a specific integration.
-   */
-  private async send(integration: IntegrationConfig, payload: Record<string, unknown>): Promise<void> {
-    switch (integration.provider) {
-      case 'webhook': {
-        const cfg = integration.config as WebhookIntegrationConfig['config'];
-        const url = cfg.url;
-        // Validate URL - only allow http and https protocols
-        try {
-          const parsed = new URL(url);
-          if (parsed.protocol !== 'http:' && parsed.protocol !== 'https:') {
-            throw new Error(`Invalid URL protocol: ${parsed.protocol}`);
-          }
-        } catch (e) {
-          if (e instanceof TypeError) {
-            // A malformed URL surfaces as its own message; we can't attach `e`
-            // as the `Error` `cause` because this package targets ES2020, whose
-            // lib types the 1-arg `Error` constructor only. Scoped disable.
-            // eslint-disable-next-line preserve-caught-error
-            throw new Error(`Invalid webhook URL: ${url}`);
-          }
-          throw e;
-        }
-        await fetch(url, {
-          method: cfg.method,
-          headers: {
-            'Content-Type': 'application/json',
-            ...cfg.headers,
-          },
-          body: JSON.stringify(payload),
-        });
-        break;
-      }
-      case 'slack': {
-        const cfg = integration.config as SlackIntegrationConfig['config'];
-        const url = cfg.webhookUrl;
-        // Validate URL - only allow https protocol for Slack webhooks
-        try {
-          const parsed = new URL(url);
-          if (parsed.protocol !== 'https:') {
-            throw new Error(`Invalid Slack webhook URL protocol: ${parsed.protocol}`);
-          }
-        } catch (e) {
-          if (e instanceof TypeError) {
-            // A malformed URL surfaces as its own message; we can't attach `e`
-            // as the `Error` `cause` because this package targets ES2020, whose
-            // lib types the 1-arg `Error` constructor only. Scoped disable.
-            // eslint-disable-next-line preserve-caught-error
-            throw new Error(`Invalid Slack webhook URL: ${url}`);
-          }
-          throw e;
-        }
-        await fetch(url, {
-          method: 'POST',
-          headers: { 'Content-Type': 'application/json' },
-          body: JSON.stringify({
-            channel: cfg.channel,
-            username: cfg.username,
-            icon_emoji: cfg.iconEmoji,
-            text: JSON.stringify(payload),
-          }),
-        });
-        break;
-      }
-      // Email and other providers would require server-side implementation
-      default:
-        break;
-    }
-  }
-}

package/src/listViewOverrides.test.ts

@@ -1,241 +0,0 @@
-/**
- * ObjectUI
- * Copyright (c) 2024-present ObjectStack Inc.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-/**
- * objectui#3774 — the view-override read and write must share one key space.
- *
- * `updateViewConfig` persists a personalization (density, column widths, sort,
- * hidden columns, inlineEdit …) as `PUT /api/v1/meta/view/{viewId}`, i.e. a
- * `sys_metadata` row with `type='view'`. `listViewOverrides` used to enumerate
- * `GET /api/v1/meta/{objectName}` — the OBJECT name in the metadata TYPE slot.
- * The two key spaces are disjoint, so the batch map came back empty for every
- * object forever and every saved preference read back as "setting didn't save".
- *
- * The measurement that picked the fix: `GET /meta/:type` accepts `?package=`
- * and `?preview=draft` and NOTHING else (framework
- * `packages/rest/src/rest-server.ts`, and `client.meta.getItems(type, {
- * packageId })`). There is no `?object=` to push the filter into, so the
- * correct route is the one `listViews()` already uses over the same rows:
- * enumerate `type='view'` once, narrow to the object client-side.
- *
- * The suite is written round-trip on purpose — a stub `sys_metadata` that the
- * adapter's own write path fills and its own read path drains. Asserting the
- * read against a hand-written fixture would have stayed green through the
- * entire bug, because the fixture is where the namespace mistake lived.
- */
-
-import { describe, it, expect, vi } from 'vitest';
-import { ObjectStackAdapter } from './index';
-
-/** A stub metadata store keyed the way `sys_metadata` is: `type` + `name`. */
-function makeMetaStore() {
-  const rows = new Map<string, any>();
-  const getItemsTypes: string[] = [];
-
-  const meta = {
-    getItems: vi.fn(async (type: string) => {
-      getItemsTypes.push(type);
-      const items = [...rows.entries()]
-        .filter(([k]) => k.startsWith(`${type}::`))
-        .map(([, v]) => v);
-      return { type, items };
-    }),
-    getItem: vi.fn(async (type: string, name: string) => {
-      const item = rows.get(`${type}::${name}`);
-      if (!item) {
-        const err: any = new Error(`Not found: ${type}/${name}`);
-        err.status = 404;
-        throw err;
-      }
-      return { type, name, item };
-    }),
-    saveItem: vi.fn(async (type: string, name: string, item: any) => {
-      rows.set(`${type}::${name}`, { ...item });
-      return { success: true, item: rows.get(`${type}::${name}`) };
-    }),
-  };
-
-  return { meta, rows, getItemsTypes };
-}
-
-function makeDS(meta: any) {
-  const ds: any = new ObjectStackAdapter({
-    baseUrl: 'http://test.local',
-    fetch: vi.fn(async () =>
-      new Response(JSON.stringify({ success: true, data: { capabilities: {}, routes: {} } }), {
-        status: 200,
-        headers: { 'Content-Type': 'application/json' },
-      })),
-  });
-  ds.connected = true;
-  ds.connectionState = 'connected';
-  ds.client = { meta };
-  return ds;
-}
-
-/**
- * What ObjectView's `persistViewPatch` actually sends on a density toggle +
- * column sort: the view's raw config plus the personalization state. The
- * adapter stamps `object` / `name` on top (and the framework's overlay heals
- * the same identity fields onto rows that lack them — objectstack#2555).
- */
-const PERSONALIZATION = {
-  type: 'grid',
-  data: { provider: 'object', object: 'crm_lead' },
-  columns: ['name', 'status'],
-  density: 'compact',
-  sort: [{ field: 'created_at', order: 'desc' }],
-};
-
-describe('ObjectStackAdapter.listViewOverrides — read/write key space (#3774)', () => {
-  it('round-trips: a config saved through updateViewConfig comes back from listViewOverrides', async () => {
-    const { meta, getItemsTypes } = makeMetaStore();
-    const ds = makeDS(meta);
-
-    await ds.updateViewConfig('crm_lead', 'crm_lead.all', { ...PERSONALIZATION });
-    const map = await ds.listViewOverrides('crm_lead');
-
-    // The bug in one assertion: pre-fix this was `{}`.
-    expect(Object.keys(map)).toEqual(['crm_lead.all']);
-    expect(map['crm_lead.all']).toMatchObject({
-      name: 'crm_lead.all',
-      object: 'crm_lead',
-      density: 'compact',
-      sort: [{ field: 'created_at', order: 'desc' }],
-    });
-
-    // …and the type slot itself, which is the actual defect. The write went to
-    // `saveItem('view', …)`; the read must enumerate the SAME type.
-    expect(meta.saveItem).toHaveBeenCalledWith('view', 'crm_lead.all', expect.any(Object));
-    expect(getItemsTypes).toEqual(['view']);
-    expect(getItemsTypes).not.toContain('crm_lead');
-  });
-
-  it('keys the map by the identity getView reads back by (drop-in for the per-view fetch)', async () => {
-    const { meta } = makeMetaStore();
-    const ds = makeDS(meta);
-
-    await ds.updateViewConfig('crm_lead', 'crm_lead.all', { ...PERSONALIZATION });
-
-    const batched = (await ds.listViewOverrides('crm_lead'))['crm_lead.all'];
-    const perView = await ds.getView('crm_lead', 'crm_lead.all');
-
-    // Same key space, same document — that equality is what lets a caller
-    // substitute one call for N without changing what it renders.
-    expect(batched).toEqual(perView);
-  });
-
-  it('finds an override whose only object association is the top-level `object` stamp', async () => {
-    const { meta } = makeMetaStore();
-    const ds = makeDS(meta);
-
-    // A saved view with no `data.object` (e.g. a kanban personalization whose
-    // provider block was never sent) — `updateViewConfig`'s stamp is the only
-    // association, and it must be enough.
-    await ds.updateViewConfig('crm_lead', 'crm_lead.pipeline', { type: 'kanban', density: 'comfortable' });
-    const map = await ds.listViewOverrides('crm_lead');
-
-    expect(map['crm_lead.pipeline']).toMatchObject({ object: 'crm_lead', density: 'comfortable' });
-  });
-
-  it('narrows the shared `type=view` enumeration to the requested object', async () => {
-    const { meta } = makeMetaStore();
-    const ds = makeDS(meta);
-
-    await ds.updateViewConfig('crm_lead', 'crm_lead.all', { ...PERSONALIZATION });
-    await ds.updateViewConfig('crm_account', 'crm_account.all', {
-      type: 'grid', data: { provider: 'object', object: 'crm_account' }, density: 'compact',
-    });
-
-    expect(Object.keys(await ds.listViewOverrides('crm_lead'))).toEqual(['crm_lead.all']);
-    expect(Object.keys(await ds.listViewOverrides('crm_account'))).toEqual(['crm_account.all']);
-  });
-
-  it('skips rows the enumeration returns without a name (nothing can address them)', async () => {
-    const { meta, rows } = makeMetaStore();
-    const ds = makeDS(meta);
-    rows.set('view::anon', { object: 'crm_lead', density: 'compact' });
-
-    expect(await ds.listViewOverrides('crm_lead')).toEqual({});
-  });
-});
-
-describe('ObjectStackAdapter.listViewOverrides — empty vs unknown (#3774)', () => {
-  it('an EMPTY successful enumeration resolves to {} — that answer is authoritative', async () => {
-    const { meta } = makeMetaStore();
-    const ds = makeDS(meta);
-
-    // No overrides exist for this object. The caller may trust this and skip
-    // the per-view reads; the batch optimization depends on it not throwing.
-    await expect(ds.listViewOverrides('crm_lead')).resolves.toEqual({});
-  });
-
-  it('a FAILING enumeration rejects — it must not masquerade as an empty map', async () => {
-    const { meta } = makeMetaStore();
-    meta.getItems.mockRejectedValueOnce(new Error('503 metadata store unavailable'));
-    const ds = makeDS(meta);
-
-    // Pre-fix this resolved to `{}`, which is indistinguishable from the
-    // authoritative "no overrides" above — that is what made ObjectView's
-    // per-view getView fallback unreachable code.
-    await expect(ds.listViewOverrides('crm_lead')).rejects.toThrow('503 metadata store unavailable');
-  });
-
-  it('a rejection is not cached — the next call re-reads instead of pinning the failure', async () => {
-    const { meta } = makeMetaStore();
-    const ds = makeDS(meta);
-    await ds.updateViewConfig('crm_lead', 'crm_lead.all', { ...PERSONALIZATION });
-    meta.getItems.mockRejectedValueOnce(new Error('transient'));
-
-    await expect(ds.listViewOverrides('crm_lead')).rejects.toThrow('transient');
-    // The store recovered; a cached failure would have starved the view of its
-    // overrides for the whole TTL.
-    expect(Object.keys(await ds.listViewOverrides('crm_lead'))).toEqual(['crm_lead.all']);
-  });
-});
-
-describe('ObjectStackAdapter.listViewOverrides — wire route (#3774)', () => {
-  /** Let the REAL `client.meta` run so the request URL is observed, not assumed. */
-  function makeHttpDS(items: any[]) {
-    const urls: string[] = [];
-    const fetchImpl = vi.fn(async (input: RequestInfo | URL) => {
-      const url = String(input);
-      urls.push(url);
-      if (url.includes('/meta/')) {
-        return new Response(JSON.stringify({ type: 'view', items }), {
-          status: 200, headers: { 'Content-Type': 'application/json' },
-        });
-      }
-      return new Response(JSON.stringify({ success: true, data: { capabilities: {}, routes: {} } }), {
-        status: 200, headers: { 'Content-Type': 'application/json' },
-      });
-    });
-    const ds: any = new ObjectStackAdapter({ baseUrl: 'http://test.local', fetch: fetchImpl });
-    ds.connected = true;
-    ds.connectionState = 'connected';
-    return { ds, urls };
-  }
-
-  it('GETs the view TYPE, never a route built from the object name', async () => {
-    const { ds, urls } = makeHttpDS([
-      { name: 'crm_lead.all', object: 'crm_lead', ...PERSONALIZATION },
-    ]);
-
-    const map = await ds.listViewOverrides('crm_lead');
-    expect(map['crm_lead.all']).toMatchObject({ density: 'compact' });
-
-    const metaUrls = urls.filter((u) => u.includes('/meta/'));
-    expect(metaUrls.length).toBeGreaterThan(0);
-    for (const u of metaUrls) {
-      expect(u).toMatch(/\/meta\/view(\?|$)/);
-      // The pre-fix route. Pinned by shape so it cannot come back as
-      // `/meta/crm_lead` under a different spelling of the same mistake.
-      expect(u).not.toContain('/meta/crm_lead');
-    }
-  });
-});

package/src/listViews.test.ts

@@ -1,175 +0,0 @@
-/**
- * ObjectUI
- * Copyright (c) 2024-present ObjectStack Inc.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-import { describe, it, expect, vi } from 'vitest';
-import { ObjectStackAdapter } from './index';
-
-/**
- * `listViews(object)` feeds the list-view switcher (ObjectView → ViewTabBar),
- * so it must return LIST-family views only. The backend exposes each view as
- * an independent ViewItem carrying a `viewKind` discriminant (ADR-0017); a
- * form-family view (e.g. `crm_activity.default`) must NOT leak in as a tab.
- */
-function makeDS(items: any[]) {
-  const ds: any = new ObjectStackAdapter({
-    baseUrl: 'http://test.local',
-    fetch: vi.fn(async () =>
-      new Response(JSON.stringify({ success: true, data: { capabilities: {}, routes: {} } }), {
-        status: 200,
-        headers: { 'Content-Type': 'application/json' },
-      })),
-  });
-  ds.connected = true;
-  ds.connectionState = 'connected';
-  ds.client = { meta: { getItems: vi.fn(async () => ({ items })) } };
-  return ds;
-}
-
-/**
- * A DS whose HTTP layer answers the `?preview=draft` metadata route with a
- * pre-overlaid list (the shape the framework already returns: draft wins by
- * name, each draft tagged `_draft`). The published `client.meta.getItems` stub
- * is present but MUST NOT be used in preview mode.
- */
-function makeDraftDS(overlaid: any[]) {
-  const fetchImpl = vi.fn(async (input: RequestInfo | URL) => {
-    const url = String(input);
-    if (url.includes('/meta/view')) {
-      return new Response(JSON.stringify({ items: overlaid }), {
-        status: 200,
-        headers: { 'Content-Type': 'application/json' },
-      });
-    }
-    return new Response(JSON.stringify({ success: true, data: {} }), {
-      status: 200,
-      headers: { 'Content-Type': 'application/json' },
-    });
-  });
-  const ds: any = new ObjectStackAdapter({ baseUrl: 'http://test.local', fetch: fetchImpl });
-  ds.connected = true;
-  ds.connectionState = 'connected';
-  ds.client = { meta: { getItems: vi.fn(async () => ({ items: [] })) } };
-  return { ds, fetchImpl };
-}
-
-describe('ObjectStackDataSource.listViews', () => {
-  // The crm_activity view set as the framework expands it.
-  const listAll = {
-    name: 'crm_activity.all', object: 'crm_activity', viewKind: 'list', isDefault: true,
-    label: 'All Activities', config: { type: 'grid', data: { object: 'crm_activity' } },
-  };
-  const listCalendar = {
-    name: 'crm_activity.calendar', object: 'crm_activity', viewKind: 'list',
-    label: 'Activity Calendar', config: { type: 'calendar', data: { object: 'crm_activity' } },
-  };
-  const formDefault = {
-    name: 'crm_activity.default', object: 'crm_activity', viewKind: 'form',
-    label: 'Activity', config: { type: 'simple' },
-  };
-
-  it('excludes form-family ViewItems from the list-view switcher', async () => {
-    const ds = makeDS([listAll, listCalendar, formDefault]);
-    const views = await ds.listViews('crm_activity');
-    const names = views.map((v: any) => v.name).sort();
-    expect(names).toEqual(['crm_activity.all', 'crm_activity.calendar']);
-    // The form view is the original leak — it must be gone.
-    expect(views.find((v: any) => v.name === 'crm_activity.default')).toBeUndefined();
-  });
-
-  it('excludes detail-family views too', async () => {
-    const detail = { name: 'crm_activity.detail', object: 'crm_activity', viewKind: 'detail', config: {} };
-    const ds = makeDS([listAll, detail]);
-    const views = await ds.listViews('crm_activity');
-    expect(views.map((v: any) => v.name)).toEqual(['crm_activity.all']);
-  });
-
-  it('filters by object binding', async () => {
-    const otherList = { name: 'crm_lead.all', object: 'crm_lead', viewKind: 'list', config: {} };
-    const ds = makeDS([listAll, otherList]);
-    const views = await ds.listViews('crm_activity');
-    expect(views.map((v: any) => v.name)).toEqual(['crm_activity.all']);
-  });
-
-  it('keeps legacy bare specs without a viewKind (saved/list views)', async () => {
-    const legacy = { name: 'saved_grid', object: 'crm_activity', type: 'grid' };
-    const wrapped = { list: { name: 'wrapped_grid', object: 'crm_activity', type: 'grid' } };
-    const ds = makeDS([legacy, wrapped]);
-    const views = await ds.listViews('crm_activity');
-    expect(views.map((v: any) => v.name).sort()).toEqual(['saved_grid', 'wrapped_grid']);
-  });
-
-  // ── #2767 draft-preview branch ───────────────────────────────────────────
-  describe('previewDrafts (#2767 P2/P3)', () => {
-    it('reads the draft-overlaid list in a SINGLE preview=draft request', async () => {
-      // Server response overlays a draft edit of `all` (draft wins by name) and
-      // a brand-new draft-only `mine`; `calendar` is untouched/published.
-      const overlaid = [
-        {
-          name: 'crm_activity.all', object: 'crm_activity', viewKind: 'list',
-          label: 'All (edited)', config: { type: 'grid', data: { object: 'crm_activity' } },
-          _draft: true,
-        },
-        {
-          name: 'crm_activity.calendar', object: 'crm_activity', viewKind: 'list',
-          label: 'Activity Calendar', config: { type: 'calendar', data: { object: 'crm_activity' } },
-        },
-        {
-          name: 'crm_activity.mine', object: 'crm_activity', viewKind: 'list',
-          label: 'Mine', config: { type: 'grid', data: { object: 'crm_activity' } },
-          _draft: true,
-        },
-      ];
-      const { ds, fetchImpl } = makeDraftDS(overlaid);
-      const views = await ds.listViews('crm_activity', { previewDrafts: true });
-
-      // Exactly one request, and it carries the preview flag.
-      const metaCalls = fetchImpl.mock.calls.filter(([u]: any[]) => String(u).includes('/meta/view'));
-      expect(metaCalls).toHaveLength(1);
-      expect(String(metaCalls[0][0])).toContain('preview=draft');
-      // Preview mode must NOT also hit the published path (that's the P2 dupe bug).
-      expect(ds.client.meta.getItems).not.toHaveBeenCalled();
-
-      // One tab per name — the draft edit of `all` does not duplicate it.
-      expect(views.map((v: any) => v.name).sort()).toEqual([
-        'crm_activity.all', 'crm_activity.calendar', 'crm_activity.mine',
-      ]);
-    });
-
-    it('flows the _draft provenance flag through for the switcher badge', async () => {
-      const overlaid = [
-        {
-          name: 'crm_activity.mine', object: 'crm_activity', viewKind: 'list',
-          label: 'Mine', config: { type: 'grid', data: { object: 'crm_activity' } }, _draft: true,
-        },
-        {
-          name: 'crm_activity.calendar', object: 'crm_activity', viewKind: 'list',
-          label: 'Cal', config: { type: 'calendar', data: { object: 'crm_activity' } },
-        },
-      ];
-      const { ds } = makeDraftDS(overlaid);
-      const views = await ds.listViews('crm_activity', { previewDrafts: true });
-      expect(views.find((v: any) => v.name === 'crm_activity.mine')?._draft).toBe(true);
-      expect(views.find((v: any) => v.name === 'crm_activity.calendar')?._draft).toBeUndefined();
-    });
-
-    it('non-preview mode uses the published getItems path (no draft fetch)', async () => {
-      const { ds, fetchImpl } = makeDraftDS([]);
-      ds.client.meta.getItems = vi.fn(async () => ({
-        items: [{
-          name: 'crm_activity.all', object: 'crm_activity', viewKind: 'list',
-          config: { type: 'grid', data: { object: 'crm_activity' } },
-        }],
-      }));
-      const views = await ds.listViews('crm_activity');
-      expect(ds.client.meta.getItems).toHaveBeenCalledTimes(1);
-      expect(fetchImpl.mock.calls.filter(([u]: any[]) => String(u).includes('/meta/view'))).toHaveLength(0);
-      expect(views.map((v: any) => v.name)).toEqual(['crm_activity.all']);
-      expect(views[0]._draft).toBeUndefined();
-    });
-  });
-});