fix(property-provider): only reject if providers are not given to avoid debugger pause

[omairvaiyani]

In VSCode, when you enable the breakpoint "Uncaught exceptions", the debugger falsely pauses in chain.js. This is because the method calls Promise.reject() without immediately catching it, therefore confusing the debugger.

Issue

#2263

Description

The promise now only creates and returns a rejection when truly required. The chain behaviour is kept using an initial Promise.resolve() instance.

Testing

Against my local environment. The debugger no longer pauses and the providers are still behaving as expected.

[ghost]

This is happening on my local development instance as well. @omairvaiyani omairvaiyani did u have a workaround?

[kuhe]

https://github.com/awslabs/smithy-typescript/blob/main/packages/property-provider/src/chain.ts

should no longer be an issue since there is no longer a promise rejection happening in the chain function.

[github-actions[bot]]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.