update command shouldnt include multi option when false

mongodb · Feb 12, 2021 · a448317 · a448317
1 parent 1f8f371
commit a448317
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/src/operations/update.ts b/src/operations/update.ts
@@ -274,7 +274,7 @@ export function makeUpdateStatement(
     op.upsert = options.upsert;
   }
 
-  if (typeof options.multi === 'boolean') {
+  if (options.multi) {
     op.multi = options.multi;
   }
 

diff --git a/test/functional/unified-spec-runner/match.ts b/test/functional/unified-spec-runner/match.ts
@@ -129,7 +129,10 @@ export function resultCheck(
 
       if (depth > 1) {
         expect(actual, `Expected actual to exist at ${path.join('')}`).to.exist;
-        expect(Object.keys(actual)).to.include.members(Object.keys(expected));
+        expect(
+          Object.keys(actual),
+          `[${Object.keys(actual)}] length !== [${Object.keys(expected)}]`
+        ).to.have.lengthOf(Object.keys(expected).length);
       }
 
       for (const [key, value] of expectedEntries) {

diff --git a/test/functional/unified-spec-runner/runner.ts b/test/functional/unified-spec-runner/runner.ts
@@ -76,7 +76,7 @@ export async function runUnifiedTest(
       ...(test.runOnRequirements ?? [])
     ];
 
-    let doesNotMeetRunOnRequirement = allRequirements.length > 0;
+    let doesNotMeetRunOnRequirement = false;
 
     for (const requirement of allRequirements) {
       const met = await topologySatisfies(ctx.configuration, requirement, utilClient);