musicblocks icon indicating copy to clipboard operation
musicblocks copied to clipboard
verified publisher icon sugarlabs

Add tests for MeterBlocks.js

Open Jetshree opened this issue 2 weeks ago • 1 comments

This PR adds Jest tests for MeterBlocks to improve test coverage and assert expected behavior of meter, BPM, beat, and tempo related blocks.

Fixes: #4854

Key Changes:

  • Verifies registration of all meter block prototypes
  • Covers parameter handling, flow logic, and status matrix behavior
  • Tests BPM clamping, stacking, and cleanup for legacy and newer BPM blocks
  • Validates error handling and edge cases for missing inputs
image

Jetshree avatar Dec 18 '25 18:12 Jetshree

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

SaveInterface.test.js

github-actions[bot] avatar Dec 18 '25 18:12 github-actions[bot]

Please run prettier on the files you changed.

walterbender avatar Dec 25 '25 16:12 walterbender

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

SaveInterface.test.js

github-actions[bot] avatar Dec 25 '25 17:12 github-actions[bot]

Please run prettier on the files you changed.

@walterbender all checks pass now. Jest test failure has been observed across multiple PRs so it doesn’t appear to be specific to this PR.

Jetshree avatar Dec 25 '25 17:12 Jetshree