Fix failing test on the main branch

[Shubh-Raj]

Description

There are test failures on the main branch that need to be addressed.

Failing Tests

1. TemplateArchiveProcessor - should draft a template (Consistent Failure)

File: test/TemplateArchiveProcessor.test.ts

Error: ValidationException: Model violation in the "org.accordproject.commonmark@0.5.0.Document" instance. The field "nodes" has a value of "undefined". Expected type of value: "Node[]".

Cause: The draft() method returns an AgreementMark document with undefined nodes field, which fails Concerto validation.

2. JavaScriptEvaluator - should pass stress test with javascript safe (Flaky/Intermittent)

File: test/JavaScriptEvaluator.test.ts

Error: expect(error).toBeFalsy() receives {code: 0, elapsed: 195, result: undefined}

Cause: This appears to be a timing/race condition in the parallel execution stress test. It fails intermittently.

Steps to Reproduce

npm install
npm test

Environment

• Node: v20+
• OS: Linux
• Branch: main

Additional Notes

• The JavaScriptEvaluator test may not fail on every run (flaky)
• A worker process warning about force exit suggests possible resource leaks

[Kovidgupta1234]

Hi Mentor,
We are seeing two test failures on the main branch:

TemplateArchiveProcessor – should draft a template (consistent failure)

The draft() method returns an AgreementMark document with nodes set to undefined.

Since the Concerto model requires nodes: Node[], this causes a validation error.

Fix: Ensure nodes is always initialized (at least as an empty array) before returning the document.

JavaScriptEvaluator – stress test with javascript safe (flaky failure)

The test fails intermittently due to a timing/race condition in parallel execution.

Fix: Properly await all async operations, isolate shared state, and add cleanup to prevent worker force-exit warnings.

We are working on these changes to stabilize the tests. Please let us know if you have any suggestions.