diff options
Diffstat (limited to 'src/mongo/db/pipeline/document_source_lookup_test.cpp')
-rw-r--r-- | src/mongo/db/pipeline/document_source_lookup_test.cpp | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/src/mongo/db/pipeline/document_source_lookup_test.cpp b/src/mongo/db/pipeline/document_source_lookup_test.cpp index 027a06c5e15..64e12231227 100644 --- a/src/mongo/db/pipeline/document_source_lookup_test.cpp +++ b/src/mongo/db/pipeline/document_source_lookup_test.cpp @@ -215,12 +215,12 @@ TEST_F(DocumentSourceLookUpTest, LookupEmptyPipelineDoesntUseDiskAndIsOKInATrans DocumentSource::TransactionRequirement::kAllowed); } -TEST_F(DocumentSourceLookUpTest, LookupWithChildStagesInheritsDiskUseRequirement) { +TEST_F(DocumentSourceLookUpTest, LookupWithOutInPipelineNotAllowed) { + auto ERROR_CODE_OUT_BANNED_IN_LOOKUP = 51047; auto expCtx = getExpCtx(); NamespaceString fromNs("test", "coll"); expCtx->setResolvedNamespace_forTest(fromNs, {fromNs, std::vector<BSONObj>{}}); - - auto docSource = + ASSERT_THROWS_CODE( DocumentSourceLookUp::createFromBson(BSON("$lookup" << BSON("from" << "coll" << "pipeline" @@ -229,14 +229,9 @@ TEST_F(DocumentSourceLookUpTest, LookupWithChildStagesInheritsDiskUseRequirement << "as" << "as")) .firstElement(), - expCtx); - auto lookup = static_cast<DocumentSourceLookUp*>(docSource.get()); - - ASSERT_TRUE(lookup->wasConstructedWithPipelineSyntax()); - ASSERT(lookup->constraints(Pipeline::SplitState::kUnsplit).diskRequirement == - DocumentSource::DiskUseRequirement::kWritesPersistentData); - ASSERT(lookup->constraints(Pipeline::SplitState::kUnsplit).transactionRequirement == - DocumentSource::TransactionRequirement::kNotAllowed); + expCtx), + AssertionException, + ERROR_CODE_OUT_BANNED_IN_LOOKUP); } TEST_F(DocumentSourceLookUpTest, LiteParsedDocumentSourceLookupContainsExpectedNamespaces) { |