Conversation
- Changed module.exports to export { StatsWindow } for proper require() destructuring
- Added comprehensive unit test for displayInfo() method in statistics.test.js
- All tests pass locally and ESLint checks pass
|
✅ All Jest tests passed! This PR is ready to merge. |
|
This PR is a duplicate of #4972, which I submitted 5 days ago and is already passing all checks. We previously discussed this here: #4970 (comment) Furthermore, this new implementation (#5011) introduces a breaking change to the source exports in @walterbender @omsuneri Please review #4972 which is the original, non-breaking, and passing submission for this task. |
|
✅ All Jest tests passed! This PR is ready to merge. |
…GitHub issue documentation
|
✅ All Jest tests passed! This PR is ready to merge. |
|
✅ All Jest tests passed! This PR is ready to merge. |
…shooting documentation
|
✅ All Jest tests passed! This PR is ready to merge. |
|
duplicate |
3 participants
Changes
module.exports = StatsWindowtomodule.exports = { StatsWindow }to support destructured require syntax (const { StatsWindow } = require('./statistics'))displayInfo()method, validating Hz calculations and statistics formattingTesting
Related Issues
Fixes the failing ESLint check on PR that was preventing merge.