From 0066417c5061a8e8d70ddee822d78d8a41f15b9e Mon Sep 17 00:00:00 2001 From: Bailey Pearson Date: Fri, 31 Mar 2023 10:12:10 -0400 Subject: [PATCH] fix tests --- .../connection.test.ts | 27 +++++++++++-------- .../node-specific/topology.test.js | 5 +++- 2 files changed, 20 insertions(+), 12 deletions(-) diff --git a/test/integration/connection-monitoring-and-pooling/connection.test.ts b/test/integration/connection-monitoring-and-pooling/connection.test.ts index b13ee006b33..4ee973ea00b 100644 --- a/test/integration/connection-monitoring-and-pooling/connection.test.ts +++ b/test/integration/connection-monitoring-and-pooling/connection.test.ts @@ -3,7 +3,9 @@ import { expect } from 'chai'; import { connect, Connection, + ConnectionOptions, LEGACY_HELLO_COMMAND, + makeClientMetadata, MongoClient, MongoServerError, ns, @@ -31,12 +33,13 @@ describe('Connection', function () { it('should execute a command against a server', { metadata: { requires: { apiVersion: false, topology: '!load-balanced' } }, test: function (done) { - const connectOptions = Object.assign( - { connectionType: Connection }, - this.configuration.options - ); + const connectOptions: Partial = { + connectionType: Connection, + ...this.configuration.options, + metadata: makeClientMetadata({ driverInfo: {} }) + }; - connect(connectOptions, (err, conn) => { + connect(connectOptions as any as ConnectionOptions, (err, conn) => { expect(err).to.not.exist; this.defer(_done => conn.destroy(_done)); @@ -53,12 +56,14 @@ describe('Connection', function () { it('should emit command monitoring events', { metadata: { requires: { apiVersion: false, topology: '!load-balanced' } }, test: function (done) { - const connectOptions = Object.assign( - { connectionType: Connection, monitorCommands: true }, - this.configuration.options - ); - - connect(connectOptions, (err, conn) => { + const connectOptions: Partial = { + connectionType: Connection, + monitorCommands: true, + ...this.configuration.options, + metadata: makeClientMetadata({ driverInfo: {} }) + }; + + connect(connectOptions as any as ConnectionOptions, (err, conn) => { expect(err).to.not.exist; this.defer(_done => conn.destroy(_done)); diff --git a/test/integration/node-specific/topology.test.js b/test/integration/node-specific/topology.test.js index 912c1443c49..ee4393efcd6 100644 --- a/test/integration/node-specific/topology.test.js +++ b/test/integration/node-specific/topology.test.js @@ -1,11 +1,14 @@ 'use strict'; const { expect } = require('chai'); +const { makeClientMetadata } = require('../../mongodb'); describe('Topology', function () { it('should correctly track states of a topology', { metadata: { requires: { apiVersion: false, topology: '!load-balanced' } }, // apiVersion not supported by newTopology() test: function (done) { - const topology = this.configuration.newTopology(); + const topology = this.configuration.newTopology({ + metadata: makeClientMetadata({ driverInfo: {} }) + }); const states = []; topology.on('stateChanged', (_, newState) => states.push(newState));