Revert "test(NODE-6317): remove flaky unnecessary assertion (#4212)"

W-A-James · web-flow · commit 258474c3c667 · 2024-09-03T14:12:36.000-04:00
This reverts commit fb13ebf.
diff --git a/test/integration/server-discovery-and-monitoring/server_discover_and_monitoring.test.ts b/test/integration/server-discovery-and-monitoring/server_discover_and_monitoring.test.ts
@@ -109,8 +109,11 @@ describe('Monitoring rtt tests', function () {
             for (const [server, durations] of Object.entries(heartbeatDurations)) {
               const relevantDurations = durations.slice(IGNORE_SIZE);
               expect(relevantDurations).to.have.length.gt(0);
+              const averageDuration =
+                relevantDurations.reduce((acc, x) => acc + x) / relevantDurations.length;
               const rtt = client.topology.description.servers.get(server).roundTripTime;
               expect(rtt).to.not.equal(0);
+              expect(rtt).to.be.approximately(averageDuration, 3);
             }
           }
         });

Original file line number	Diff line number	Diff line change
`@@ -109,8 +109,11 @@ describe('Monitoring rtt tests', function () {`
`109`	`109`	`for (const [server, durations] of Object.entries(heartbeatDurations)) {`
`110`	`110`	`const relevantDurations = durations.slice(IGNORE_SIZE);`
`111`	`111`	`expect(relevantDurations).to.have.length.gt(0);`
	`112`	`+ const averageDuration =`
	`113`	`+ relevantDurations.reduce((acc, x) => acc + x) / relevantDurations.length;`
`112`	`114`	`const rtt = client.topology.description.servers.get(server).roundTripTime;`
`113`	`115`	`expect(rtt).to.not.equal(0);`
	`116`	`+ expect(rtt).to.be.approximately(averageDuration, 3);`
`114`	`117`	`}`
`115`	`118`	`}`
`116`	`119`	`});`