Merge pull request #2257 from demergent-labs/battle_testing

Make benchmarks work again
demergent-labs · Nov 1, 2024 · 67e8fab · 67e8fab
2 parents c788c8c + 24938a9
commit 67e8fab
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 13 deletions.
diff --git a/src/build/experimental/commands/compile/get_context.ts b/src/build/experimental/commands/compile/get_context.ts
@@ -31,12 +31,7 @@ export async function getContext(
         ...stableContext.wasmData,
         consumer,
         managementDid,
-        recordBenchmarks:
-            process.env.npm_lifecycle_event === 'pre_tests' ||
-            process.env.npm_lifecycle_event === 'pretest' ||
-            process.env.npm_lifecycle_event === 'test'
-                ? process.env.AZLE_RECORD_BENCHMARKS !== 'false'
-                : process.env.AZLE_RECORD_BENCHMARKS === 'true'
+        recordBenchmarks: process.env.AZLE_RECORD_BENCHMARKS === 'true'
     };
 
     return {

diff --git a/src/build/stable/commands/compile/get_context.ts b/src/build/stable/commands/compile/get_context.ts
@@ -28,12 +28,7 @@ export function getContext(
     const wasmData: WasmData = {
         envVars,
         mainJsPath: join(canisterPath, `main.js`),
-        recordBenchmarks:
-            process.env.npm_lifecycle_event === 'pre_tests' ||
-            process.env.npm_lifecycle_event === 'pretest' ||
-            process.env.npm_lifecycle_event === 'test'
-                ? process.env.AZLE_RECORD_BENCHMARKS !== 'false'
-                : process.env.AZLE_RECORD_BENCHMARKS === 'true'
+        recordBenchmarks: process.env.AZLE_RECORD_BENCHMARKS === 'true'
     };
 
     return {

diff --git a/test/index.ts b/test/index.ts
@@ -137,7 +137,7 @@ function processEnvVars(): {
 } {
     const runTests = process.env.AZLE_RUN_TESTS ?? 'true';
     const runTypeChecks = process.env.AZLE_RUN_TYPE_CHECKS ?? 'true';
-    const recordBenchmarks = process.env.AZLE_RECORD_BENCHMARKS ?? 'true';
+    const recordBenchmarks = process.env.AZLE_RECORD_BENCHMARKS ?? 'false';
 
     const hasOnly = [runTests, runTypeChecks].includes('only');