test(NODE-3404): revert expect.fail use in ChangeStream test

andymina · andymina · commit d2f57e87e161 · 2021-08-10T14:30:00.000-04:00
diff --git a/test/functional/change_stream.test.js b/test/functional/change_stream.test.js
@@ -2007,7 +2007,7 @@ describe('Change Streams', function () {
           if (counter === 2) {
             changeStream.close(close);
           } else if (counter >= 3) {
-            expect.fail('Should not have received more than 2 events');
+            close(new Error('Should not have received more than 2 events'));
           }
         });
         changeStream.on('error', err => close(err));

Original file line number	Diff line number	Diff line change
`@@ -2007,7 +2007,7 @@ describe('Change Streams', function () {`
`2007`	`2007`	`if (counter === 2) {`
`2008`	`2008`	`changeStream.close(close);`
`2009`	`2009`	`} else if (counter >= 3) {`
`2010`		`- expect.fail('Should not have received more than 2 events');`
	`2010`	`+ close(new Error('Should not have received more than 2 events'));`
`2011`	`2011`	`}`
`2012`	`2012`	`});`
`2013`	`2013`	`changeStream.on('error', err => close(err));`