@onlineapps/conn-orch-validator 3.2.0 → 3.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/conn-orch-validator",
-  "version": "3.2.0",
+  "version": "3.2.2",
   "description": "Validation orchestrator for OA Drive microservices - coordinates validation across all layers (base, infra, orch, business)",
   "main": "src/index.js",
   "bin": {

package/src/CookbookTestRunner.js

@@ -185,11 +185,17 @@ class CookbookTestRunner {
       // { modulePath, exportName } (v3 handler dispatch is the only mode).
       const spec = await this.resolveOperation(step.service, step.operation);
 
-      const validationTenantId = process.env.OA_VALIDATION_TENANT_ID;
-      const validationWorkspaceId = process.env.OA_VALIDATION_WORKSPACE_ID;
-      if (!validationTenantId || !validationWorkspaceId) {
+      const rawTenantId = process.env.OA_VALIDATION_TENANT_ID;
+      const rawWorkspaceId = process.env.OA_VALIDATION_WORKSPACE_ID;
+      if (!rawTenantId || !rawWorkspaceId) {
         throw new Error('[CookbookTestRunner] Missing required environment variables OA_VALIDATION_TENANT_ID and/or OA_VALIDATION_WORKSPACE_ID');
       }
+      // Coerce numeric env vars to integers so ctx matches production
+      // ContextBuilder (which receives already-typed values from the MQ envelope).
+      // Handlers that check `Number.isInteger(ctx.tenant_id)` would otherwise
+      // reject the string form and every workspace-scoped cookbook would fail.
+      const validationTenantId = /^\d+$/.test(rawTenantId) ? parseInt(rawTenantId, 10) : rawTenantId;
+      const validationWorkspaceId = /^\d+$/.test(rawWorkspaceId) ? parseInt(rawWorkspaceId, 10) : rawWorkspaceId;
 
       await this._dispatchViaHandler({
         step,
@@ -404,9 +410,16 @@ class CookbookTestRunner {
         continue;
       }
 
-      // Type check
+      // Type check — recognise arrays as 'array' (not 'object') and null as 'null'.
       if (expectation.type) {
-        const actualType = typeof actualValue;
+        let actualType;
+        if (Array.isArray(actualValue)) {
+          actualType = 'array';
+        } else if (actualValue === null) {
+          actualType = 'null';
+        } else {
+          actualType = typeof actualValue;
+        }
         if (actualType !== expectation.type) {
           errors.push(`Field ${field}: expected type ${expectation.type}, got ${actualType}`);
         }