feat(NODE-7311): Replace process.platform with os.platform() (#4822)

Author: PavelSafronov

src/cmap/auth/gssapi.ts

@@ -1,4 +1,5 @@
 import * as dns from 'dns';
+import * as os from 'os';
 
 import { getKerberos, type Kerberos, type KerberosClient } from '../../deps';
 import { MongoInvalidArgumentError, MongoMissingCredentialsError } from '../../error';
@@ -97,7 +98,7 @@ async function makeKerberosClient(authContext: AuthContext): Promise<KerberosCli
   }
 
   const spnHost = mechanismProperties.SERVICE_HOST ?? host;
-  let spn = `${serviceName}${process.platform === 'win32' ? '/' : '@'}${spnHost}`;
+  let spn = `${serviceName}${os.platform() === 'win32' ? '/' : '@'}${spnHost}`;
   if ('SERVICE_REALM' in mechanismProperties) {
     spn = `${spn}@${mechanismProperties.SERVICE_REALM}`;
   }

src/cmap/handshake/client_metadata.ts

@@ -157,7 +157,7 @@ export async function makeClientMetadata(
 
   // Note: order matters, os.type is last so it will be removed last if we're at maxSize
   const osInfo = new Map()
-    .set('name', process.platform)
+    .set('name', os.platform())
     .set('architecture', os.arch())
     .set('version', os.release())
     .set('type', os.type());

test/benchmarks/driver_bench/src/main.mts

@@ -71,7 +71,7 @@ const systemInfo = () =>
     `\n- cpu: ${platform.name}`,
     `- cores: ${platform.cores}`,
     `- arch: ${os.arch()}`,
-    `- os: ${process.platform} (${os.release()})`,
+    `- os: ${os.platform()} (${os.release()})`,
     `- ram: ${platform.ram}`,
     `- node: ${process.version}`,
     `- running ${total} benchmarks`,

test/integration/node-specific/examples/change_streams.test.js

@@ -4,9 +4,10 @@
 const { setTimeout } = require('timers');
 const setupDatabase = require('../../shared').setupDatabase;
 const expect = require('chai').expect;
+const os = require('os');
 
 // TODO: NODE-3819: Unskip flaky MacOS/Windows tests.
-const maybeDescribe = process.platform !== 'linux' ? describe.skip : describe;
+const maybeDescribe = os.platform() !== 'linux' ? describe.skip : describe;
 maybeDescribe('examples(change-stream):', function () {
   let client;
   let db;

test/integration/node-specific/ipv6.test.ts

@@ -1,6 +1,6 @@
 import { expect } from 'chai';
 import * as net from 'net';
-import * as process from 'process';
+import * as os from 'os';
 import * as sinon from 'sinon';
 
 import {
@@ -16,7 +16,7 @@ describe('IPv6 Addresses', () => {
 
   beforeEach(async function () {
     if (
-      process.platform === 'linux' ||
+      os.platform() === 'linux' ||
       this.configuration.topologyType !== TopologyType.ReplicaSetWithPrimary
     ) {
       if (this.currentTest) {

test/integration/uri-options/uri.test.ts

@@ -1,4 +1,5 @@
 import { expect } from 'chai';
+import * as os from 'os';
 import * as sinon from 'sinon';
 
 import { Topology } from '../../../src/sdam/topology';
@@ -45,7 +46,7 @@ describe('URI', function () {
     metadata: { requires: { topology: 'single' } },
 
     test: async function () {
-      if (process.platform === 'win32') {
+      if (os.platform() === 'win32') {
         return;
       }
 

test/manual/kerberos.test.ts

@@ -1,5 +1,6 @@
 import * as chai from 'chai';
 import { promises as dns } from 'dns';
+import * as os from 'os';
 import * as sinon from 'sinon';
 
 import { MongoClient } from '../../src';
@@ -50,7 +51,7 @@ describe('Kerberos', function () {
 
   context('when passing in CANONICALIZE_HOST_NAME', function () {
     beforeEach(function () {
-      if (process.platform === 'darwin') {
+      if (os.platform() === 'darwin') {
         this.currentTest.skipReason =
           'DNS does not resolve with proper CNAME record on evergreen MacOS';
         this.skip();

test/manual/tls_support.test.ts

@@ -4,6 +4,7 @@ import * as tls from 'node:tls';
 import { expect } from 'chai';
 import { promises as fs } from 'fs';
 import ConnectionString from 'mongodb-connection-string-url';
+import * as os from 'os';
 import * as sinon from 'sinon';
 
 import { MongoClient, type MongoClientOptions, MongoServerSelectionError } from '../../src';
@@ -36,7 +37,7 @@ describe('TLS Support', function () {
   beforeEach(function () {
     if (
       this.currentTest?.title === 'should connect with tls via url options' &&
-      process.platform === 'win32'
+      os.platform() === 'win32'
     ) {
       this.currentTest.skipReason = 'TODO(NODE-5803): Un-skip Windows TLS tests via URL';
       return this.skip();

test/tools/runner/filters/client_encryption_filter.ts

@@ -1,4 +1,5 @@
 import { readFile } from 'fs/promises';
+import * as os from 'os';
 import { dirname, resolve } from 'path';
 import * as process from 'process';
 import { satisfies } from 'semver';
@@ -74,7 +75,7 @@ export class ClientSideEncryptionFilter extends Filter {
     }
 
     // TODO(NODE-3401): unskip csfle tests on windows
-    if (process.env.TEST_CSFLE && process.platform !== 'win32') {
+    if (process.env.TEST_CSFLE && os.platform() !== 'win32') {
       if (this.version == null) {
         throw new Error('FLE tests must run, but mongodb client encryption was not installed.');
       }

test/tools/runner/filters/os_filter.ts

@@ -1,3 +1,5 @@
+import * as os from 'os';
+
 import { Filter } from './filter';
 
 /**
@@ -17,7 +19,7 @@ export class OSFilter extends Filter {
   constructor() {
     super();
     // Get environmental variables that are known
-    this.platform = process.platform;
+    this.platform = os.platform();
   }
 
   filter(test: { metadata?: MongoDBMetadataUI }) {