-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test(NODE-3713): skip/fix failing evergreen tests #3069
Conversation
ac34bd4
to
0af06ca
Compare
5ecc331
to
eb0351d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went ahead and marked the lines where we need unskip tickets (don't necessarily need a separate ticket for each one, but just something that's actually in our backlog in the Testing Improvements epic to address in the future)
@@ -71,7 +71,7 @@ describe('Kerberos', function () { | |||
client.connect(function (err) { | |||
expect(err).to.exist; | |||
expect(err.message).to.match( | |||
/(Error from KDC: LOOKING_UP_SERVER)|(not found in Kerberos database)/ | |||
/(Error from KDC: LOOKING_UP_SERVER)|(not found in Kerberos database)|(UNKNOWN_SERVER)/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is interesting - just curious how/when/why this manifested?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It started happening last week and only on MacOS. I'm trying to pin down the root cause at the moment but Kerberos knowledge is obviously not one of my strengths.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the tests that link to a MacOS skip, let's just skip them conditionally on the OS instead of always skipping
This has been added now. |
315da13
to
23f9155
Compare
This reverts commit 5fc39af.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just wanted to double check about these 3 tests:
Description
Skips or fixes failing non Ubuntu evergreen tests
What is changing?
Skips failing tests on MacOS and Windows plus increases test run timeouts.
Notes
The failing client side encryption corpus tests I believe will be fixed by mongodb/libmongocrypt#225
KDC error messages updated for failing Kerberos tests.
Documentation Changes
No
What is the motivation for this change?
NODE-3713
Double check the following
npm run check:lint
script<type>(NODE-xxxx)<!>: <description>
https://spruce.mongodb.com/version/61bc8d837742ae2cc78c16e9/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC