GODRIVER-2701 Implement a variant of DropIndex that gets keys rather than name

[timothy-kim-mongo]

GODRIVER-2701

Summary

The changes in this pull-request answers the ticket GODRIVER-2701 of implements a variant of DropIndex that searches for the index using the key rather than the name of the index. This creates an additional method that allows users to drop index with the freedom of either a name or key.

Background & Motivation

The motivation behind this is to be consistent with the functionality of the MongoDB db.collection.dropIndex() function which allows dropping the index with either a key or name. This is an optional part of the spec - https://github.com/mongodb/specifications/blob/master/source/index-management/index-management.md#standard-api.

[timothy-kim-mongo]

Note: 56850c7 and before have an implementation of DropOne method taking in a generic of either the key or name which could be considered for 2.0.

[mongodb-drivers-pr-bot]

API Change Report

./mongo

compatible changes

IndexView.DropWithKey: added

./x/mongo/driver/operation

incompatible changes

(*DropIndexes).Index: changed from func(string) *DropIndexes to func(any) *DropIndexes
NewDropIndexes: changed from func(string) *DropIndexes to func(any) *DropIndexes

[prestonvasquez]

LGTM!

To fix the failings tests, suggest verifying that the indexes were created before dropping by key. You can accomplish this using the verifyIndexExists helper function:

for _, name := range indexNames {
	verifyIndexExists(mt, iv, index{
		Key:  test.index,
		Name: name,
	})
}

You will also need to type the +/- 1 index values as int32, e.g. 1 -> int32(1). See here for the suggested changes.

[timothy-kim-mongo]

LGTM!

To fix the failings tests, suggest verifying that the indexes were created before dropping by key. You can accomplish this using the verifyIndexExists helper function:

for _, name := range indexNames {
	verifyIndexExists(mt, iv, index{
		Key:  test.index,
		Name: name,
	})
}

You will also need to type the +/- 1 index values as int32, e.g. 1 -> int32(1). See here for the suggested changes.

It seems like the failed tests for the other tests in older version of RHEL are continuing to happen. I will try to investigate this.

[prestonvasquez]

LGTM!