Aias00 opened a new pull request, #4501: URL: https://github.com/apache/rocketmq-dashboard/pull/4501
### Which Issue(s) This PR Fixes - Fixes #4488 ### Brief Description - Keep stale queue-pull completions from deleting the in-flight lock owned by a newer request generation. - Add a deferred-promise regression covering reload, replacement pull, stale completion, and duplicate-click rejection. ### How Did You Test This Change? - `cd web && npm test -- src/components/__tests__/QueueBrowser.test.tsx` — 10 tests passed. - `cd web && npm test -- --retry=2 --reporter=dot --silent` — 122 files and 1035 tests passed. - `cd web && npm run lint` — 0 errors; 10 existing warnings. - `cd web && npm run build` — TypeScript and Vite production build passed. - `cd web && npx prettier --check src/components/QueueBrowser.tsx src/components/__tests__/QueueBrowser.test.tsx` — passed. - `git diff HEAD^ --check` — passed. Two standard full-suite diagnostic runs exposed unrelated timing-sensitive failures in ConsumerPage, Producer, and UserManagement tests. Each failing test passed repeatedly in isolation; the complete suite passed with Vitest retry enabled. ### Checklist - [x] One coherent change; unrelated modifications are not bundled in - [x] Commit subject follows Conventional Commits (`feat:` / `fix:` / `refactor:` / `chore:` / `docs:` / `perf:`) - [x] Tests added or updated for non-trivial changes, test methods named `...Test` - [x] New UI text has both Chinese and English entries under `web/src/i18n/` (no new UI text) - [x] Architecture constraints stay green (`mvn test` runs the ArchUnit checks; backend not changed) - [x] New source files carry the ASF license header (no new source files) - [x] Documentation touched where behaviour changed (no documentation change required) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
