Changeset 58077

Timestamp:

05/02/2024 02:40:30 PM (13 months ago)

Author:

swissspidy

Message:

Build/Test Tools: Fix performance tests logging script after [58076].

Removes some unintended debug cruft, whoops!

See #59900.

File:

• trunk/tests/performance/log-results.js (modified) (4 diffs)

trunk/tests/performance/log-results.js

@@ -32 +32 @@
 const afterStats = parseFile( 'performance-results.json' );
 
+if ( ! afterStats.length ) {
+    console.error( 'No results file found' );
+    process.exit( 1 );
+}
+
 /**
  * @type {Array<{file: string, title: string, results: Record<string,number[]>[]}>}
@@ -37 +42 @@
 const baseStats = parseFile( 'base-performance-results.json' );
 
+if ( ! baseStats.length ) {
+    console.error( 'No base results file found' );
+    process.exit( 1 );
+}
+
 /**
  * @type {Record<string, number>}
  */
 const metrics = {};
+
 /**
  * @type {Record<string, number>}
@@ -66 +77 @@
 }
 
-process.exit( 0 );
-
-/**
- * Gets the array of metrics from a list of results.
- *
- * @param {Object[]} results A list of results to format.
- * @return {Object} Metrics.
- */
-const formatResults = ( results ) => {
-    return results.reduce( ( result, { key, file } ) => {
-        return {
-            ...result,
-            ...Object.fromEntries(
-                Object.entries( parseFile( file ) ?? {} ).map(
-                    ( [ metric, value ] ) => [
-                        key + '-' + metric,
-                        median( value ),
-                    ]
-                )
-            ),
-        };
-    }, {} );
-};
-
 const data = new TextEncoder().encode(
     JSON.stringify( {
@@ -96 +83 @@
         baseHash,
         timestamp: parseInt( timestamp, 10 ),
-        metrics: formatResults( testResults ),
-        baseMetrics: formatResults( baseResults ),
+        metrics: metrics,
+        baseMetrics: baseMetrics,
     } )
 );