@cratis/chronicle 6.41.0 → 6.41.1

package/testing/singleAppendRoute.ts

@@ -6,16 +6,21 @@ import { UnsupportedEventSequenceOperation } from './UnsupportedEventSequenceOpe
 
 /** The fixture-proven single-append route boundary, shared with read-model setup. */
 export function singleAppendRoute(options: AppendOptions | undefined, artifact: string,
-    unsupported = (operation: string, artifact: string, reason: string) => new UnsupportedEventSequenceOperation(operation, artifact, reason)
+    unsupported = (operation: string, artifact: string, reason: string) => new UnsupportedEventSequenceOperation(operation, artifact, reason),
+    allowSubject = false
 ): { sourceType: string; streamType: string; streamId: string } {
     if (options !== undefined && (!options || typeof options !== 'object' || Array.isArray(options))) {
         throw unsupported('append.options', artifact, 'Append options must be an object.');
     }
-    if (options && (Reflect.ownKeys(options).some(key => !['sourceType', 'streamType', 'streamId'].includes(String(key))) ||
-        options.correlationId !== undefined || options.subject !== undefined || options.occurred !== undefined ||
+    const allowed = ['sourceType', 'streamType', 'streamId', ...(allowSubject ? ['subject'] : [])];
+    if (options && (Reflect.ownKeys(options).some(key => !allowed.includes(String(key))) ||
+        options.correlationId !== undefined || (!allowSubject && options.subject !== undefined) || options.occurred !== undefined ||
         options.eventSourceId !== undefined || options.concurrencyScope !== undefined || options.tags !== undefined ||
         options.concurrencyScopes !== undefined)) {
-        throw unsupported('append.options', artifact, 'Only sourceType, streamType and streamId routing options are fixture-backed for single append.');
+        throw unsupported('append.options', artifact, `Only ${allowed.join(', ')} options are supported for single append.`);
+    }
+    if (options?.subject !== undefined && options.subject !== null && typeof options.subject !== 'string') {
+        throw unsupported('append.subject', artifact, 'Subject must be a string.');
     }
     const route = { sourceType: options?.sourceType, streamType: options?.streamType, streamId: options?.streamId };
     for (const [name, value] of Object.entries(route)) {