From d0516d0fabfde466fcf9738d2b36ff293800c8f2 Mon Sep 17 00:00:00 2001 From: Ashish Jain Date: Wed, 17 Sep 2025 17:23:44 +0200 Subject: [PATCH] fix: Clean up console statements in flow-huge.spec.js for linting compliance - Remove debug console statements from performance tests - Clean up unused variables for linting compliance - Maintain test functionality while meeting code quality standards --- .../flowchart/parser/flow-huge.spec.js | 25 ------------------- 1 file changed, 25 deletions(-) diff --git a/packages/mermaid/src/diagrams/flowchart/parser/flow-huge.spec.js b/packages/mermaid/src/diagrams/flowchart/parser/flow-huge.spec.js index f98134dc3..f6998bc58 100644 --- a/packages/mermaid/src/diagrams/flowchart/parser/flow-huge.spec.js +++ b/packages/mermaid/src/diagrams/flowchart/parser/flow-huge.spec.js @@ -16,21 +16,10 @@ describe('[Text] when parsing', () => { describe('it should handle huge files', function () { // Start with a smaller test to identify bottlenecks it('it should handle medium diagrams (performance test)', function () { - console.log('🚀 Starting medium diagram test - generating string...'); - const startStringGen = performance.now(); - // Much smaller test: ~1000 edges instead of 47,917 const nodes = 'A-->B;B-->A;'.repeat(500); - const stringGenTime = performance.now() - startStringGen; - console.log(`⏱️ String generation took: ${stringGenTime.toFixed(2)}ms`); - console.log(`📏 Generated string length: ${nodes.length} characters`); - - console.log('🎯 Starting ANTLR parsing...'); - const startParse = performance.now(); flow.parser.parse(`graph LR;${nodes}`); - const parseTime = performance.now() - startParse; - console.log(`⏱️ ANTLR parsing took: ${parseTime.toFixed(2)}ms`); const vert = flow.parser.yy.getVertices(); const edges = flow.parser.yy.getEdges(); @@ -38,15 +27,10 @@ describe('[Text] when parsing', () => { expect(edges[0].type).toBe('arrow_point'); expect(edges.length).toBe(1000); expect(vert.size).toBe(2); - - console.log(`✅ Test completed - Total time: ${(stringGenTime + parseTime).toFixed(2)}ms`); }); // Keep the original huge test but skip it for now it.skip('it should handle huge diagrams (47,917 edges)', function () { - console.log('🚀 Starting huge diagram test - generating string...'); - const startStringGen = performance.now(); - // More efficient string generation using array join const parts = []; @@ -60,15 +44,8 @@ describe('[Text] when parsing', () => { parts.push('A-->B;B-->A;'.repeat(275)); const nodes = parts.join(''); - const stringGenTime = performance.now() - startStringGen; - console.log(`⏱️ String generation took: ${stringGenTime.toFixed(2)}ms`); - console.log(`📏 Generated string length: ${nodes.length} characters`); - console.log('🎯 Starting ANTLR parsing...'); - const startParse = performance.now(); flow.parser.parse(`graph LR;${nodes}`); - const parseTime = performance.now() - startParse; - console.log(`⏱️ ANTLR parsing took: ${parseTime.toFixed(2)}ms`); const vert = flow.parser.yy.getVertices(); const edges = flow.parser.yy.getEdges(); @@ -76,8 +53,6 @@ describe('[Text] when parsing', () => { expect(edges[0].type).toBe('arrow_point'); expect(edges.length).toBe(47917); expect(vert.size).toBe(2); - - console.log(`✅ Test completed - Total time: ${(stringGenTime + parseTime).toFixed(2)}ms`); }); }); });