fix(NODE-4074): ensure getTopology doens't throw

[robertsLando]

Description

What is changing?

See https://www.mongodb.com/community/forums/t/nodejs-driver-v4-4-1-doesnt-handle-errors-correctly/151675

Is there new documentation needed for these changes?

No

What is the motivation for this change?

Fixes https://www.mongodb.com/community/forums/t/nodejs-driver-v4-4-1-doesnt-handle-errors-correctly/151675

Double check the following

• Ran npm run check:lint script
• Self-review completed using the steps outlined here
• PR title follows the correct format: <type>(NODE-xxxx)<!>: <description>
• Changes are covered by tests
• New TODOs have a related JIRA ticket

[durran]

Thanks for opening the PR. I've created a NODE ticket to track this: NODE-4074

[robertsLando]

@durran Thanks. I'm even not sure about this ones:

node-mongodb-native/src/collection.ts

Line 762 in 966ddfa

getTopology(this),

node-mongodb-native/src/collection.ts

Line 1411 in 966ddfa

getTopology(this),

BTW Feel free to edit my pr based on your needs

[robertsLando]

@durran Anyone that could give a look at this?

[baileympearson]

Hey @robertsLando - thanks for this submission! There are a couple of changes to start I'm going to make and push to your fork

• I'm going to rebase with main (there is a merge conflict)
• I'm going to fix a failing test and add more tests to cover this new scenario
• I'm going to try and get rid of the type cast as Topology in execute operation

[baileympearson]

Hey @robertsLando I'm sorry - I messed up your fork when I attempted to push my changes. I copied your work to this PR: #3172 (and added the fixes I mentioned).

[robertsLando]

@baileympearson No worries thanks for looking at this :)