mirrors/mongodb
mirrors/mongodb
0
0
Fork 0
mirror of https://github.com/mongodb/mongo.git synced 2024-12-01 09:32:32 +01:00
Code Issues Releases Activity

SERVER-50287 drop_index.js fails when run in passthrough suite with stepdown

Browse Source
This commit is contained in:
Yuhong Zhang 2021-07-01 18:49:22 +00:00 committed by Evergreen Agent
parent 63a7a4d461
commit d8170c86a9
2 changed files with 67 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
jstests/core/drop_index.js
View File

@ -70,31 +70,7 @@ for (const dropIndexArg of ['_id_', {_id: 1}]) {
assert.commandWorked(t.createIndex({a: 1}));
assertIndexes(['a_1', 'c_1', 'd_1', 'e_1'], 'recreating {a: 1}');
// Drop multiple indexes.
assert.commandWorked(t.dropIndexes(['c_1', 'd_1']));
assertIndexes(['a_1', 'e_1'], 'dropping {c: 1} and {d: 1}');
// Must drop all the indexes provided or none at all - for example, if one of the index names
// provided is invalid.
let ex = assert.throws(() => {
t.dropIndexes(['a_1', '_id_']);
});
assert.commandFailedWithCode(ex, ErrorCodes.InvalidOptions);
assertIndexes(['a_1', 'e_1'], 'failed dropIndexes command with _id index');
// List of index names must contain only strings.
ex = assert.throws(() => {
t.dropIndexes(['a_1', 123]);
});
assert.commandFailedWithCode(ex, ErrorCodes.TypeMismatch);
assertIndexes(['a_1', 'e_1'], 'failed dropIndexes command with non-string index name');
// Test "deleteIndexes" alias.
assert.commandWorked(t.createIndex({f: 1}));
assert.commandWorked(db.runCommand({deleteIndexes: t.getName(), index: 'f_1'}));
assertIndexes(['a_1', 'e_1'], 'failed deleteIndexes command alias');
// Drop all indexes.
assert.commandWorked(t.dropIndexes());
assertIndexes([], 'dropping all indexes');
// Drop single index with dropIndexes().
assert.commandWorked(t.dropIndexes(['c_1']));
assertIndexes(['a_1', 'd_1', 'e_1'], 'dropping {c: 1}');
}());

64
jstests/core/drop_indexes.js Normal file
View File

@ -0,0 +1,64 @@
// Cannot implicitly shard accessed collections because of extra shard key index in sharded
// collection. Cannot be handled correctly in a stepdown suite since dropIndexes() with multiple
// names cannot be retried properly.
// @tags: [assumes_no_implicit_index_creation, does_not_support_stepdowns]
(function() {
'use strict';
const t = db.drop_indexes;
t.drop();
/**
* Extracts index names from listIndexes result.
*/
function getIndexNames(cmdRes) {
return t.getIndexes().map(spec => spec.name);
}
/**
* Checks that collection contains the given list of non-id indexes and nothing else.
*/
function assertIndexes(expectedIndexNames, msg) {
const actualIndexNames = getIndexNames();
const testMsgSuffix = () => msg + ': expected ' + tojson(expectedIndexNames) + ' but got ' +
tojson(actualIndexNames) + ' instead.';
assert.eq(expectedIndexNames.length + 1,
actualIndexNames.length,
'unexpected number of indexes after ' + testMsgSuffix());
assert(actualIndexNames.includes('_id_'),
'_id index missing after ' + msg + ': ' + tojson(actualIndexNames));
for (let expectedIndexName of expectedIndexNames) {
assert(actualIndexNames.includes(expectedIndexName),
expectedIndexName + ' index missing after ' + testMsgSuffix());
}
}
assert.commandWorked(t.insert({_id: 1, a: 2, b: 3, c: 1, d: 1, e: 1}));
assertIndexes([], 'inserting test document');
assert.commandWorked(t.createIndex({a: 1}));
assert.commandWorked(t.createIndex({b: 1}));
assert.commandWorked(t.createIndex({c: 1}));
assert.commandWorked(t.createIndex({d: 1}));
assert.commandWorked(t.createIndex({e: 1}));
assertIndexes(['a_1', 'b_1', 'c_1', 'd_1', 'e_1'], 'creating indexes');
// Drop multiple indexes.
assert.commandWorked(t.dropIndexes(['c_1', 'd_1']));
assertIndexes(['a_1', 'b_1', 'e_1'], 'dropping {c: 1} and {d: 1}');
// Must drop all the indexes provided or none at all - for example, if one of the index names
// provided is invalid.
let ex = assert.throws(() => {
t.dropIndexes(['a_1', '_id_']);
});
assert.commandFailedWithCode(ex, ErrorCodes.InvalidOptions);
assertIndexes(['a_1', 'b_1', 'e_1'], 'failed dropIndexes command with _id index');
// List of index names must contain only strings.
ex = assert.throws(() => {
t.dropIndexes(['a_1', 123]);
});
assert.commandFailedWithCode(ex, ErrorCodes.TypeMismatch);
assertIndexes(['a_1', 'b_1', 'e_1'], 'failed dropIndexes command with non-string index name');
}());