web: minor Timer/Timeout change for CW - Githubissues

aws / aws-toolkit-vscode

Amazon Q, CodeCatalyst, Local Lambda debug, SAM/CFN syntax, ECS Terminal, AWS resources

https://marketplace.visualstudio.com/items?itemName=AmazonWebServices.amazon-q-vscode

Apache License 2.0

1.49k stars 418 forks source link

web: minor Timer/Timeout change for CW #5258

Closed nkomonen-amazon closed 3 months ago

nkomonen-amazon commented 3 months ago

Problem

When in Web mode, we were using the Node.js Timeout refresh() method. This does not exist in web mode.

Solution

Update the use with our environment agnostic Timeout class, which works in web or node.
Copy the timeout unit tests from our regular test/ in to the web mode tests. Fixed a minor bug when refreshing in web mode, and now they all pass without any modifications to the tests.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.