Feature/remove promise queue

athombv / node-homey-oauth2app

This module does the heavy lifting for a Homey App that talks to any OAuth2 Web API.

https://athombv.github.io/node-homey-oauth2app/

5 stars 5 forks source link

Feature/remove promise queue #70

Closed hmeerlo closed 2 years ago

hmeerlo commented 2 years ago

Removed promise-queue and added testcase for the old and new situation. The old situation is tested first in the first commit to assert that requests are queued and that the fastest request will not be finished first. Then the promise-queue is deleted and the testcase is updated to assert that all 4 requests are sent simultaneously (hence 4x the 401 error) and that the fastest request will finish first.

CLAassistant commented 2 years ago

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}