[Test] Enable unit test suite: telemetry_sender.test.ts

[ananzh]

Description

All the unit tests related to unused telemetry are temporarily
skipped at forking. Unit tests of the disabled telemetry
functions should also be modified correspondingly. To build
a clean unit test, we decide to modify and enable all the
working unit tests. This PR modifies and enables
telemetry_sender.test.ts.

Signed-off-by: Anan Zhuang [email protected]

Issues Resolved

#498

Test results

unit test for telemetry_sender.test.ts

yarn test:jest /home/anan/work/OpenSearch-Dashboards/src/plugins/telemetry/public/services/telemetry_sender.test.ts
yarn run v1.22.10
$ node scripts/jest /home/anan/work/OpenSearch-Dashboards/src/plugins/telemetry/public/services/telemetry_sender.test.ts
 PASS  src/plugins/telemetry/public/services/telemetry_sender.test.ts
  TelemetrySender
    constructor
      ✓ defaults lastReport if unset (5 ms)
      ✓ uses lastReport if set (1 ms)
    saveToBrowser
      ✓ uses lastReport
    shouldSendReport
      ✓ returns false whenever optIn is false
      ✓ returns true if lastReported is undefined (1 ms)
      ✓ returns true if lastReported passed REPORT_INTERVAL_MS (1 ms)
      ✓ returns false if lastReported is within REPORT_INTERVAL_MS
      ✓ returns true if lastReported is malformed (1 ms)
      sendIfDue
        ✓ does not send if already sending (1 ms)
        ✓ does not send if shouldSendReport returns false (1 ms)
        ✓ sends report if due (1 ms)
        ✓ sends report separately for every cluster
        ✓ updates last lastReported and calls saveToBrowser (1 ms)
        ✓ catches fetchTelemetry errors and sets isSending to false (1 ms)
        ✓ catches fetch errors and sets isSending to false (1 ms)
    startChecking
      ✓ calls sendIfDue every 60000 ms (1 ms)

Test Suites: 1 passed, 1 total
Tests:       16 passed, 16 total
Snapshots:   0 total
Time:        3.94 s, estimated 4 s

Overall test result:

Check List

• New functionality includes testing.
  • All tests pass
• New functionality has been documented.
  • New functionality has javadoc added
• Commits are signed per the DCO using --signoff

[opensearch-ci-bot]

❌   DCO Check Failed 7f2e731
Run ./dev-tools/signoff-check.sh remotes/origin/main 7f2e731ebc52ae3cb09b31d93e151eb6384a4ab8 to check locally
Use git commit with -s to add 'Signed-of-by: {EMAIL}' on impacted commits

[opensearch-ci-bot]

✅   DCO Check Passed 4a0ad4a

[opensearch-ci-bot]

✅   DCO Check Passed 506ffb7

[ananzh]

Have cleaned the commented lines. please take another look. thanks @tmarkley @kavilla

[tmarkley]

Looks good!