From 27da922959c12df48ae2ecf8aad0f37f42f71451 Mon Sep 17 00:00:00 2001 From: David Storch Date: Wed, 19 Jul 2023 21:45:30 +0000 Subject: SERVER-78752 Add missing JS test tags --- .../suites/clustered_collection_passthrough.yml | 3 +++ jstests/core/sbe_plan_cache_duplicate_or_clauses.js | 18 +++++++++++++++--- 2 files changed, 18 insertions(+), 3 deletions(-) diff --git a/buildscripts/resmokeconfig/suites/clustered_collection_passthrough.yml b/buildscripts/resmokeconfig/suites/clustered_collection_passthrough.yml index d726ef519d9..cb3abf6cc5c 100644 --- a/buildscripts/resmokeconfig/suites/clustered_collection_passthrough.yml +++ b/buildscripts/resmokeconfig/suites/clustered_collection_passthrough.yml @@ -68,6 +68,9 @@ selector: exclude_with_any_tags: - assumes_standalone_mongod + # Tags tests which assume that the collections they are running against are not implicitly created + # as clustered collections. + - does_not_support_clustered_collections executor: archive: diff --git a/jstests/core/sbe_plan_cache_duplicate_or_clauses.js b/jstests/core/sbe_plan_cache_duplicate_or_clauses.js index f78fa7c6fad..13adc282aac 100644 --- a/jstests/core/sbe_plan_cache_duplicate_or_clauses.js +++ b/jstests/core/sbe_plan_cache_duplicate_or_clauses.js @@ -4,10 +4,22 @@ * dependency between the values of the $or expression * * @tags: [ + * # $planCacheStats requires readConcern:"local". + * assumes_read_concern_unchanged, + * # Plan cache state is node-local so the test assumes it is always reading from the same node. + * assumes_read_preference_unchanged, * assumes_unsharded_collection, + * cqf_incompatible, + * # This test assumes that the SBE plan cache is in use, but SBE does not support clustered + * # collections in 7.0. + * does_not_support_clustered_collections, + * # Plan cache state is node-local, so this test assumes it is always operating against the same + * # mongod. + * does_not_support_stepdowns, + * # $planCacheStats is not supported in transactions. + * does_not_support_transactions, * # Plan cache state is node-local and will not get migrated alongside tenant data. * tenant_migration_incompatible, - * cqf_incompatible, * ] */ (function() { @@ -30,8 +42,8 @@ assert.eq(1, coll.find({a: 1, b: 1, $or: [{c: 1}, {c: 1, d: {$eq: null}}]}).itco // Check that we have 2 distinct plans in the cache. let cacheEntries = coll.getPlanCache().list(); -assert.eq(2, cacheEntries.length); +assert.eq(2, cacheEntries.length, cacheEntries); // The query from above should still return 2 results. assert.eq(2, coll.find({a: 1, b: 1, $or: [{c: 2}, {c: 3, d: {$eq: null}}]}).itcount()); -}()); \ No newline at end of file +}()); -- cgit v1.2.3