zmuxuny commented on PR #7163:
URL: 
https://github.com/apache/rocketmq-dashboard/pull/7163#issuecomment-6363179942

   CI follow-up: this PR and #7102, #7113, #7215, #7143, and #7114 now each 
carry the same five-line ConsumerPage test-isolation fix from 381c5c42 
(previously validated in #6758, with credit to pilichoumao's #6521). The 
failing manual stack/subscription tests could race the real two-second 
background polling and consume mock responses. Only those two tests now fake 
setInterval/clearInterval, and real timers are restored after cleanup. 
Production polling, all existing assertions, and the independent real 
two-second auto-refresh test are unchanged. No business changes were combined 
across PRs.
   
   Validation: all six branches passed their focused suites, changed-file 
ESLint/Prettier, a forced TypeScript build, production build, and license 
checks. Focused results were 181 tests here, 146 in #7102, 381 in #7113, 85 in 
#7215, and 59 each in #7143/#7114. This PR additionally passed the complete 
frontend suite: 162 files / 1,845 tests. Backend Java/Go code was untouched and 
was not rerun locally. New hosted CI is queued, not yet green: 
https://github.com/apache/rocketmq-dashboard/actions/runs/39941472643 .
   
   AI assistance was used for diagnosis, patch verification, and validation.


-- 
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]

Reply via email to