fix: correctly clear timeout in `BlockCache` class

JKRhb commented 1 year ago

In the context of https://github.com/eclipse/thingweb.node-wot/pull/846, I noticed that the server is currently not cleaning up timeouts in the BlockCache class correctly. This PR implements a simple fix.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 3335895103

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+7.4%) to 61.059%

Totals
Change from base Build 3332174505:	7.4%
Covered Lines:	1913
Relevant Lines:	3059

💛 - Coveralls

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 3335911026

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+7.4%) to 61.059%

Totals
Change from base Build 3332174505:	7.4%
Covered Lines:	1913
Relevant Lines:	3059

coapjs / node-coap