Changeset 58077 for trunk

Timestamp:

05/02/2024 02:40:30 PM (2 years 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,
         } )
 );