feat: timerVariant option to choose between native and worker timers

[olso-nordsec]

Closes #1806

[codecov]

Codecov Report

Attention: Patch coverage is 78.94737% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 79.27%. Comparing base (50776a7) to head (9e956aa).

❗ Current head 9e956aa differs from pull request most recent head 8116828. Consider uploading reports for the commit 8116828 to get more accurate results

Files	Patch %	Lines
src/lib/get-timer.ts	66.66%	2 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1818      +/-   ##
==========================================
- Coverage   81.02%   79.27%   -1.76%     
==========================================
  Files          24       25       +1     
  Lines        1402     1438      +36     
  Branches      330      335       +5     
==========================================
+ Hits         1136     1140       +4     
- Misses        183      215      +32     
  Partials       83       83              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[olso-nordsec]

@robertsLando what about the codecov issue, worker case wont work in node environment 🤔

do you have a suggestion how to test it? or just dummy test case

[robertsLando]

@olso-nordsec unfortunately codecov doesn't consider the coverage of tests running on browser (with wrt, check test:browser script ), ignore that no worries. I think there is also a way to tell codecov to ignore some lines from coverage but I never digged into that

[robertsLando]

@olso-nordsec FYI: #1819

[robertsLando]

LGTM

[robertsLando]

Tests are failing now, fix them (I think last change did something wrong)

In order to only trigger pingtimer tests you can use node -r esbuild-register --test test/pingTimer.ts

[robertsLando]

@olso-nordsec The problem is that test rely on timer private var that now is named timerId so you should replace pingTimer['timer'] with pingTimer['timerId'] in test/pingTimer.ts

[robertsLando]

@olso-nordsec news here?

[olso-nordsec]

@robertsLando it was weekend, will look at it today 🙏

[robertsLando]

@olso-nordsec Yeah no worries, thanks 🙏🏼

[robertsLando]

LGTM. Thanks