@node-i3x/app 0.1.0 → 0.2.3

package/tests/e2e.test.ts

@@ -1,766 +0,0 @@
-// ─────────────────────────────────────────────────────────────
-// End-to-end test: real node-opcua server → full i3X stack
-// ─────────────────────────────────────────────────────────────
-//
-// This test spins up a representative OPC UA server with:
-//   - A "ProductionLine" folder containing two machines
-//   - Each machine has temperature + speed variables
-//   - One machine has a "Reset" method
-//   - Variables update on a timer to exercise subscriptions
-//
-// Then it boots the full i3X stack (connector → core → REST)
-// and validates end-to-end flows against the HTTP API.
-// ─────────────────────────────────────────────────────────────
-
-import {
-  consoleLogger,
-  HistoryService,
-  ModelService,
-  SubscriptionService,
-  ValueService,
-} from '@node-i3x/core';
-import { OpcUaClient, OpcUaDataSourceAdapter } from '@node-i3x/opcua-connector';
-import { createApp } from '@node-i3x/rest-server';
-import {
-  DataType,
-  type Namespace,
-  nodesets,
-  OPCUAServer,
-  StatusCodes,
-  Variant,
-} from 'node-opcua';
-import { afterAll, beforeAll, describe, expect, it } from 'vitest';
-
-// ── Server factory ───────────────────────────────────────────
-
-let serverPort: number;
-let opcuaServer: OPCUAServer;
-
-function getRandomPort(): number {
-  return 48_100 + Math.floor(Math.random() * 900);
-}
-
-async function startTestOpcUaServer(): Promise<OPCUAServer> {
-  serverPort = getRandomPort();
-  const server = new OPCUAServer({
-    port: serverPort,
-    resourcePath: '/UA/i3xTest',
-    nodeset_filename: [nodesets.standard],
-    maxConnectionsPerEndpoint: 10,
-    serverInfo: {
-      applicationName: { text: 'i3x E2E Test Server' },
-      applicationUri: 'urn:i3x:e2e:test',
-      productUri: 'urn:i3x:e2e:test',
-    },
-  });
-
-  await server.initialize();
-
-  const addressSpace = server.engine.addressSpace;
-  if (!addressSpace) throw new Error('Address space not initialized');
-  const namespace: Namespace = addressSpace.getOwnNamespace();
-
-  // ── ProductionLine (folder) ──────────────────────────────
-  const productionLine = namespace.addObject({
-    organizedBy: addressSpace.rootFolder.objects,
-    browseName: 'ProductionLine',
-    displayName: 'Production Line #1',
-  });
-
-  // ── Machine A ────────────────────────────────────────────
-  const machineA = namespace.addObject({
-    componentOf: productionLine,
-    browseName: 'MachineA',
-    displayName: 'CNC Milling Machine',
-  });
-
-  let temperatureA = 65.0;
-  namespace.addVariable({
-    componentOf: machineA,
-    browseName: 'Temperature',
-    displayName: 'Temperature',
-    dataType: DataType.Double,
-    value: {
-      get: () => new Variant({ dataType: DataType.Double, value: temperatureA }),
-    },
-  });
-
-  let speedA = 1500;
-  namespace.addVariable({
-    componentOf: machineA,
-    browseName: 'SpindleSpeed',
-    displayName: 'Spindle Speed',
-    dataType: DataType.Int32,
-    value: {
-      get: () => new Variant({ dataType: DataType.Int32, value: speedA }),
-    },
-  });
-
-  namespace.addVariable({
-    componentOf: machineA,
-    browseName: 'Status',
-    displayName: 'Status',
-    dataType: DataType.String,
-    value: {
-      get: () => new Variant({ dataType: DataType.String, value: 'Running' }),
-    },
-  });
-
-  // Method: Reset
-  const uaResetMethod = namespace.addMethod(machineA, {
-    browseName: 'Reset',
-    displayName: 'Reset Machine',
-    inputArguments: [],
-    outputArguments: [],
-  });
-  uaResetMethod.bindMethod(async (_inputArguments, _context) => {
-    temperatureA = 25.0;
-    speedA = 0;
-    return { statusCode: StatusCodes.Good };
-  });
-
-  // ── Machine B ────────────────────────────────────────────
-  const machineB = namespace.addObject({
-    componentOf: productionLine,
-    browseName: 'MachineB',
-    displayName: 'Laser Cutter',
-  });
-
-  namespace.addVariable({
-    componentOf: machineB,
-    browseName: 'Temperature',
-    displayName: 'Temperature',
-    dataType: DataType.Double,
-    value: {
-      get: () => new Variant({ dataType: DataType.Double, value: 42.7 }),
-    },
-  });
-
-  namespace.addVariable({
-    componentOf: machineB,
-    browseName: 'LaserPower',
-    displayName: 'Laser Power',
-    dataType: DataType.Float,
-    value: {
-      get: () => new Variant({ dataType: DataType.Float, value: 2400.5 }),
-    },
-  });
-
-  namespace.addVariable({
-    componentOf: machineB,
-    browseName: 'JobCount',
-    displayName: 'Job Count',
-    dataType: DataType.UInt32,
-    value: {
-      get: () => new Variant({ dataType: DataType.UInt32, value: 1247 }),
-    },
-  });
-
-  // ── CoffeeMachine (deeply nested for subscription test) ──
-  const coffeeMachine = namespace.addObject({
-    componentOf: productionLine,
-    browseName: 'CoffeeMachine',
-    displayName: 'Coffee Machine Pro 3000',
-  });
-
-  // ParameterSet (sub-object containing UAVariables)
-  const parameterSet = namespace.addObject({
-    componentOf: coffeeMachine,
-    browseName: 'ParameterSet',
-    displayName: 'ParameterSet',
-  });
-
-  // All CoffeeMachine variables MONOTONICALLY INCREASE
-  // so every OPC UA sample is a guaranteed DataChange
-  let brewTemperature = 93.0;
-  let _brewTick = 0;
-  namespace.addVariable({
-    componentOf: parameterSet,
-    browseName: 'BrewTemperature',
-    displayName: 'Brew Temperature',
-    dataType: DataType.Double,
-    value: {
-      get: () => new Variant({ dataType: DataType.Double, value: brewTemperature }),
-    },
-  });
-
-  let pumpPressure = 9.0;
-  namespace.addVariable({
-    componentOf: parameterSet,
-    browseName: 'PumpPressure',
-    displayName: 'Pump Pressure',
-    dataType: DataType.Double,
-    value: {
-      get: () => new Variant({ dataType: DataType.Double, value: pumpPressure }),
-    },
-  });
-
-  let waterLevel = 100.0;
-  namespace.addVariable({
-    componentOf: parameterSet,
-    browseName: 'WaterLevel',
-    displayName: 'Water Level',
-    dataType: DataType.Float,
-    value: {
-      get: () => new Variant({ dataType: DataType.Float, value: waterLevel }),
-    },
-  });
-
-  // GrinderUnit — nested 2 levels deep inside CoffeeMachine
-  const grinderUnit = namespace.addObject({
-    componentOf: coffeeMachine,
-    browseName: 'GrinderUnit',
-    displayName: 'Grinder Unit',
-  });
-
-  let grinderRPM = 1200;
-  namespace.addVariable({
-    componentOf: grinderUnit,
-    browseName: 'RPM',
-    displayName: 'Grinder RPM',
-    dataType: DataType.Int32,
-    value: {
-      get: () => new Variant({ dataType: DataType.Int32, value: grinderRPM }),
-    },
-  });
-
-  let grindSizeTick = 0;
-  const grindSizes = ['Coarse', 'Medium', 'Fine', 'Extra-Fine'];
-  namespace.addVariable({
-    componentOf: grinderUnit,
-    browseName: 'GrindSize',
-    displayName: 'Grind Size',
-    dataType: DataType.String,
-    value: {
-      get: () =>
-        new Variant({
-          dataType: DataType.String,
-          value: grindSizes[grindSizeTick % grindSizes.length],
-        }),
-    },
-  });
-
-  // Monotonically increase every 200ms for clear evidence
-  const interval = setInterval(() => {
-    _brewTick++;
-    temperatureA += 0.5;
-    brewTemperature += 0.1; // 93.0 → 93.1 → 93.2 → ...
-    pumpPressure += 0.05; //  9.0 →  9.05 →  9.10 → ...
-    waterLevel -= 0.3; // 100 → 99.7 → 99.4 → ...
-    grinderRPM += 10; // 1200 → 1210 → 1220 → ...
-    grindSizeTick++; // cycles through sizes
-  }, 200);
-  (server as Record<string, unknown>)._e2eInterval = interval;
-
-  await server.start();
-  return server;
-}
-
-// ── Test suite ───────────────────────────────────────────────
-
-describe('E2E: OPC UA Server → i3X REST API', () => {
-  let app: Awaited<ReturnType<typeof createApp>>;
-  let modelService: ModelService;
-  let subscriptionService: SubscriptionService;
-
-  beforeAll(async () => {
-    // 1. Start a real OPC UA server
-    opcuaServer = await startTestOpcUaServer();
-    const endpointUrl = `opc.tcp://localhost:${serverPort}/UA/i3xTest`;
-
-    // 2. Create the connector
-    const logger = consoleLogger;
-    const opcuaClient = new OpcUaClient(
-      {
-        endpointUrl,
-        securityMode: 'None',
-        optimizedClient: 'auto',
-      },
-      logger,
-    );
-    const dataSource = new OpcUaDataSourceAdapter(opcuaClient, logger);
-
-    // 3. Domain services
-    modelService = new ModelService(dataSource, logger);
-    const valueService = new ValueService(dataSource, modelService, logger);
-    const historyService = new HistoryService(dataSource, modelService, logger);
-    subscriptionService = new SubscriptionService(dataSource, modelService, logger, 1);
-
-    // 4. REST server
-    app = await createApp({
-      dataSource,
-      modelService,
-      valueService,
-      historyService,
-      subscriptionService,
-      logger,
-    });
-
-    // 5. Connect and preload
-    await dataSource.connect();
-    await modelService.preloadModel();
-  }, 30_000);
-
-  afterAll(async () => {
-    await subscriptionService.close();
-    const ds = (app as Record<string, unknown>).deps as Record<
-      string,
-      { disconnect: () => Promise<void> }
-    >;
-    if (ds?.dataSource) await ds.dataSource.disconnect();
-    clearInterval(
-      (opcuaServer as Record<string, unknown>)._e2eInterval as NodeJS.Timeout,
-    );
-    await opcuaServer.shutdown(500);
-  }, 15_000);
-
-  // ── Info ─────────────────────────────────────────────────
-
-  it('GET /v1/info returns server capabilities', async () => {
-    const res = await app.inject({ method: 'GET', url: '/v1/info' });
-    expect(res.statusCode).toBe(200);
-    const body = res.json();
-    expect(body.success).toBe(true);
-    expect(body.result.specVersion).toBe('beta');
-    expect(body.result.capabilities.query.history).toBe(true);
-  });
-
-  // ── Health ───────────────────────────────────────────────
-
-  it('GET /health returns ok', async () => {
-    const res = await app.inject({ method: 'GET', url: '/health' });
-    expect(res.statusCode).toBe(200);
-    expect(res.json().status).toBe('ok');
-  });
-
-  it('GET /ready returns ready when connected', async () => {
-    const res = await app.inject({ method: 'GET', url: '/ready' });
-    expect(res.statusCode).toBe(200);
-  });
-
-  // ── Namespaces ───────────────────────────────────────────
-
-  it('GET /v1/namespaces returns real OPC UA namespaces', async () => {
-    const res = await app.inject({ method: 'GET', url: '/v1/namespaces' });
-    expect(res.statusCode).toBe(200);
-    const body = res.json();
-    expect(body.success).toBe(true);
-    expect(body.result.length).toBeGreaterThanOrEqual(2);
-    // First namespace is always the OPC UA standard namespace
-    expect(body.result[0].uri).toContain('opcfoundation.org');
-  });
-
-  // ── Object types ─────────────────────────────────────────
-
-  it('GET /v1/objecttypes returns browsed types', async () => {
-    const res = await app.inject({ method: 'GET', url: '/v1/objecttypes' });
-    expect(res.statusCode).toBe(200);
-    const body = res.json();
-    expect(body.success).toBe(true);
-    expect(body.result.length).toBeGreaterThanOrEqual(1);
-  });
-
-  // ── Objects ──────────────────────────────────────────────
-
-  it('GET /v1/objects returns root-level objects from OPC UA', async () => {
-    const res = await app.inject({ method: 'GET', url: '/v1/objects' });
-    expect(res.statusCode).toBe(200);
-    const body = res.json();
-    expect(body.success).toBe(true);
-    expect(body.result.length).toBeGreaterThanOrEqual(1);
-    // Should find our ProductionLine
-    const names = body.result.map((r: Record<string, string>) => r.displayName);
-    expect(names).toContain('Production Line #1');
-  });
-
-  it('POST /v1/objects/list resolves a real element by id', async () => {
-    const model = await modelService.getOrBuildModel();
-    const rootId = model.rootIds[0]!;
-
-    const res = await app.inject({
-      method: 'POST',
-      url: '/v1/objects/list',
-      payload: { elementIds: [rootId] },
-    });
-    expect(res.statusCode).toBe(200);
-    const body = res.json();
-    expect(body.results[0].success).toBe(true);
-    expect(body.results[0].result.displayName).toBeTruthy();
-  });
-
-  // ── Values ───────────────────────────────────────────────
-
-  it('POST /v1/objects/value reads real OPC UA variable values', async () => {
-    const model = await modelService.getOrBuildModel();
-
-    // Find a property node (a Variable)
-    const propId = [...model.propertyToSource.keys()][0]!;
-
-    const res = await app.inject({
-      method: 'POST',
-      url: '/v1/objects/value',
-      payload: { elementIds: [propId], maxDepth: 1 },
-    });
-    expect(res.statusCode).toBe(200);
-    const body = res.json();
-    expect(body.results[0].success).toBe(true);
-    expect(body.results[0].result.value).not.toBeNull();
-    expect(body.results[0].result.quality).toBe('Good');
-  });
-
-  it('POST /v1/objects/value returns composition for asset nodes', async () => {
-    const model = await modelService.getOrBuildModel();
-
-    // Find a root asset with children
-    const assetId = model.rootIds.find((id) => {
-      const node = model.nodesById.get(id);
-      return node && node.children.length > 0;
-    });
-
-    if (!assetId) return; // skip if server has no suitable nodes
-
-    const res = await app.inject({
-      method: 'POST',
-      url: '/v1/objects/value',
-      payload: { elementIds: [assetId], maxDepth: 2 },
-    });
-    expect(res.statusCode).toBe(200);
-    const body = res.json();
-    expect(body.results[0].success).toBe(true);
-    expect(body.results[0].result.isComposition).toBe(true);
-  });
-
-  // ── Subscriptions (full lifecycle) ───────────────────────
-
-  it('full subscription lifecycle: create → register → sync → delete', async () => {
-    const model = await modelService.getOrBuildModel();
-    const propId = [...model.propertyToSource.keys()][0]!;
-
-    // Create subscription
-    const createRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions',
-      payload: { clientId: 'e2e-test', displayName: 'E2E Subscription' },
-    });
-    expect(createRes.statusCode).toBe(200);
-    const subId = createRes.json().result.subscriptionId;
-    expect(subId).toBeTruthy();
-
-    // Register a monitored item
-    const regRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/register',
-      payload: { subscriptionId: subId, elementIds: [propId], maxDepth: 1 },
-    });
-    expect(regRes.statusCode).toBe(200);
-    expect(regRes.json().success).toBe(true);
-    expect(regRes.json().results[0].success).toBe(true);
-    expect(regRes.json().results[0].elementId).toBe(propId);
-
-    // Wait a moment for data changes to arrive
-    await new Promise((r) => setTimeout(r, 2000));
-
-    // Sync — should have updates
-    const syncRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/sync',
-      payload: { subscriptionId: subId, acknowledgeSequence: 0 },
-    });
-    expect(syncRes.statusCode).toBe(200);
-    const updates = syncRes.json().result;
-    // Updates may or may not be there depending on timing,
-    // but the call must succeed
-    expect(Array.isArray(updates)).toBe(true);
-
-    // List subscriptions
-    const listRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/list',
-      payload: { subscriptionIds: [subId] },
-    });
-    expect(listRes.statusCode).toBe(200);
-    expect(listRes.json().results[0].success).toBe(true);
-    expect(listRes.json().results[0].subscriptionId).toBe(subId);
-    expect(listRes.json().results[0].result.subscriptionId).toBe(subId);
-
-    // Delete
-    const delRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/delete',
-      payload: { subscriptionIds: [subId] },
-    });
-    expect(delRes.statusCode).toBe(200);
-    expect(delRes.json().results[0].success).toBe(true);
-  }, 15_000);
-
-  // ── Deep subscription: CoffeeMachine nested monitoring ───
-
-  it('deep subscribe: monitoring CoffeeMachine auto-discovers nested ParameterSet + GrinderUnit variables', async () => {
-    const model = await modelService.getOrBuildModel();
-
-    // 1. Find the CoffeeMachine asset (top-level object)
-    const coffeeNode = [...model.nodesById.values()].find(
-      (n) => n.name === 'Coffee Machine Pro 3000',
-    );
-    expect(coffeeNode).toBeTruthy();
-    const coffeeId = coffeeNode!.id;
-
-    // Verify nested structure exists in the model
-    const childIds = model.childrenById.get(coffeeId) ?? [];
-    expect(childIds.length).toBeGreaterThanOrEqual(2); // ParameterSet + GrinderUnit
-
-    // Find ParameterSet children
-    const paramSetNode = [...model.nodesById.values()].find(
-      (n) => n.name === 'ParameterSet' && childIds.includes(n.id),
-    );
-    expect(paramSetNode).toBeTruthy();
-
-    const paramChildren = model.childrenById.get(paramSetNode!.id) ?? [];
-    const paramPropertyNames = paramChildren
-      .map((id) => model.nodesById.get(id))
-      .filter(Boolean)
-      .filter((n) => n!.kind === 'property')
-      .map((n) => n!.name);
-    expect(paramPropertyNames).toContain('Brew Temperature');
-    expect(paramPropertyNames).toContain('Pump Pressure');
-    expect(paramPropertyNames).toContain('Water Level');
-
-    // Find GrinderUnit children
-    const grinderNode = [...model.nodesById.values()].find(
-      (n) => n.name === 'Grinder Unit' && childIds.includes(n.id),
-    );
-    expect(grinderNode).toBeTruthy();
-
-    const grinderChildren = model.childrenById.get(grinderNode!.id) ?? [];
-    const grinderPropertyNames = grinderChildren
-      .map((id) => model.nodesById.get(id))
-      .filter(Boolean)
-      .filter((n) => n!.kind === 'property')
-      .map((n) => n!.name);
-    expect(grinderPropertyNames).toContain('Grinder RPM');
-    expect(grinderPropertyNames).toContain('Grind Size');
-
-    // 2. Create subscription
-    const createRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions',
-      payload: { clientId: 'deep-test', displayName: 'Deep CoffeeMachine Sub' },
-    });
-    expect(createRes.statusCode).toBe(200);
-    const subId = createRes.json().result.subscriptionId;
-
-    // 3. Register the TOP-LEVEL CoffeeMachine with maxDepth=3
-    //    This should auto-discover ALL nested variables:
-    //    CoffeeMachine (depth 0)
-    //      → ParameterSet (depth 1)
-    //          → BrewTemperature, PumpPressure, WaterLevel (depth 2, property)
-    //      → GrinderUnit (depth 1)
-    //          → RPM, GrindSize (depth 2, property)
-    const regRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/register',
-      payload: {
-        subscriptionId: subId,
-        elementIds: [coffeeId],
-        maxDepth: 3,
-      },
-    });
-    expect(regRes.statusCode).toBe(200);
-    expect(regRes.json().success).toBe(true);
-    expect(regRes.json().results[0].success).toBe(true);
-    expect(regRes.json().results[0].elementId).toBe(coffeeId);
-
-    // 4. Verify list shows the CoffeeMachine as monitored
-    const listRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/list',
-      payload: { subscriptionIds: [subId] },
-    });
-    expect(listRes.statusCode).toBe(200);
-    const detail = listRes.json().results[0].result;
-    expect(detail.monitoredObjects).toHaveLength(1);
-    expect(detail.monitoredObjects[0].elementId).toBe(coffeeId);
-    expect(detail.monitoredObjects[0].maxDepth).toBe(3);
-    // Should be running in native mode (real OPC UA subscription)
-    expect(detail.mode).toBe('native');
-
-    // 5. Wait for data change notifications
-    //    Values change every 200ms, subscription publishes at 1s
-    //    → after 3s we should have multiple notifications
-    await new Promise((r) => setTimeout(r, 3000));
-
-    // 6. Sync — should have composite updates for the CoffeeMachine
-    const syncRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/sync',
-      payload: { subscriptionId: subId, acknowledgeSequence: 0 },
-    });
-    expect(syncRes.statusCode).toBe(200);
-    const updates = syncRes.json().result;
-    expect(Array.isArray(updates)).toBe(true);
-    expect(updates.length).toBeGreaterThan(0);
-
-    // With composite values, all updates have elementId = CoffeeMachine
-    for (const update of updates) {
-      expect(update.elementId).toBe(coffeeId);
-      expect(update.sequenceNumber).toBeGreaterThan(0);
-      expect(update.timestamp).toBeTruthy();
-    }
-
-    // The latest update should be a composite with components
-    // from nested properties (BrewTemperature, PumpPressure, etc.)
-    const latest = updates[updates.length - 1];
-    expect(latest.value).toBeTruthy();
-    expect(latest.value.isComposition).toBe(true);
-    expect(latest.value.components).toBeTruthy();
-
-    const componentKeys = Object.keys(latest.value.components);
-    // Should have at least 3 nested property components
-    expect(componentKeys.length).toBeGreaterThanOrEqual(3);
-
-    // Each component should be a VQT
-    for (const key of componentKeys) {
-      const vqt = latest.value.components[key];
-      expect(vqt.quality).toBeTruthy();
-      expect(vqt.timestamp).toBeTruthy();
-      // Value should be defined (initial data change fired)
-      expect(vqt.value).toBeDefined();
-    }
-
-    // ── Print evidence ──
-    console.log(`\n╔══════════════════════════════════════════════════╗`);
-    console.log(`║  Deep Subscription: CoffeeMachine composite       ║`);
-    console.log(
-      `║  ${updates.length} updates, ${componentKeys.length} components                    ║`,
-    );
-    console.log(`╠══════════════════════════════════════════════════╣`);
-    for (const key of componentKeys) {
-      const vqt = latest.value.components[key];
-      const name = model.nodesById.get(key)?.name ?? key;
-      const val =
-        typeof vqt.value === 'number' ? vqt.value.toFixed(2) : String(vqt.value);
-      console.log(`║  ${name.padEnd(20)} │ ${val}`);
-    }
-    console.log(`╚══════════════════════════════════════════════════╝\n`);
-
-    // 7. Cleanup
-    const delRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/delete',
-      payload: { subscriptionIds: [subId] },
-    });
-    expect(delRes.statusCode).toBe(200);
-  }, 20_000);
-
-  // ── Subscription value must match /objects/value shape ─────
-
-  it('subscription composite matches /objects/value format for the same asset', async () => {
-    const model = await modelService.getOrBuildModel();
-
-    // Find the CoffeeMachine asset
-    const coffeeNode = [...model.nodesById.values()].find(
-      (n) => n.name === 'Coffee Machine Pro 3000',
-    );
-    expect(coffeeNode).toBeTruthy();
-    const coffeeId = coffeeNode!.id;
-
-    // ── Step 1: Read the canonical value via /objects/value ──
-    const valueRes = await app.inject({
-      method: 'POST',
-      url: '/v1/objects/value',
-      payload: { elementIds: [coffeeId], maxDepth: 3 },
-    });
-    expect(valueRes.statusCode).toBe(200);
-    const valueResult = valueRes.json().results[0].result;
-    expect(valueResult.isComposition).toBe(true);
-    expect(valueResult.components).toBeTruthy();
-
-    const canonicalKeys = Object.keys(valueResult.components).sort();
-    expect(canonicalKeys.length).toBeGreaterThanOrEqual(3);
-
-    // ── Step 2: Create subscription and register same asset ──
-    const createRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions',
-      payload: { clientId: 'match-test' },
-    });
-    const subId = createRes.json().result.subscriptionId;
-
-    const regRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/register',
-      payload: { subscriptionId: subId, elementIds: [coffeeId], maxDepth: 3 },
-    });
-    expect(regRes.statusCode).toBe(200);
-    expect(regRes.json().results[0].success).toBe(true);
-
-    // ── Step 3: Wait for initial data, then sync ──
-    await new Promise((r) => setTimeout(r, 3000));
-
-    const syncRes = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/sync',
-      payload: { subscriptionId: subId, acknowledgeSequence: 0 },
-    });
-    expect(syncRes.statusCode).toBe(200);
-    const updates = syncRes.json().result;
-    expect(updates.length).toBeGreaterThan(0);
-
-    // ── Step 4: The critical assertion ──
-    // The latest sync update must have the SAME component keys
-    // as /objects/value — if they differ, the explorer can't
-    // correlate subscription data with its object model.
-    const latest = updates[updates.length - 1];
-
-    // elementId must match what we registered
-    expect(latest.elementId).toBe(coffeeId);
-
-    // value must be a composition
-    expect(latest.value.isComposition).toBe(true);
-    expect(latest.value.components).toBeTruthy();
-    expect(latest.value.components).not.toEqual({});
-
-    const subscriptionKeys = Object.keys(latest.value.components).sort();
-
-    // THE CRITICAL CHECK: same component keys as /objects/value
-    expect(subscriptionKeys).toEqual(canonicalKeys);
-
-    // Each component must have VQT shape (value, quality, timestamp)
-    for (const key of subscriptionKeys) {
-      const vqt = latest.value.components[key];
-      expect(vqt).toHaveProperty('value');
-      expect(vqt).toHaveProperty('quality');
-      expect(vqt).toHaveProperty('timestamp');
-    }
-
-    // Cleanup
-    await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/delete',
-      payload: { subscriptionIds: [subId] },
-    });
-  }, 20_000);
-
-  // ── Error handling ───────────────────────────────────────
-
-  it('POST /v1/objects/list returns 404 for unknown elements', async () => {
-    const res = await app.inject({
-      method: 'POST',
-      url: '/v1/objects/list',
-      payload: { elementIds: ['nonexistent-element-id'] },
-    });
-    expect(res.statusCode).toBe(200);
-    const body = res.json();
-    expect(body.results[0].success).toBe(false);
-    expect(body.results[0].error.code).toBe(404);
-  });
-
-  it('POST /v1/subscriptions/stream returns 404 for unknown subscription', async () => {
-    const res = await app.inject({
-      method: 'POST',
-      url: '/v1/subscriptions/stream',
-      payload: { subscriptionId: 'does-not-exist' },
-    });
-    expect(res.statusCode).toBe(404);
-  });
-});