test(service-job,service-automation): bind the two new engine doubles to assertEngineUpdateDispatch (#5548)

Author: os-help

packages/services/service-automation/package.json

@@ -24,6 +24,7 @@
   },
   "devDependencies": {
     "@objectstack/driver-sql": "workspace:*",
+    "@objectstack/metadata-core": "workspace:*",
     "@objectstack/objectql": "workspace:*",
     "@objectstack/plugin-security": "workspace:*",
     "@objectstack/service-job": "workspace:*",

packages/services/service-automation/src/builtin/wait-node-degraded-run.test.ts

@@ -1,6 +1,7 @@
 // Copyright (c) 2025 ObjectStack. Licensed under the Apache-2.0 license.
 
 import { describe, it, expect } from 'vitest';
+import { assertEngineUpdateDispatch } from '@objectstack/metadata-core';
 import { DbJobAdapter } from '@objectstack/service-job';
 import type { IJobService, JobSchedule, JobHandler } from '@objectstack/spec/contracts';
 import { AutomationEngine } from '../engine.js';
@@ -66,7 +67,11 @@ function makeFakeEngine() {
       tables.set(table, t);
       return { id: data.id };
     },
-    async update(table: string, patch: any) {
+    async update(table: string, patch: any, options?: any) {
+      // Same binding as the sibling double in `service-job`: the fake refuses
+      // exactly what `ObjectQLEngine.update` refuses, so the audit writes this
+      // test asserts are writes a real server would have accepted.
+      assertEngineUpdateDispatch(patch, options);
       const t = tables.get(table) ?? [];
       const r = t.find((x) => x.id === patch.id);
       if (!r) throw new Error(`row ${patch.id} not in ${table}`);

packages/services/service-job/package.json

@@ -25,6 +25,7 @@
     "croner": "^10.0.1"
   },
   "devDependencies": {
+    "@objectstack/metadata-core": "workspace:*",
     "@types/node": "^26.1.2",
     "typescript": "^6.0.3",
     "vitest": "^4.1.10"

packages/services/service-job/src/db-job-adapter.degraded-outcome.test.ts

@@ -1,6 +1,7 @@
 // Copyright (c) 2025 ObjectStack. Licensed under the Apache-2.0 license.
 
 import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { assertEngineUpdateDispatch } from '@objectstack/metadata-core';
 import type { JobRunOutcome } from '@objectstack/spec/contracts';
 import { DbJobAdapter } from './db-job-adapter.js';
 import { IntervalJobAdapter } from './interval-job-adapter.js';
@@ -39,7 +40,12 @@ function makeFakeEngine() {
       tables.set(table, t);
       return { id: data.id };
     },
-    async update(table: string, patch: any) {
+    async update(table: string, patch: any, options?: any) {
+      // Bind the double to the producer's own dispatch rule rather than
+      // mirroring it by hand: `DbJobAdapter` always updates by a scalar
+      // payload id, and a fake that accepts a call a real server answers 500
+      // to would let that drift through unnoticed.
+      assertEngineUpdateDispatch(patch, options);
       const t = tables.get(table) ?? [];
       const r = t.find((x) => x.id === patch.id);
       if (!r) throw new Error(`row ${patch.id} not in ${table}`);