Skip to content
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

CDRIVER-4799 sync Index Management tests #1493

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

kevinAlbs
Copy link
Collaborator

@kevinAlbs kevinAlbs commented Dec 19, 2023

Summary

Sync tests from mongodb/specifications#1485

Background & Motivation

SERVER-83290 changed an error message causing test assertions to fail latest server builds. For example asan-sasl-cyrus-openssl-ubuntu2004-clang-test-latest-replica-auth failed with:

error: expected error to contain "Search index commands are only supported with Atlas", but got: "Using Atlas Search Database Commands and the $listSearchIndexes aggregation stage requires additional configuration. Please connect to Atlas or an AtlasCLI local deployment to enable. For more information on how to connect, see https://dochub.mongodb.org/core/atlas-cli-deploy-local-reqs."

The message assertions were updated in mongodb/specifications#1485

Tests were run in C driver to verify they are now passing: https://spruce.mongodb.com/version/6581c5d12a60ed6df70bbe25

@kevinAlbs kevinAlbs changed the title sync tests from DRIVERS-2797 DRIVERS-2797 sync Index Management tests Dec 19, 2023
@kevinAlbs kevinAlbs changed the title DRIVERS-2797 sync Index Management tests CDRIVER-4799 sync Index Management tests Dec 19, 2023
@kevinAlbs kevinAlbs marked this pull request as ready for review December 19, 2023 19:46
Copy link
Collaborator

@rcsanchez97 rcsanchez97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kevinAlbs kevinAlbs merged commit 5e80cca into mongodb:master Dec 20, 2023
31 of 33 checks passed
kevinAlbs added a commit that referenced this pull request Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants