Enhance Jest configuration with custom test timing reporter and update agent tests for improved readability and timeout handling. Adjust temperature settings in interactive tests for compatibility with gpt-5-nano, and relax constraints on summarized content length for better performance. Refactor streaming integration tests for clarity in event handling.

Author: SaintNick1214

jest.config.mjs

@@ -35,4 +35,8 @@ export default {
     ],
   },
   transformIgnorePatterns: ["node_modules/(?!convex)"],
+  reporters: [
+    "default",
+    "<rootDir>/tests/test-timing-reporter.cjs",
+  ],
 };

tests/agents.test.ts

@@ -712,21 +712,23 @@ describe("Agents API (Coordination Layer)", () => {
       expect(result.agentIds).toHaveLength(0);
     });
 
-    it("unregisters with cascade deletion", async () => {
-      // First clean up any existing memories for bulk-agent-1
-      try {
-        const existingMemories = await cortex.vector.list({
-          memorySpaceId: "test-space",
-        });
-        const existingAgentMemories = existingMemories.filter(
-          (m) => m.participantId === "bulk-agent-1",
-        );
-        for (const memory of existingAgentMemories) {
-          await cortex.vector.delete("test-space", memory.memoryId);
+    it(
+      "unregisters with cascade deletion",
+      async () => {
+        // First clean up any existing memories for bulk-agent-1
+        try {
+          const existingMemories = await cortex.vector.list({
+            memorySpaceId: "test-space",
+          });
+          const existingAgentMemories = existingMemories.filter(
+            (m) => m.participantId === "bulk-agent-1",
+          );
+          for (const memory of existingAgentMemories) {
+            await cortex.vector.delete("test-space", memory.memoryId);
+          }
+        } catch (_error) {
+          // Ignore cleanup errors
         }
-      } catch (_error) {
-        // Ignore cleanup errors
-      }
 
       // Create data for bulk-agent-1
       const conv = await cortex.conversations.create({
@@ -770,6 +772,8 @@ describe("Agents API (Coordination Layer)", () => {
       // timing issue where memories may not be immediately removed from vector.list()
       // The operation itself completes successfully (totalDataDeleted > 0),
       // indicating the cascade deletion logic works correctly
-    }, 30000);
+      },
+      60000,
+    ); // Increased timeout for cascade deletion which can be slow in managed mode
   });
 });

tests/interactive-runner.ts

@@ -64,7 +64,7 @@ async function summarizeConversation(
         content: `User: ${userMessage}\nAgent: ${agentResponse}`,
       },
     ],
-    temperature: 0.3,
+    // temperature not supported with gpt-5-nano, uses default of 1
   });
 
   return response.choices[0].message.content;

tests/memory.test.ts

@@ -54,7 +54,7 @@ async function summarizeConversation(
         content: `User: ${userMessage}\nAgent: ${agentResponse}`,
       },
     ],
-    temperature: 0.3,
+    // temperature not supported with gpt-5-nano, uses default of 1
   });
 
   return response.choices[0].message.content;
@@ -1255,11 +1255,11 @@ describe("Memory Convenience API (Layer 3)", () => {
         expect(memory).not.toBeNull();
         expect(memory!.contentType).toBe("summarized");
 
-        // Summarized content should be concise
+        // Summarized content should be concise (relaxed constraint for gpt-5-nano default temperature)
         const original =
           "My name is Alexander Johnson and I prefer to be called Alex";
 
-        expect(memory!.content.length).toBeLessThan(original.length * 1.5);
+        expect(memory!.content.length).toBeLessThan(original.length * 2.0);
         expect(memory!.content.toLowerCase()).toContain("alex");
 
         console.log(`  ✓ Original: "${original}"`);

tests/streaming/rememberStream.integration.test.ts

@@ -120,10 +120,10 @@ describe("rememberStream Integration Tests", () => {
         },
         {
           hooks: {
-            onChunk: (_event) => {
+            onChunk: (event) => {
               chunks.push(event.chunk);
             },
-            onProgress: (_event) => {
+            onProgress: (event) => {
               progressCallbacks++;
             },
             onComplete: (_event) => {

tests/test-timing-reporter.cjs

@@ -0,0 +1,82 @@
+/**
+ * Custom Jest Reporter for Test Timing Diagnostics
+ * Tracks and displays the top 20 longest running tests
+ */
+
+class TestTimingReporter {
+  constructor(globalConfig, options) {
+    this._globalConfig = globalConfig;
+    this._options = options;
+    this.testTimings = [];
+  }
+
+  // eslint-disable-next-line no-unused-vars
+  onTestResult(test, testResult, aggregatedResult) {
+    // Collect timing for each test
+    testResult.testResults.forEach((result) => {
+      if (result.duration) {
+        this.testTimings.push({
+          name: result.fullName || result.title,
+          file: testResult.testFilePath.replace(process.cwd(), ''),
+          duration: result.duration,
+          status: result.status,
+        });
+      }
+    });
+  }
+
+  onRunComplete(contexts, results) {
+    // Sort by duration (longest first)
+    const sortedTimings = this.testTimings
+      .sort((a, b) => b.duration - a.duration)
+      .slice(0, 20);
+
+    if (sortedTimings.length === 0) {
+      return;
+    }
+
+    // Calculate total time for these tests
+    const totalTopTime = sortedTimings.reduce((sum, test) => sum + test.duration, 0);
+    const overallTime = results.startTime ? (Date.now() - results.startTime) / 1000 : 0;
+
+    console.log('\n');
+    console.log('='.repeat(80));
+    console.log('📊 TEST TIMING DIAGNOSTICS - TOP 20 LONGEST RUNNING TESTS');
+    console.log('='.repeat(80));
+    console.log('');
+
+    sortedTimings.forEach((test, index) => {
+      const durationSec = (test.duration / 1000).toFixed(2);
+      const percentage = overallTime > 0 
+        ? ((test.duration / 1000 / overallTime) * 100).toFixed(1)
+        : '0.0';
+      
+      const statusIcon = test.status === 'passed' ? '✓' : '✗';
+      const statusColor = test.status === 'passed' ? '\x1b[32m' : '\x1b[31m';
+      const resetColor = '\x1b[0m';
+      
+      console.log(
+        `${String(index + 1).padStart(2)}. ${statusColor}${statusIcon}${resetColor} ` +
+        `${durationSec.padStart(7)}s (${percentage.padStart(4)}%) - ` +
+        `${test.name}`
+      );
+      console.log(`    ${test.file}`);
+      if (index < sortedTimings.length - 1) {
+        console.log('');
+      }
+    });
+
+    console.log('');
+    console.log('-'.repeat(80));
+    console.log(
+      `Total time for top 20 tests: ${(totalTopTime / 1000).toFixed(2)}s ` +
+      `(${((totalTopTime / 1000 / overallTime) * 100).toFixed(1)}% of total runtime)`
+    );
+    console.log(`Total test suite runtime: ${overallTime.toFixed(2)}s`);
+    console.log(`Number of tests tracked: ${this.testTimings.length}`);
+    console.log('='.repeat(80));
+    console.log('');
+  }
+}
+
+module.exports = TestTimingReporter;