test: add unit tests for performanceTracker.js#6459
test: add unit tests for performanceTracker.js#6459eyeaadil wants to merge 1 commit intosugarlabs:masterfrom
Conversation
|
✅ All Jest tests passed! This PR is ready to merge. |
|
@walterbender If time permits you, Can You please review this PR |
There was a problem hiding this comment.
@eyeaadil the test additions look good and improve coverage
the failing check seems to be from npm audit (path-to-regexp, picomatch, yaml). I’m noticing similar failures in other PRs as well, so this might be coming from upstream dependencies rather than this PR.
@omsuneri , is this something we should treat as a repo-level issue, or is it expected for contributors to handle within their PRs?
|
This duplicates the existing PR (#6284) for |
mahesh-09-12
left a comment
There was a problem hiding this comment.
Updating review based on duplicate PR.
Description
This PR addresses issue #6458 by adding a comprehensive Jest test suite for js/utils/performanceTracker.js.
The new test file (js/utils/tests/performanceTracker.test.js) includes 32 tests covering:
Test Coverage Improvement
Related Issue
Fixes #6458