@memberjunction/react-test-harness 2.88.0 → 2.90.0

package/README.md

@@ -526,6 +526,104 @@ interface FluentMatcher {
 }
 ```
 
+## Parallel Testing
+
+### Important: Test Harness Instance Limitations
+
+The ReactTestHarness uses a single browser page instance and is **NOT safe for parallel test execution** on the same instance. This is due to Playwright's internal limitations with `exposeFunction` and potential race conditions when multiple tests try to modify the same page context simultaneously.
+
+### Sequential Testing (Single Instance)
+
+For sequential test execution, you can safely reuse a single harness instance:
+
+```typescript
+const harness = new ReactTestHarness({ headless: true });
+await harness.initialize();
+
+// ✅ CORRECT - Sequential testing on same instance
+for (const test of tests) {
+  const result = await harness.testComponent(test.code, test.props);
+  // Each test runs one after another, no conflicts
+}
+
+await harness.close();
+```
+
+### Parallel Testing (Multiple Instances)
+
+For parallel test execution, you **MUST** create separate ReactTestHarness instances:
+
+```typescript
+// ✅ CORRECT - Parallel testing with separate instances
+const results = await Promise.all(tests.map(async (test) => {
+  const harness = new ReactTestHarness({ headless: true });
+  await harness.initialize();
+  
+  try {
+    return await harness.testComponent(test.code, test.props);
+  } finally {
+    await harness.close(); // Clean up each instance
+  }
+}));
+```
+
+### Common Mistake to Avoid
+
+```typescript
+// ❌ WRONG - DO NOT DO THIS
+const harness = new ReactTestHarness({ headless: true });
+await harness.initialize();
+
+// This will cause conflicts and errors!
+const results = await Promise.all(
+  tests.map(test => harness.testComponent(test.code, test.props))
+);
+```
+
+This approach will fail with errors like:
+- "Function '__mjGetEntityObject' has been already registered"
+- "Cannot read properties of undefined (reading 'addBinding')"
+
+### Performance Considerations
+
+While creating multiple harness instances has some overhead (each launches its own browser context), the benefits of parallel execution typically outweigh this cost:
+
+- **Sequential (1 instance)**: Lower memory usage, but tests run one by one
+- **Parallel (N instances)**: Higher memory usage, but tests complete much faster
+
+### Example: Test Runner with Configurable Parallelism
+
+```typescript
+class TestRunner {
+  async runTests(tests: TestCase[], parallel = false) {
+    if (parallel) {
+      // Create new instance for each test
+      return Promise.all(tests.map(async (test) => {
+        const harness = new ReactTestHarness({ headless: true });
+        await harness.initialize();
+        try {
+          return await harness.testComponent(test.code, test.props);
+        } finally {
+          await harness.close();
+        }
+      }));
+    } else {
+      // Reuse single instance for all tests
+      const harness = new ReactTestHarness({ headless: true });
+      await harness.initialize();
+      
+      const results = [];
+      for (const test of tests) {
+        results.push(await harness.testComponent(test.code, test.props));
+      }
+      
+      await harness.close();
+      return results;
+    }
+  }
+}
+```
+
 ## Usage Examples for TypeScript Projects
 
 ### Creating a Reusable Test Utility

package/dist/lib/component-linter.d.ts

@@ -1,3 +1,4 @@
+import { ComponentSpec } from '@memberjunction/interactive-component-types';
 export interface LintResult {
     success: boolean;
     violations: Violation[];
@@ -22,7 +23,7 @@ export interface FixSuggestion {
 }
 export declare class ComponentLinter {
     private static universalComponentRules;
-    static lintComponent(code: string, componentName: string, componentSpec?: any, isRootComponent?: boolean): Promise<LintResult>;
+    static lintComponent(code: string, componentName: string, componentSpec?: ComponentSpec, isRootComponent?: boolean): Promise<LintResult>;
     private static validateDataRequirements;
     private static getFunctionName;
     private static deduplicateViolations;

package/dist/lib/component-linter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"component-linter.d.ts","sourceRoot":"","sources":["../../src/lib/component-linter.ts"],"names":[],"mappings":"AAIA,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,UAAU,EAAE,SAAS,EAAE,CAAC;IACxB,WAAW,EAAE,aAAa,EAAE,CAAC;IAC7B,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,SAAS;IACxB,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,UAAU,GAAG,MAAM,GAAG,QAAQ,GAAG,KAAK,CAAC;IACjD,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,aAAa;IAC5B,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAaD,qBAAa,eAAe;IAE1B,OAAO,CAAC,MAAM,CAAC,uBAAuB,CAuyCpC;WAEkB,aAAa,CAC/B,IAAI,EAAE,MAAM,EACZ,aAAa,EAAE,MAAM,EACrB,aAAa,CAAC,EAAE,GAAG,EACnB,eAAe,CAAC,EAAE,OAAO,GACxB,OAAO,CAAC,UAAU,CAAC;IAuEtB,OAAO,CAAC,MAAM,CAAC,wBAAwB;IAgQvC,OAAO,CAAC,MAAM,CAAC,eAAe;IA2B9B,OAAO,CAAC,MAAM,CAAC,qBAAqB;IAyBpC,OAAO,CAAC,MAAM,CAAC,sBAAsB;CA6pBtC"}
+{"version":3,"file":"component-linter.d.ts","sourceRoot":"","sources":["../../src/lib/component-linter.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,aAAa,EAAE,MAAM,6CAA6C,CAAC;AAE5E,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,UAAU,EAAE,SAAS,EAAE,CAAC;IACxB,WAAW,EAAE,aAAa,EAAE,CAAC;IAC7B,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,SAAS;IACxB,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,UAAU,GAAG,MAAM,GAAG,QAAQ,GAAG,KAAK,CAAC;IACjD,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,aAAa;IAC5B,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAaD,qBAAa,eAAe;IAE1B,OAAO,CAAC,MAAM,CAAC,uBAAuB,CA2pCpC;WAEkB,aAAa,CAC/B,IAAI,EAAE,MAAM,EACZ,aAAa,EAAE,MAAM,EACrB,aAAa,CAAC,EAAE,aAAa,EAC7B,eAAe,CAAC,EAAE,OAAO,GACxB,OAAO,CAAC,UAAU,CAAC;IAuEtB,OAAO,CAAC,MAAM,CAAC,wBAAwB;IAgQvC,OAAO,CAAC,MAAM,CAAC,eAAe;IA2B9B,OAAO,CAAC,MAAM,CAAC,qBAAqB;IAyBpC,OAAO,CAAC,MAAM,CAAC,sBAAsB;CA6pBtC"}