test: allow specifying that tests require legacy topologies

mbroadst · mbroadst · commit 5434962bef31 · 2019-11-05T12:12:10.000-05:00
diff --git a/test/runner/filters/unified_filter.js b/test/runner/filters/unified_filter.js
@@ -14,9 +14,10 @@ class UnifiedTopologyFilter {
   filter(test) {
     if (!test.metadata) return true;
     if (!test.metadata.requires) return true;
-    if (!test.metadata.requires.unifiedTopology) return true;
+    if (typeof test.metadata.requires.unifiedTopology !== 'boolean') return true;
 
-    return !!process.env.MONGODB_UNIFIED_TOPOLOGY;
+    const requireUnifiedTopology = !!test.metadata.requires.unifiedTopology;
+    return requireUnifiedTopology === !!process.env.MONGODB_UNIFIED_TOPOLOGY;
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -14,9 +14,10 @@ class UnifiedTopologyFilter {`
`14`	`14`	`filter(test) {`
`15`	`15`	`if (!test.metadata) return true;`
`16`	`16`	`if (!test.metadata.requires) return true;`
`17`		`- if (!test.metadata.requires.unifiedTopology) return true;`
	`17`	`+ if (typeof test.metadata.requires.unifiedTopology !== 'boolean') return true;`
`18`	`18`
`19`		`- return !!process.env.MONGODB_UNIFIED_TOPOLOGY;`
	`19`	`+ const requireUnifiedTopology = !!test.metadata.requires.unifiedTopology;`
	`20`	`+ return requireUnifiedTopology === !!process.env.MONGODB_UNIFIED_TOPOLOGY;`
`20`	`21`	`}`
`21`	`22`	`}`
`22`	`23`