Skip to content

Commit d341a72

Browse files
ci: skip flaky tests in FLE test suite (#4467)
1 parent 0083b8e commit d341a72

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

Diff for: test/integration/client-side-encryption/client_side_encryption.prose.test.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1561,15 +1561,15 @@ describe('Client Side Encryption Prose Tests', metadata, function () {
15611561
keyName: 'foo'
15621562
};
15631563

1564-
it('should fail with no TLS', metadata, async function () {
1564+
it.skip('should fail with no TLS', metadata, async function () {
15651565
try {
15661566
await clientEncryptionNoTls.createDataKey('azure', { masterKey });
15671567
expect.fail('it must fail with no tls');
15681568
} catch (e) {
15691569
//Expect an error indicating TLS handshake failed.
15701570
expect(e.cause.message).to.include('certificate required');
15711571
}
1572-
});
1572+
}).skipReason = 'TODO(NODE-6861): fix flaky test';
15731573

15741574
it('should succeed with valid TLS options', metadata, async function () {
15751575
try {

Diff for: test/integration/client-side-encryption/client_side_encryption.spec.test.ts

+8
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,14 @@ describe('Client Side Encryption (Unified)', function () {
149149
if (typeof shouldSkip === 'string') return shouldSkip;
150150
}
151151

152+
const flakyTests = {
153+
'rewrap to azure:name1': 'TODO(NODE-6860): fix flaky tests'
154+
};
155+
156+
const skipReason = flakyTests[description];
157+
158+
if (skipReason) return skipReason;
159+
152160
return isServerless ? 'Unified CSFLE tests to not run on serverless' : false;
153161
}
154162
);

0 commit comments

Comments
 (0)