data-primals-engine 1.0.11 → 1.1.1

package/test/globalSetup.js

@@ -4,7 +4,7 @@ import {getCollection} from "../src/modules/mongodb.js";
 
 export default async function () {
 
-// --- Configuration initiale ---
+    // --- Configuration initiale ---
     Config.Set("modules", ["mongodb", "data", "file", "bucket", "workflow","user", "assistant"]);
 
     console.log('\n--- Global Setup: Initializing engine and database for all tests ---');
@@ -12,4 +12,4 @@ export default async function () {
 
     // Initialisez et exportez les collections
     console.log('--- Global Setup: Complete. Engine is running. ---');
-}
+}

package/test/import_export.integration.test.js

@@ -10,7 +10,7 @@ import {
 
 import {
     modelsCollection as getAppModelsCollection,
-    getCollectionForUser as getAppUserCollection,
+    getCollectionForUser as getAppUserCollection
 } from 'data-primals-engine/modules/mongodb';
 import {sleep} from "data-primals-engine/core";
 import fs from "node:fs";
@@ -33,8 +33,8 @@ const impexTestModel = {
         { name: 'name', type: 'string', required: true },
         { name: 'sku', type: 'string', unique: true },
         { name: 'price', type: 'number', required: true },
-        { name: 'inStock', type: 'boolean', default: true },
-    ],
+        { name: 'inStock', type: 'boolean', default: true }
+    ]
 };
 
 // --- Setup de l'environnement de test ---
@@ -62,7 +62,7 @@ describe('Intégration des fonctions d\'Import/Export', () => {
     // Préparation avant chaque test du bloc
     beforeEach(async () => {
         testModelsColInstance = getAppModelsCollection;
-        testDatasColInstance = getAppUserCollection(mockUser);
+        testDatasColInstance = await getAppUserCollection(mockUser);
 
         // Nettoyage complet pour un état propre
         await testDatasColInstance.deleteMany({ _user: "testuserImpex"});
@@ -74,7 +74,7 @@ describe('Intégration des fonctions d\'Import/Export', () => {
         await insertData(impexTestModel.name, [
             { name: 'Produit A', sku: 'SKU-A', price: 10.50, inStock: true },
             { name: 'Produit B', sku: 'SKU-B', price: 25.00, inStock: false },
-            { name: 'Produit C', sku: 'SKU-C', price: 99.99, inStock: true },
+            { name: 'Produit C', sku: 'SKU-C', price: 99.99, inStock: true }
         ], {}, mockUser, false);
     });
 
@@ -82,7 +82,7 @@ describe('Intégration des fonctions d\'Import/Export', () => {
         it('devrait exporter les données en format JSON', async () => {
             const res= await exportData({
                 models: [impexTestModel.name],
-                depth: 1,
+                depth: 1
             }, mockUser);
 
             expect(res.success).toBeTruthy();

package/test/model.integration.test.js

@@ -38,7 +38,7 @@ async function setupTestContext() {
             {name: 'relatedName', type: 'string', required: true, unique: true},
             {name: 'relatedValue', type: 'number'}
         ],
-        maxRequestData: 10,
+        maxRequestData: 10
     };
 
     const comprehensiveTestModelDefinition = {
@@ -90,9 +90,9 @@ async function setupTestContext() {
             {name: 'objectField', type: 'object'},
             // Model & ModelField (validation of string format, not existence)
             {name: 'modelNameField', type: 'model'},
-            {name: 'modelFieldNameField', type: 'modelField'}, // Note: modelField type expects an object {model: 'modelName', field: 'fieldName'}
+            {name: 'modelFieldNameField', type: 'modelField'} // Note: modelField type expects an object {model: 'modelName', field: 'fieldName'}
         ],
-        maxRequestData: 50,
+        maxRequestData: 50
     };
 
     // Insérer les modèles en base
@@ -108,7 +108,7 @@ async function setupTestContext() {
     return {
         currentTestUser,
         comprehensiveTestModelDefinition,
-        relatedModelDefinition,
+        relatedModelDefinition
     };
 }
 
@@ -127,22 +127,21 @@ describe('CRUD on model definitions and integrity tests', () => {
         it('should create and drop index when field.index is toggled (premium user)', async () => {
             // --- SETUP ---
             const { currentTestUser, comprehensiveTestModelDefinition } = await setupTestContext();
-            const premiumUser = { ...currentTestUser, userPlan: 'premium' };
-            const dataCollection = getCollectionForUser(premiumUser);
+            const dataCollection = await getCollectionForUser(currentTestUser);
             const fieldToIndex = 'stringUnique'; // Utiliser un champ qui existe vraiment dans le modèle
 
+            // --- FIX: Ensure the collection exists before any operation ---
+            // By inserting and deleting a dummy document, we force MongoDB to create the
+            // collection and its default indexes. This prevents the "ns does not exist"
+            // error in asynchronous listeners (like workflow triggers).
+            const dummyDoc = await dataCollection.insertOne({ _model: 'dummy', _user: currentTestUser.username });
+            await dataCollection.deleteOne({ _id: dummyDoc.insertedId });
+
+
             // --- VERIFICATION INITIALE ---
             // S'assurer qu'aucun index n'existe au départ.
-            // On enveloppe l'appel dans un try/catch pour gérer le cas où la collection n'existe pas.
-            let initialIndexes = [];
-            try {
-                initialIndexes = await dataCollection.indexes();
-            } catch (e) {
-                // C'est normal si la collection n'existe pas encore. On considère qu'il n'y a pas d'index.
-                if (e.codeName !== 'NamespaceNotFound') {
-                    throw e; // Relancer les autres erreurs inattendues
-                }
-            }
+            // Cet appel ne plantera plus car la collection est maintenant créée.
+            const initialIndexes = await dataCollection.indexes();
             expect(initialIndexes.some(i => i.key[fieldToIndex] === 1)).toBe(false);
 
             // --- ACTION 1 : AJOUTER UN INDEX ---
@@ -150,9 +149,9 @@ describe('CRUD on model definitions and integrity tests', () => {
                 ...comprehensiveTestModelDefinition,
                 fields: comprehensiveTestModelDefinition.fields.map(f =>
                     f.name === fieldToIndex ? { ...f, index: true } : f
-                ),
+                )
             };
-            await editModel(premiumUser, testModelId, modelWithIndex);
+            await editModel(currentTestUser, testModelId, modelWithIndex);
 
             // --- VERIFICATION 1 ---
             // Maintenant, la collection et l'index doivent exister.
@@ -163,7 +162,7 @@ describe('CRUD on model definitions and integrity tests', () => {
             // Le filtre partiel est crucial pour que l'index ne s'applique qu'aux bonnes données
             expect(newIndex.partialFilterExpression).toEqual({
                 _model: comprehensiveTestModelDefinition.name,
-                _user: premiumUser.username,
+                _user: currentTestUser.username
             });
 
             // --- ACTION 2 : SUPPRIMER L'INDEX ---
@@ -171,14 +170,14 @@ describe('CRUD on model definitions and integrity tests', () => {
                 ...comprehensiveTestModelDefinition,
                 fields: comprehensiveTestModelDefinition.fields.map(f =>
                     f.name === fieldToIndex ? { ...f, index: false } : f
-                ),
+                )
             };
-            await editModel(premiumUser, testModelId, modelWithoutIndex);
+            await editModel(currentTestUser, testModelId, modelWithoutIndex);
 
             // --- VERIFICATION 2 ---
             const indexesAfterDeletion = await dataCollection.indexes();
             expect(indexesAfterDeletion.some(i => i.key[fieldToIndex] === 1)).toBe(false);
-        }, 20000); // Augmenter le timeout si nécessaire pour les opérations d'index
+        }, 20000);
 
         it('should not save extra, non-defined fields in the model definition', async () => {
             const { currentTestUser, comprehensiveTestModelDefinition, relatedModelDefinition } = await setupTestContext();

package/test/workflow.integration.test.js

@@ -33,7 +33,7 @@ const targetDataModel = {
         { name: 'projectName', type: 'string', required: true },
         { name: 'status', type: 'string', required: true }, // ex: 'new', 'active', 'archived'
         { name: 'budget', type: 'number' }
-    ],
+    ]
 };
 
 // Les modèles qui définissent le système de workflow lui-même.
@@ -130,9 +130,6 @@ const workflowMetaModels = [
 
 let testModelsColInstance;
 let testDatasColInstance;
-let engineInstance;
-
-const port = process.env.PORT || getUniquePort(); // Port différent
 
 // On "espionne" `c` pour vérifier qu'elle est appelée sans l'exécuter réellement.
 // Cela nous permet de tester uniquement la logique de déclenchement.
@@ -140,7 +137,7 @@ const processWorkflowRunSpy = vi.spyOn(workflowModule, 'processWorkflowRun');
 
 beforeEach(async () => {
     testModelsColInstance = getAppModelsCollection;
-    testDatasColInstance = getCollectionForUser(mockUser);
+    testDatasColInstance = await getCollectionForUser(mockUser);
     // Nettoyer les données avant chaque test
     await testDatasColInstance.deleteMany({_user: "testuserWorkflow"});
 

package/test/workflow.robustness.test.js

@@ -1,18 +1,11 @@
 import { ObjectId } from 'mongodb';
-import { expect, describe, it, beforeEach, afterAll, beforeAll, vi } from 'vitest';
+import { expect, describe, it, beforeEach, beforeAll, vi } from 'vitest';
 import { Config } from "data-primals-engine/config";
-
-// --- Configuration initiale (similaire à workflow.integration.test.js) ---
-let mongod;
-let testDbUri;
-const testDbName = 'testRobustnessDbHO_Workflow';
-
 // --- Importations des modules de l'application ---
-import { Engine } from "data-primals-engine/engine";
 import { insertData, editData } from 'data-primals-engine/modules/data';
 import { modelsCollection as getAppModelsCollection, getCollectionForUser, getCollection } from 'data-primals-engine/modules/mongodb';
 import * as workflowModule from 'data-primals-engine/modules/workflow';
-import {getUniquePort, initEngine} from "../src/setenv.js";
+import {initEngine} from "../src/setenv.js";
 import {maxExecutionsByStep} from "data-primals-engine/constants";
 
 vi.mock('data-primals-engine/modules/workflow', { spy: true })
@@ -44,7 +37,7 @@ beforeAll(async () =>{
 
 beforeEach(async () => {
     testModelsColInstance = getAppModelsCollection;
-    testDatasColInstance = getCollectionForUser(mockUser);
+    testDatasColInstance = await getCollectionForUser(mockUser);
     await testDatasColInstance.deleteMany({ _user: mockUser.username });
     await getCollection('job_locks').deleteMany({}); // Nettoyer les verrous
     const mods = await testModelsColInstance.find({ $and: [{_user: mockUser.username}, {$or: [{name: targetDataModel.name}, ...workflowMetaModels.map(m =>({name: m.name}))] }]}).toArray();
@@ -71,7 +64,7 @@ describe('Tests de robustesse et des cas limites du module Workflow', () => {
             const now = new Date();
             await jobsCollection.insertOne({
                 jobId: jobId,
-                lockedUntil: new Date(now.getTime() + 10 * 60 * 1000), // Verrouillé pour 10 min
+                lockedUntil: new Date(now.getTime() + 10 * 60 * 1000) // Verrouillé pour 10 min
             });
 
             // 2. Act: Tenter de lancer le job
@@ -89,7 +82,7 @@ describe('Tests de robustesse et des cas limites du module Workflow', () => {
             // 1. Arrange: Insérer un verrou qui a déjà expiré (simule un crash)
             await jobsCollection.insertOne({
                 jobId: jobId,
-                lockedUntil: new Date(Date.now() - 1000), // Expiré depuis 1 seconde
+                lockedUntil: new Date(Date.now() - 1000) // Expiré depuis 1 seconde
             });
 
             // 2. Act: Lancer le job
@@ -155,7 +148,7 @@ describe('Tests de robustesse et des cas limites du module Workflow', () => {
                 // Cette condition est conçue pour échouer car la donnée n'existera pas
                 conditions: { status: 'must-be-this-to-succeed' },
                 onSuccessStep: successStep.toString(),
-                onFailureStep: failureStep.toString(),
+                onFailureStep: failureStep.toString()
             };
             const startStep = (await insertData('workflowStep', startStepDef, {}, mockUser)).insertedIds[0];
             const workflow = (await insertData('workflow', { name: 'Failure Path Workflow', startStep: startStep.toString() }, {}, mockUser)).insertedIds[0];

package/vitest.config.js

@@ -11,6 +11,6 @@ export default defineConfig({
         testTimeout: 25000, // 20 seconds
 
         // Tell Vitest where to find your global setup and teardown files
-        globalTeardown: './test/globalTeardown.js',
-    },
+        globalTeardown: './test/globalTeardown.js'
+    }
 });