test: run V8 Fast API tests in release mode too

Only keep the call count assertions under `common.isDebug`. PR-URL: #54570 Reviewed-By: Robert Nagy <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Sep 1, 2024 · 522d5a3 · 522d5a3
1 parent edbecf5
commit 522d5a3
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 27 deletions.
diff --git a/doc/contributing/adding-v8-fast-api.md b/doc/contributing/adding-v8-fast-api.md
@@ -173,25 +173,24 @@ A typical function that communicates between JavaScript and C++ is as follows.
   // We could also require a function that uses the internal binding internally.
   const { divide } = internalBinding('custom_namespace');
 
-  if (common.isDebug) {
-    const { getV8FastApiCallCount } = internalBinding('debug');
-
-    // The function that will be optimized. It has to be a function written in
-    // JavaScript. Since `divide` comes from the C++ side, we need to wrap it.
-    function testFastPath(a, b) {
-      return divide(a, b);
-    }
+  // The function that will be optimized. It has to be a function written in
+  // JavaScript. Since `divide` comes from the C++ side, we need to wrap it.
+  function testFastPath(a, b) {
+    return divide(a, b);
+  }
 
-    eval('%PrepareFunctionForOptimization(testFastPath)');
-    // This call will let V8 know about the argument types that the function expects.
-    assert.strictEqual(testFastPath(6, 3), 2);
+  eval('%PrepareFunctionForOptimization(testFastPath)');
+  // This call will let V8 know about the argument types that the function expects.
+  assert.strictEqual(testFastPath(6, 3), 2);
 
-    eval('%OptimizeFunctionOnNextCall(testFastPath)');
-    assert.strictEqual(testFastPath(8, 2), 4);
-    assert.throws(() => testFastPath(1, 0), {
-      code: 'ERR_INVALID_STATE',
-    });
+  eval('%OptimizeFunctionOnNextCall(testFastPath)');
+  assert.strictEqual(testFastPath(8, 2), 4);
+  assert.throws(() => testFastPath(1, 0), {
+    code: 'ERR_INVALID_STATE',
+  });
 
+  if (common.isDebug) {
+    const { getV8FastApiCallCount } = internalBinding('debug');
     assert.strictEqual(getV8FastApiCallCount('custom_namespace.divide.ok'), 1);
     assert.strictEqual(getV8FastApiCallCount('custom_namespace.divide.error'), 1);
   }

diff --git a/test/parallel/test-whatwg-url-canparse.js b/test/parallel/test-whatwg-url-canparse.js
@@ -19,9 +19,8 @@ assert.throws(() => {
 // It should not throw when called without a base string
 assert.strictEqual(URL.canParse('https://example.org'), true);
 
-if (common.isDebug) {
-  const { getV8FastApiCallCount } = internalBinding('debug');
-
+{
+  // V8 Fast API
   function testFastPaths() {
     // `canParse` binding has two overloads.
     assert.strictEqual(URL.canParse('https://www.example.com/path/?query=param#hash'), true);
@@ -33,6 +32,9 @@ if (common.isDebug) {
   eval('%OptimizeFunctionOnNextCall(URL.canParse)');
   testFastPaths();
 
-  assert.strictEqual(getV8FastApiCallCount('url.canParse'), 1);
-  assert.strictEqual(getV8FastApiCallCount('url.canParse.withBase'), 1);
+  if (common.isDebug) {
+    const { getV8FastApiCallCount } = internalBinding('debug');
+    assert.strictEqual(getV8FastApiCallCount('url.canParse'), 1);
+    assert.strictEqual(getV8FastApiCallCount('url.canParse.withBase'), 1);
+  }
 }
diff --git a/test/sequential/test-crypto-timing-safe-equal.js b/test/sequential/test-crypto-timing-safe-equal.js
@@ -93,10 +93,8 @@ assert.throws(
   }
 );
 
-if (common.isDebug) {
-  const { internalBinding } = require('internal/test/binding');
-  const { getV8FastApiCallCount } = internalBinding('debug');
-
+{
+  // V8 Fast API
   const foo = Buffer.from('foo');
   const bar = Buffer.from('bar');
   const longer = Buffer.from('longer');
@@ -111,6 +109,11 @@ if (common.isDebug) {
   assert.throws(() => testFastPath(foo, longer), {
     code: 'ERR_CRYPTO_TIMING_SAFE_EQUAL_LENGTH',
   });
-  assert.strictEqual(getV8FastApiCallCount('crypto.timingSafeEqual.ok'), 2);
-  assert.strictEqual(getV8FastApiCallCount('crypto.timingSafeEqual.error'), 1);
+
+  if (common.isDebug) {
+    const { internalBinding } = require('internal/test/binding');
+    const { getV8FastApiCallCount } = internalBinding('debug');
+    assert.strictEqual(getV8FastApiCallCount('crypto.timingSafeEqual.ok'), 2);
+    assert.strictEqual(getV8FastApiCallCount('crypto.timingSafeEqual.error'), 1);
+  }
 }