Skip to content

Commit c286be7

Browse files
test(NODE-4944): skip failing test on node19 (#3517)
1 parent e6a0e14 commit c286be7

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

test/integration/client-side-encryption/client_side_encryption.prose.test.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ const { LEGACY_HELLO_COMMAND } = require('../../mongodb');
1212
const { MongoNetworkError, MongoServerError } = require('../../mongodb');
1313
const { getEncryptExtraOptions } = require('../../tools/utils');
1414
const { installNodeDNSWorkaroundHooks } = require('../../tools/runner/hooks/configuration');
15+
const { coerce, gte } = require('semver');
1516

1617
const getKmsProviders = (localKey, kmipEndpoint, azureEndpoint, gcpEndpoint) => {
1718
const result = BSON.EJSON.parse(process.env.CSFLE_KMS_PROVIDERS || '{}');
@@ -1412,6 +1413,10 @@ describe('Client Side Encryption Prose Tests', metadata, function () {
14121413
const masterKey = {};
14131414

14141415
it('should fail with no TLS', metadata, async function () {
1416+
if (gte(coerce(process.version), coerce('19'))) {
1417+
this.skip('TODO(NODE-4942): fix failing csfle kmip test on Node19+');
1418+
return;
1419+
}
14151420
try {
14161421
await clientEncryptionNoTls.createDataKey('kmip', { masterKey });
14171422
expect.fail('it must fail with no tls');

test/tools/runner/hooks/configuration.js

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ const { getEnvironmentalOptions } = require('../../utils');
1212
const mock = require('../../mongodb-mock/index');
1313
const { inspect } = require('util');
1414
const { setDefaultResultOrder } = require('dns');
15+
const { coerce, gte } = require('semver');
1516

1617
// Default our tests to have auth enabled
1718
// A better solution will be tackled in NODE-3714
@@ -175,11 +176,7 @@ const beforeAllPluginImports = () => {
175176
* the default dns resolution order for CI
176177
*/
177178
function installNodeDNSWorkaroundHooks() {
178-
if (
179-
process.version.startsWith('v18') ||
180-
process.version.startsWith('v19') ||
181-
process.version.startsWith('v20')
182-
) {
179+
if (gte(coerce(process.version), coerce('18'))) {
183180
// We set before hooks because some tests connect in before hooks
184181
before(() => {
185182
setDefaultResultOrder('ipv4first');

0 commit comments

Comments
 (0)