fix: replace pause() with delayExecution() in updateBounds (#6038)

[piyushdotcomm]

Summary

Fixes #6038

Root Cause

Block.updateCache() contains a retry loop called updateBounds
that called that.pause(delayTime) — but pause() was never
defined on the Block class. This caused an uncaught promise
rejection that broke block rendering during collapse/expand
operations.

Fix

Replaced that.pause(delayTime) with delayExecution(delayTime)
— the same global utility that _createCache() already uses
correctly for the identical pattern.

Testing

Ran the block-related test suite — 34/34 tests pass:
Test Suites: 1 passed, 1 total
Tests: 34 passed, 34 total

PR Category

• Bug Fix
• Feature
• Performance
• Tests
• Documentation

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[Ashutoshx7]

some changes are unrelated please remove

[github-actions]

✅ All Jest tests passed! This PR is ready to merge.

[piyushdotcomm]

some changes are unrelated please remove

Thanks for pointing that out. I have removed the unrelated changes and updated the PR so that it only contains the fix for replacing pause() with delayExecution() in updateBounds.

[github-actions]

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

Failed Tests:

GraphicsBlocks.test.js

[Ashutoshx7]

@piyushdotcomm lgtm
no need to fix graphics block.test,js its not related to your pr
so it good to go
i feel this is your first pr in sugarlabs /musicblocks
welcome

[piyushdotcomm]

Thanks for reviewing and merging the PR! I appreciate the feedback and look forward to contributing more to musicblocks.