pr feedback:

- avoid throwing by toUTF8
- clarify comments

Author: PavelSafronov

src/cmap/auth/scram.ts

@@ -167,8 +167,8 @@ async function continueScramConversation(
   const serverKey = await HMAC(cryptoMethod, saltedPassword, 'Server Key');
   const storedKey = await H(cryptoMethod, clientKey);
   const firstMessageBytes = clientFirstMessageBare(username, nonce);
-  const firstMessage = ByteUtils.toUTF8(firstMessageBytes, 0, firstMessageBytes.length, true);
-  const payloadString = ByteUtils.toUTF8(payload.buffer, 0, payload.position, true);
+  const firstMessage = ByteUtils.toUTF8(firstMessageBytes, 0, firstMessageBytes.length, false);
+  const payloadString = ByteUtils.toUTF8(payload.buffer, 0, payload.position, false);
   const authMessage = [firstMessage, payloadString, withoutProof].join(',');
 
   const clientSignature = await HMAC(cryptoMethod, storedKey, authMessage);
@@ -204,7 +204,7 @@ async function continueScramConversation(
 }
 
 function parsePayload(payload: Binary) {
-  const payloadStr = ByteUtils.toUTF8(payload.buffer, 0, payload.position, true);
+  const payloadStr = ByteUtils.toUTF8(payload.buffer, 0, payload.position, false);
   const dict: Document = {};
   const parts = payloadStr.split(',');
   for (let i = 0; i < parts.length; i++) {

test/tools/utils.ts

@@ -627,12 +627,13 @@ export const runtime: Runtime = resolveRuntimeAdapters({});
 
 /**
  * Metadata that can be used to skip tests in nodeless environments.
+ * Use this for tests that rely on Node.js-specific features and cannot be run in nodeless environments like Deno or the browser.
  */
 export const runOnlyInNodeMetadata: MongoDBMetadataUI = {
   requires: {
     predicate: _test => {
       if (runNodelessTests) {
-        return 'Test is not node specific and should be run in nodeless environments';
+        return 'Test is a node specific test and should be not be run in nodeless environments';
       } else {
         return true;
       }